You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently, this helm chart uses the label app across resources such as here. This works great for the most part, except internally, we have tooling that relies on the Kubernetes recommended labels, specifically app.kubernetes.io/instance.
Hey @alex-bezek, that is an absolute reasonable request, thanks for taking the time to propose it.
Please do open a PR and I will review it as soon as I can. Going the route of many other charts seems very reasonable to me as well.
Currently, this helm chart uses the label
app
across resources such as here. This works great for the most part, except internally, we have tooling that relies on the Kubernetes recommended labels, specificallyapp.kubernetes.io/instance
.Would you be open to accepting a contribution that adds this label, or alternatively, add a helm value for passing additional common labels similar to how other helm charts work https://github.com/bitnami/charts/blob/main/bitnami/postgresql/values.yaml#L66
Thank you for the great app and chart, if you are open to either of these options, let me know and I can make a PR!
The text was updated successfully, but these errors were encountered: