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

Hide deployment if reality matches VPA recommendation #321

Open
Fodoj opened this issue Jul 5, 2021 · 2 comments
Open

Hide deployment if reality matches VPA recommendation #321

Fodoj opened this issue Jul 5, 2021 · 2 comments
Labels
enhancement Adding additional functionality or improvements good first issue Good for newcomers pinned Prevents stalebot from removing

Comments

@Fodoj
Copy link

Fodoj commented Jul 5, 2021

If user sets requests and limits to be exactly like VPA suggests, then would be good to have a way inside Dashboard to hide such deployment from the list - for a quicker overview of what's not yet adjusted.

@sudermanjr sudermanjr added enhancement Adding additional functionality or improvements good first issue Good for newcomers labels Jul 6, 2021
@sudermanjr
Copy link
Member

This is great idea. I would love to see a toggle that allows users to hide or show those

@sudermanjr sudermanjr added the pinned Prevents stalebot from removing label Jul 21, 2021
@milanmayr
Copy link

What about a default marker that shows that they already match? Or perhaps a different color scheme for the resource? I think it could be a good iterative change without adding a configurable option just yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Adding additional functionality or improvements good first issue Good for newcomers pinned Prevents stalebot from removing
Projects
Status: No status
Development

No branches or pull requests

3 participants