doehyunbaek opened a new issue, #263:
URL: https://github.com/apache/pulsar-helm-chart/issues/263

   **Is your feature request related to a problem? Please describe.**
   Setting node_exporter and alert_manager to true here does not seem to do 
anything(https://github.com/apache/pulsar-helm-chart/blob/cee3fcfe569e2a8cc87fb5f8046b29f47f1d4e39/charts/pulsar/values.yaml#L139-L141).
 
   
   Maybe it is due to copy of values.yaml file from streamnative 
charts(https://github.com/streamnative/charts/blob/0ba284e3deffa7cae9d4ea18435301f44bda9543/charts/pulsar/values.yaml#L118-L120),
 where there exists template for node_exporter and alert_manager.
   
   Related issue was once posted 
here(https://github.com/apache/pulsar-helm-chart/issues/103), but I thought 
making clear what purposes these values serve would help future pulsar users.
   
   **Describe the solution you'd like**
   I can contribute copy of templates from stremnative charts repo with 
appropriate modification.
   
   **Describe alternatives you've considered**
   If it is intended that node_exporter and alert_manager values do nothing, it 
seems there are two ways to reduce possible confusion.
   
   1. remove two values from values.yaml file.
   2. add comment that they do not do nothing.


-- 
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: dev-unsubscr...@pulsar.apache.org.apache.org

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

Reply via email to