[GitHub] [pulsar-helm-chart] ftertre opened a new issue #46: Grafana Ingress requires deprecated values

[GitBox <gi...@apache.org>]

ftertre opened a new issue #46:
URL: https://github.com/apache/pulsar-helm-chart/issues/46


   Hi all, a small problem with the chart and grafana ingress.
   
   **Describe the bug**
   To be able to deploy the ingress rules of Grafana component, we need to set the value of a deprecated options to true.
   
   **To Reproduce**
   Steps to reproduce the behavior:
   1. Go to `values.yaml` (L117), the `monitoring: false`has to be set to `true` to have Grafana Ingress deployed, but this value is part of the extra that have been deprecated.
   2. Check on `grafana-ingress.yaml` (L20, just after the licence), it check `.Values.extra.monitoring`
   
   **Expected behavior**
   No to use some extra values which are deprecated to be able to deploy ingress for Grafana.
   (i.e. not to have L20 and L50 of `grafana-ingress.yaml`
   
   I can do the changes and a PR if needed.


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

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

[GitHub] [pulsar-helm-chart] sijie closed issue #46: Grafana Ingress requires deprecated values

[GitBox <gi...@apache.org>]

sijie closed issue #46:
URL: https://github.com/apache/pulsar-helm-chart/issues/46


   


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

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