-
-
Notifications
You must be signed in to change notification settings - Fork 240
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
Configurable Overview page #614
Conversation
Hm, I need to fix the tests... |
I also forgot to update |
Oups, disabling |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is fine, but the decreasing test coverage should be fixed
Thanks @sebastianrakel , but I have to make it work with RESOURCES_STATS_ENABLED=True and NODES_STATUS_DETAIL_ENABLED=False without actually all the processing that would be done if NODES_STATUS_DETAIL_ENABLED=True. |
Implements #544 and #533.
Firstly I moved the "population" - number of nodes - to the same row as other metrics that show the number of nodes in various states.
Then I added 2 new configuration settings to allow disabling 2 sections of that page:
Both are set to
True
by default for backward compatibility.I also organized the docs for all the settings in the README.