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

Switch to use the new configmaps informer for retrieving configmaps #7304

Open
terrytangyuan opened this issue Nov 30, 2021 · 2 comments
Open
Labels
area/controller Controller issues, panics type/tech-debt

Comments

@terrytangyuan
Copy link
Member

terrytangyuan commented Nov 30, 2021

Summary

What change needs making?

Switch to use the new configmaps informer added in #6644 for retrieving all types of configmaps we need. Parameters loading is already using it #6662 and cache GC soon #6850.

Need to provide a migration path.

Use Cases

When would you use this?


Message from the maintainers:

Love this enhancement proposal? Give it a 👍. We prioritise the proposals with the most 👍.

@terrytangyuan terrytangyuan added the type/feature Feature request label Nov 30, 2021
@alexec alexec added area/controller Controller issues, panics type/tech-debt and removed type/feature Feature request labels Feb 7, 2022
@terrytangyuan terrytangyuan self-assigned this Feb 8, 2022
@stale

This comment was marked as resolved.

@stale stale bot added the problem/stale This has not had a response in some time label Mar 2, 2022
@terrytangyuan

This comment was marked as resolved.

@stale stale bot removed the problem/stale This has not had a response in some time label Mar 2, 2022
@terrytangyuan terrytangyuan removed their assignment Oct 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/controller Controller issues, panics type/tech-debt
Projects
None yet
Development

No branches or pull requests

2 participants