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

vm-status: adapt for new grafana panels #10

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

Conversation

leona-ya
Copy link
Member

This changes the panels of the vm-status dashboard for compatibility with future Grafana versions (the old ones are currently deprecated). It has almost no UI changes.
There's one intentional change: The left axis of the "Conntrack Table Fullness" Panel now has a scale 0-100%, so that it's a better visual indicator if there's a problem.

screenshot-2024-02-25_18-02-1708883224
screenshot-2024-02-25_18-02-1708883230

Current state

I tested this on my private grafana, but not on a FCIO one. ToDos:

  • Test on FCIO instance, Also whether the remaining panels work
  • Test for upgradeablity on customer instances
  • Communicate the change of the Conntrack table graph(?) – when we decide to change this.

Change conntrack table fullness scale to 0-100% instead of dynamic.
@ctheune ctheune requested a review from zagy February 28, 2024 09:54
@ctheune
Copy link
Contributor

ctheune commented Feb 28, 2024

I'm a bit lost here with the process. @zagy signaled that this is gnarly ... -_-

@zagy
Copy link
Member

zagy commented Feb 29, 2024

The export seems to have become better so that there is no hostname in there by default. Checking.

@zagy
Copy link
Member

zagy commented Feb 29, 2024

Hm … i can't get it imported, yet. So, later.

@leona-ya
Copy link
Member Author

I already manipulated the export :/ The default is really dumb (e.g inlcudes hostname, data source identifier)

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.

3 participants