pietrogoddibit2win opened a new issue, #874: URL: https://github.com/apache/apisix-helm-chart/issues/874
The issue is in https://github.com/apache/apisix-helm-chart/commit/ba02f571913d8b7e2e8fff5f3f3fe52bb67bed1b#diff-2e4fbf14d889534cc22104278b531a5717547dfd6b87ce1c7b25ea92c75bd794 The highlighted snippet is: ```yaml lua_shared_dict: {{- range $dict := .Values.apisix.nginx.luaSharedDicts }} - {{ $dict.name }}: {{ $dict.size }} {{- end }} ``` - **YAML Structure Ambiguity:** The snippet produces a list of strings in the format `- key: value`, which is interpreted in YAML as a list of strings, not as a map/object. This could lead to confusion, especially as `lua_shared_dict` usually expects a map of key-value pairs. ### Concrete Recommendations **Use a Map Instead of a List for `lua_shared_dict`** If `lua_shared_dict` should be a map of dict names to sizes, it is clearer and more idiomatic to render it as a YAML map, not a list. For example: ```yaml lua_shared_dict: dict1: 10m dict2: 20m ``` **Helm Template Fix:** ```yaml lua_shared_dict: {{- range $dict := .Values.apisix.nginx.luaSharedDicts }} {{ $dict.name }}: {{ $dict.size }} {{- end }} ``` This approach renders a proper YAML map. To fix it change the snippet from rendering a list of strings to a YAML map. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
