tulequ commented on issue #502:
URL: 
https://github.com/apache/apisix-helm-chart/issues/502#issuecomment-1666554200

   I really don't have any patience more. I will create the ServiceMonitor 
myself!
   
   As my test below, the ` serviceMonitor.namespace=monitoring` is garbage
   
   `ingress-controller.serviceMonitor.namespace=monitoring` is perfect
   
   ### Test 1
   
   ```bash
   helm template -n ingress-apisix mytest apisix/apisix --version 1.5.1 \
       --set serviceMonitor.enabled=true
   ```
   
   Result
   ```yaml
   apiVersion: monitoring.coreos.com/v1
   kind: ServiceMonitor
   metadata:
     name: mytest-apisix
     namespace: ingress-apisix
   spec:
     namespaceSelector:
       matchNames:
       - ingress-apisix
   ...
   ```
   
   ### Test 2
   ```bash
   helm template -n ingress-apisix mytest apisix/apisix --version 1.5.1 \
       --set serviceMonitor.enabled=true \
       --set serviceMonitor.namespace=monitoring
   ```
   Result:
   ```yaml
   apiVersion: monitoring.coreos.com/v1
   kind: ServiceMonitor
   metadata:
     name: mytest-apisix
     namespace: monitoring
   spec:
     namespaceSelector:
       matchNames:
       - monitoring
   ...
   ```
   


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