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

Add support for custom labels for RootSyncs and RepoSyncs #1380

Open
samirtahir91 opened this issue Aug 7, 2024 · 6 comments
Open

Add support for custom labels for RootSyncs and RepoSyncs #1380

samirtahir91 opened this issue Aug 7, 2024 · 6 comments

Comments

@samirtahir91
Copy link

Hi

Is there plan to add support for additional labels for RootSyncs and RepoSyncs that will reflect in their respective reconiler deployments? If not, can this be added please?

Thanks
Samir

@sdowell
Copy link
Contributor

sdowell commented Aug 7, 2024

This is not currently planned. Could you provide a bit more detail of the behavior you are looking for and some example use cases?

@samirtahir91
Copy link
Author

For example, in my org they like to label every workload so that costing can be billed to departments/teams internally.

@mikebz
Copy link
Contributor

mikebz commented Aug 8, 2024

@samirtahir91 is this billing for the workloads that Config Sync creates or Config Sync itself?

@samirtahir91
Copy link
Author

@samirtahir91 is this billing for the workloads that Config Sync creates or Config Sync itself?

The actual Root/RepoSync deployments, since there will be many in a large multi-tenant cluster scenario.

@janetkuo
Copy link
Contributor

@samirtahir91 I assume you'd want to provide team information in Root/RepoSync CR objects, correct? Would annotations instead of labels work for your case? Is labeling Deployments enough, or do you actually need to label pods?

This could potentially be supported by copying all user-specified labels from Root/RepoSync objects to their corresponding reconciler Deployments. For consistency, we might consider propagating labels to other resources that reconciler manager creates (such as service accounts) too.

Given that labeling affects how controllers select objects, this requires some design to make sure label propagation won't cause any side effects.

@samirtahir91
Copy link
Author

@janetkuo Correct, we need some way to apply labels we want on the objects created/used for reconciling in the Root/RepoSync objects themselves, in my org we use labels only.

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

No branches or pull requests

4 participants