espizo commented on code in PR #717:
URL: https://github.com/apache/apisix-helm-chart/pull/717#discussion_r1484786889


##########
charts/apisix/values.yaml:
##########
@@ -305,6 +305,24 @@ apisix:
     # ref: https://apisix.apache.org/docs/apisix/deployment-modes/
     role: "traditional"
 
+    # -- Standalone rules configuration
+    #
+    # ref: https://apisix.apache.org/docs/apisix/deployment-modes/#standalone
+    standalone:
+      # -- Rules which are set to the default apisix.yaml configmap.
+      # -- if apisix.delpoyment.standalone.existingConfigMap is empty, these 
are used.
+      config: |
+        routes:
+        -
+          uri: /hi
+          upstream:
+            nodes:
+              "127.0.0.1:1980": 1
+            type: roundrobin
+      # -- Specifies the name of the ConfigMap that contains the rule 
configurations.
+      # The configuration must be set to the key named `apisix.yaml` in the 
configmap.
+      existingConfigMap: ""

Review Comment:
   We need this as well. The current Helm chart adds a apisix.yaml ConfigMap 
when it is installed in standalone mode. This ConfigMap contains a /hi route 
that is impossible to modify or remove. We need the ability to specify our own 
rule config instead of this example config.
   
   One workaround would maybe be to "overwrite" the example config mount using 
`extraVolumes` and `extraVolumeMounts` but I much rather prefer this solution.



-- 
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]

Reply via email to