Skip to content

[helm] StatefulSets and main services should support annotations #3688

Description

@fresh-borzoni

Search before asking

  • I searched in the issues and found nothing similar.

Description

The chart is inconsistent about annotation hooks: the metrics services take them (metrics.prometheus.service.annotations), pods take them (podAnnotations), but the statefulset metadata and the main headless services take none at all.

Workload and service annotations are where a lot of standard tooling hooks in - Reloader for restart-on-secret-rotation, Argo CD sync-waves, external-dns, org-mandated ownership annotations enforced by policy engines. Today the only way to attach any of these is patching the rendered output out-of-band, which defeats values.yaml as the single source of truth.

Add coordinator.annotations / tablet.annotations for the statefulsets and the same under service: for the headless services, matching the existing podAnnotations style.

Willingness to contribute

  • I'm willing to submit a PR!

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    Fields

    No fields configured for Task.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions