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

GC should run in master so that it can use informer cache #2021

Open
ivelichkovich opened this issue Jan 15, 2025 · 1 comment
Open

GC should run in master so that it can use informer cache #2021

ivelichkovich opened this issue Jan 15, 2025 · 1 comment
Labels
kind/feature Categorizes issue or PR as related to a new feature.

Comments

@ivelichkovich
Copy link

What would you like to be added:

GC running using gocron within master pod and leveraging it's informer cache

Why is this needed:

reduce memory overhead from the additional pod and load on api-server in high scale environments

@ivelichkovich ivelichkovich added the kind/feature Categorizes issue or PR as related to a new feature. label Jan 15, 2025
@marquiz
Copy link
Contributor

marquiz commented Jan 22, 2025

I think this idea is worth exploring and the concerns are valid. GC does NodeResourceTopology CRs too, so that functionality should be moved to nfd-topology-updater before ditching nfd-gc entirely.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes issue or PR as related to a new feature.
Projects
None yet
Development

No branches or pull requests

2 participants