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: us...@infra.apache.org
