Skip to content

feat: FluentBit annotations via operator chart values#1986

Open
clive-jevons wants to merge 1 commit into
fluent:masterfrom
clive-jevons:feature/fluent-bit-annotations-via-operator-chart-values
Open

feat: FluentBit annotations via operator chart values#1986
clive-jevons wants to merge 1 commit into
fluent:masterfrom
clive-jevons:feature/fluent-bit-annotations-via-operator-chart-values

Conversation

@clive-jevons
Copy link
Copy Markdown

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #1985

Does this PR introduced a user-facing change?

Added ability to customize the `FluentBit.metadata.annotations` generated via the `fluent-operator` Helm chart via the values with `fluentbit.metadata.annotations`

Additional documentation, usage docs, etc.:

Example for customizing the `FluentBit` annotations:

fluentbit:
  metadata:
    annotations:
      my.org/some-annotation: some-value
      reloader.stakater.com/auto: "true"

Copilot AI review requested due to automatic review settings June 2, 2026 14:39
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adds support for user-defined annotations on the Fluent Bit (FluentBit CR) resource generated by the fluent-operator Helm chart.

Changes:

  • Introduces fluentbit.metadata.annotations in values.yaml.
  • Renders optional metadata.annotations into the FluentBit manifest template when provided.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
charts/fluent-operator/values.yaml Adds a new values entry for custom FluentBit metadata annotations.
charts/fluent-operator/templates/fluentbit-fluentBit.yaml Conditionally renders metadata.annotations from the new values entry.

Comment thread charts/fluent-operator/templates/fluentbit-fluentBit.yaml
Comment thread charts/fluent-operator/templates/fluentbit-fluentBit.yaml Outdated
@clive-jevons clive-jevons force-pushed the feature/fluent-bit-annotations-via-operator-chart-values branch from b0fc8b2 to a473d4a Compare June 2, 2026 14:57
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.

Allow specifying FluentBit.metadata.annotations via values.yaml in fluent-operator chart

2 participants