Add a new .prometheus.enabled flag (defaults true) and a .legacyPrometheus flag (defaults false) to revert to old behavior. A simple legacyPrometheus=true should revert back to backwards-compatible behavior, but we want folks to switch to the new endpoint, so that is the new default!
Also removed the default / graphite values from the values.yaml file for simplicity. As per usual, settings that a user sets in our .Values.config will take precedence over convenience helpers like prometheus.port
This PR follows closely what @colearendt did for Connect here: https://github.com/rstudio/helm/pull/516 Closes: https://github.com/rstudio/helm/issues/490
Add a new .prometheus.enabled flag (defaults true) and a .legacyPrometheus flag (defaults false) to revert to old behavior. A simple
legacyPrometheus=trueshould revert back to backwards-compatible behavior, but we want folks to switch to the new endpoint, so that is the new default!Also removed the default / graphite values from the
values.yamlfile for simplicity. As per usual, settings that a user sets in our.Values.configwill take precedence over convenience helpers likeprometheus.port