[GitHub] [pulsar-helm-chart] ckdarby commented on issue #9: Grafana dashboards not entirely working

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

ckdarby commented on issue #9:
URL: https://github.com/apache/pulsar-helm-chart/issues/9#issuecomment-625304530


   >Node metrics
   This repo doesn't have the node_exporter.yaml template like Streamnative does.
   
   >Proxy metrics
   Yes, we enabled authentication. I do see in the Streamnative handling the metric endpoint for JWT but this repo doesn't. So, you are correct about the back push.
   
   > Pulsar Logs
   Good to know :)
   
   Are there efforts for consolidation with Streamnative's charts & this official chart? I see the Streamnative charts has a lot of functionality that would be nice to end up in here as well eventually. I'm hopeful that Streamnative's changes end up here and then Streamnative can use an "umbrella" helm to add this as a dependency and keep their custom only required stuff there.
   
   As PMC for Pulsar if you haven't already can you loop in the rest of the contributions to make sure we minimize the changes happening in apache/pulsar for helm? Would prefer to not have massive PR "upstream" have to also be merged here and also trying to pull in Streamnative's version into this :P


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