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

dashboard resources cluster: display only namespaced resources #872

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fallmor
Copy link

@fallmor fallmor commented Oct 3, 2023

Proposal: for Memory Usage (w/o cache) dashboard show only metrics scoped with namespace.

@povilasv
Copy link
Contributor

povilasv commented Nov 8, 2023

Hey, sorry for late review. Could you elaborate why should we exclude empty ns?

@fallmor
Copy link
Author

fallmor commented Nov 8, 2023

Hey, i want to say resources that are not scoped to a namespace, i think my title is a lit bit misleading i'll rewrite it

@povilasv
Copy link
Contributor

povilasv commented Nov 8, 2023

Okay, but why do you want to filter out those?

@fallmor
Copy link
Author

fallmor commented Nov 8, 2023

Because in the memory usage panel we will see a misleading value that we will show the entire memory of the cluster and not only the one scoped in a namespace. So my merge request permit us to hide other resources in the cluster/nodes that are not attached to a namespace

@fallmor fallmor changed the title dashboard resources cluster: filter empty ns for memory dashboard resources cluster: display only namespaced resources Nov 9, 2023
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.

None yet

2 participants