Skip to content

Conversation

jddcarreira
Copy link

The current helm chart allows adding custom labels, although they are not fully propagated to the deployment.

This merge requests adds the already existing variable for labels to the deployment of Fluent Operator.

Signed-off-by: Joao Carreira <[email protected]>
Signed-off-by: Joao Carreira <[email protected]>
Signed-off-by: Joao Carreira <[email protected]>
@jddcarreira jddcarreira force-pushed the addLabelsToDeployment branch from cc629d3 to b154d36 Compare July 24, 2025 15:57
@lpratas
Copy link

lpratas commented Aug 13, 2025

Hi! Can we have a review of this PR please? I'm also waiting for this change. Thank you

@jddcarreira
Copy link
Author

Any update on this one @repeatedly @jvanzyl @azuwis ?

@cw-Guo
Copy link
Collaborator

cw-Guo commented Sep 18, 2025

Thanks @jddcarreira for your contributions.

It's kind of awkward here. Current we have two places to host the charts.
The first one is in the fluent-operator repo see: https://github.com/fluent/fluent-operator/tree/master/charts/fluent-operator, mostly for development.
The second one is this helm-charts repo, mostly for release.

Can you try to raise a pull request to the https://github.com/fluent/fluent-operator/tree/master/charts/fluent-operator first?

we can then sync the changes from fluent-operator repo to this helm chart repo.

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.

3 participants