Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[influxdb] Support configurable StatefulSet labels/annotations #666

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lovetoburnswhen
Copy link

@lovetoburnswhen lovetoburnswhen commented Aug 20, 2024

Adds the statefulSetAnnotations/statefulSetLabels properties to values.yaml to allow users to specify custom labels and annotations on the resulting StatefulSets.

This feature would be tremendously useful for environments that require certain labels to be set - the only alternative would be to fork the chart internally.


Tested with:

❯ helm template ./charts/influxdb -s templates/statefulset.yaml --set 'statefulSetLabels.foo=bar' --set 'statefulSetAnnotations.bar=baz'      
---
# Source: influxdb/templates/statefulset.yaml
apiVersion: apps/v1
kind: StatefulSet
metadata:
  name: release-name-influxdb
  labels:
    helm.sh/chart: influxdb-4.12.6
    app.kubernetes.io/name: influxdb
    app.kubernetes.io/instance: release-name
    app.kubernetes.io/version: "1.8.10"
    app.kubernetes.io/managed-by: Helm
    foo: bar
  annotations:
    bar: baz

  • [ ] CHANGELOG.md updated None found
  • Rebased/mergable
  • Tests pass
  • Sign CLA (if not already signed)

Adds the `statefulSetAnnotations`/`statefulSetLabels` properties to
`values.yaml` to allow users to specify custom labels and annotations on
the resulting StatefulSets.

This feature would be tremendously useful for environments that require
certain labels to be set - the only alternative would be to fork the
chart internally.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant