You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@apisix.apache.org by "jieqian (via GitHub)" <gi...@apache.org> on 2023/02/03 01:57:02 UTC

[GitHub] [apisix-helm-chart] jieqian opened a new issue, #457: apisix chart中configmap模板，渲染“kubernetes服务发现”的minimal配置会换行，导致apisix起来后读取配置

jieqian opened a new issue, #457:
URL: https://github.com/apache/apisix-helm-chart/issues/457

   ### Current Behavior
   
   branch: master
   file: https://github.com/apache/apisix-helm-chart/blob/master/charts/apisix/templates/configmap.yaml
   line num: 200 ~ 204
   
   按照https://github.com/apache/apisix-helm-chart/blob/master/charts/apisix/values.yaml里comment配置，minimal的kubernetes服务发现：
   ```
   discovery:
     enabled: true
     registry: 
       kubernetes: {}
   ```
   渲染出来的config.yaml中“{}”换行了，导致lua解析yaml失败
   ```
   discovery:
     kubernetes:
       {}
   ```
   
   
   ### Expected Behavior
   
   ```
   discovery:
     kubernetes: {}
   ```
   
   ### Error Logs
   
   _No response_
   
   ### Steps to Reproduce
   
   is as mentioned in "Current Behavior"
   
   ### Environment
   
   apisix installed via helm


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [apisix-helm-chart] tao12345666333 commented on issue #457: apisix chart中configmap模板，渲染“kubernetes服务发现”的minimal配置会换行，导致apisix起来后读取配置

Posted by "tao12345666333 (via GitHub)" <gi...@apache.org>.
tao12345666333 commented on issue #457:
URL: https://github.com/apache/apisix-helm-chart/issues/457#issuecomment-1501120677

   https://github.com/apache/apisix-helm-chart/pull/526 will fix this issue


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [apisix-helm-chart] tokers commented on issue #457: apisix chart中configmap模板，渲染“kubernetes服务发现”的minimal配置会换行，导致apisix起来后读取配置

Posted by "tokers (via GitHub)" <gi...@apache.org>.
tokers commented on issue #457:
URL: https://github.com/apache/apisix-helm-chart/issues/457#issuecomment-1414689766

   The template breaks the line here(https://github.com/apache/apisix-helm-chart/blob/master/charts/apisix/templates/configmap.yaml#L203). Maybe we can have a check if the object is empty.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [apisix-helm-chart] yswtrue commented on issue #457: apisix chart中configmap模板，渲染“kubernetes服务发现”的minimal配置会换行，导致apisix起来后读取配置

Posted by "yswtrue (via GitHub)" <gi...@apache.org>.
yswtrue commented on issue #457:
URL: https://github.com/apache/apisix-helm-chart/issues/457#issuecomment-1501080005

   我照着文档配置好像也是报错的
   ```
   /usr/local/openresty//luajit/bin/luajit ./apisix/cli/apisix.lua init
   --
   Sun, Apr 9 2023 4:48:34 pm | /usr/local/openresty//luajit/bin/luajit: /usr/local/apisix/deps/share/lua/5.1/tinyyaml.lua:707: failed to classify line: {}
   Sun, Apr 9 2023 4:48:34 pm | stack traceback:
   Sun, Apr 9 2023 4:48:34 pm | [C]: in function 'error'
   Sun, Apr 9 2023 4:48:34 pm | /usr/local/apisix/deps/share/lua/5.1/tinyyaml.lua:707: in function 'parsemap'
   Sun, Apr 9 2023 4:48:34 pm | /usr/local/apisix/deps/share/lua/5.1/tinyyaml.lua:743: in function 'parsemap'
   Sun, Apr 9 2023 4:48:34 pm | /usr/local/apisix/deps/share/lua/5.1/tinyyaml.lua:743: in function 'parsemap'
   Sun, Apr 9 2023 4:48:34 pm | /usr/local/apisix/deps/share/lua/5.1/tinyyaml.lua:793: in function 'parsedocuments'
   Sun, Apr 9 2023 4:48:34 pm | /usr/local/apisix/deps/share/lua/5.1/tinyyaml.lua:818: in function 'parse'
   Sun, Apr 9 2023 4:48:34 pm | /usr/local/apisix/apisix/cli/file.lua:241: in function 'read_yaml_conf'
   Sun, Apr 9 2023 4:48:34 pm | /usr/local/apisix/apisix/cli/ops.lua:174: in function </usr/local/apisix/apisix/cli/ops.lua:157>
   Sun, Apr 9 2023 4:48:34 pm | /usr/local/apisix/apisix/cli/ops.lua:955: in function 'execute'
   Sun, Apr 9 2023 4:48:34 pm | ./apisix/cli/apisix.lua:40: in main chunk
   Sun, Apr 9 2023 4:48:34 pm | [C]: at 0x557ed710c3d0
   
   ```
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [apisix-helm-chart] tao12345666333 closed issue #457: apisix chart中configmap模板，渲染“kubernetes服务发现”的minimal配置会换行，导致apisix起来后读取配置

Posted by "tao12345666333 (via GitHub)" <gi...@apache.org>.
tao12345666333 closed issue #457: apisix chart中configmap模板，渲染“kubernetes服务发现”的minimal配置会换行，导致apisix起来后读取配置
URL: https://github.com/apache/apisix-helm-chart/issues/457


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org