-
Notifications
You must be signed in to change notification settings - Fork 229
Description
Bug description
In the chart's readme as well as the default provided values.yaml file, both 'podAnnotations' and 'annotations' are included. However only the 'annotations' value is read by the helm chart.
Version of Helm and Kubernetes
helm version
version.BuildInfo{Version:"v3.16.2", GitCommit:"13654a52f7c70a143b1dd51416d633e1071faffb", GitTreeState:"dirty", GoVersion:"go1.23.2"}
Client Version: version.Info{Major:"1", Minor:"25", GitVersion:"v1.25.2", GitCommit:"5835544ca568b757a8ecae5c153f317e5736700e", GitTreeState:"clean", BuildDate:"2022-09-21T14:33:49Z", GoVersion:"go1.19.1", Compiler:"gc", Platform:"darwin/arm64"}
Kustomize Version: v4.5.7
Server Version: version.Info{Major:"1", Minor:"30", GitVersion:"v1.30.10-gke.1070000", GitCommit:"9f9e8abfdc8ece6029e5e22ba0985541e4675f8e", GitTreeState:"clean", BuildDate:"2025-02-17T04:14:36Z", GoVersion:"go1.22.12 X:boringcrypto", Compiler:"gc", Platform:"linux/amd64"}
Which chart?
https://github.com/newrelic/helm-charts/tree/master/charts/synthetics-job-manager v 3.0.7
What happened?
As documented in the readme, I added the following to a local values.yaml file to annotate the synthetics job runner pods
podAnnotations: {splunk.com/index: Some_Splunk_Index}
What you expected to happen?
I expected the deployment synthetics-job-manager pods to have the annotation.
How to reproduce it?
I inspected the chart's source files and found that pod annotations could be added with the following in my values.yaml file
annotations: {splunk.com/index: Taulia_Infra_LongLived}
The chart's readme & default values.yaml file should be updated to avoid confusion for folks like me in the future.
Anything else we need to know?
Surprise us!