Skip to content

Add deployment labels to workers#256

Merged
golfdish merged 4 commits intotryretool:mainfrom
AlayaCare:fix-workers-deployement-labels
Oct 30, 2025
Merged

Add deployment labels to workers#256
golfdish merged 4 commits intotryretool:mainfrom
AlayaCare:fix-workers-deployement-labels

Conversation

@T0RAT0RA
Copy link
Contributor

Hi,

I'd like to bring the $.Values.deployment.labels to the workers deployements, similare to the backend deployment: deployment_backend.yaml#L8-L10

Thank you.

@golfdish
Copy link
Contributor

golfdish commented Oct 28, 2025

Looks fine to me, but could you say a little more about your use case (what is this fixing)?

@T0RAT0RA
Copy link
Contributor Author

We're rolling out a new feature that will enforce labels on all our k8s resources, using Gatekeeper, making sure they're appropriately tagged (resource owner, environment, slack-channel, etc...)

So I need to be able to label retool deployments

@T0RAT0RA
Copy link
Contributor Author

Thanks for the review!
Would you know when can this get released?

@golfdish
Copy link
Contributor

Thanks for the review! Would you know when can this get released?

I'll push the new chart version today after merging.

@golfdish golfdish merged commit e0eed81 into tryretool:main Oct 30, 2025
@golfdish
Copy link
Contributor

@T0RAT0RA the new release has been published!

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.

2 participants