michaeljmarshall opened a new pull request, #299:
URL: https://github.com/apache/pulsar-helm-chart/pull/299

   Related issues #294 #65
   
   Supersedes #296 and #297
   
   ### Motivation
   
   Our helm chart is out of date. I propose we make a breaking change for the 
monitoring solution and start using the `kube-prometheus-stack` as a 
dependency. This should make upgrades easier and will let users leverage all of 
that chart's features.
   
   This change will result in the removal of the StreamNative Grafana 
Dashboards. We'll need to figure out the right way to address that. The 
apache/pulsar project has grafana dashboards, but they have not been 
maintained. With this added dependency, we'll have the benefit of being able to 
use k8s `ConfigMap`s to configure grafana dashboards.
   
   ### Modifications
   
   * Remove old prometheus and grafana configuration
   * Add kube-prometheus-stack chart as a dependency
   * Enable several components by default. I am not opinionated on these, but 
it is based on the other values in the chart.
   
   ### Verifying this change
   
   This is a large change that will require manual validation, and may break 
deployments. I propose this triggers a helm chart 3.0.0 release.
   


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

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

Reply via email to