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

[Feat] support splitting single graph into multiple graphs via imagefilters #41

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

fabianbees
Copy link

@fabianbees fabianbees commented Dec 19, 2024

Summary

This change allows the Grafana graph to be split into multiple separate graphs.

This is useful, if multiple values are plotted into the same graph and their value ranges are quite different. So small value changes are much more visible.

Here is an example with and without this change:

Without change

Bildschirmfoto vom 2024-12-19 11-54-28

With change

Bildschirmfoto vom 2024-12-19 11-53-51 (Kopie)
Bildschirmfoto vom 2024-12-19 11-53-51 (Kopie 2)

Configuration

The values, which should be separated into an additional graph are specified via the Custom variable grafanaimagefiltersarray.

Each entry inside the grafanaimagefiltersarray defines a new graph and its value specifies which value should be omitted via a var-metricfilter. See the following example:
Bildschirmfoto vom 2024-12-19 11-54-49

@martialblog
Copy link
Member

Hi, thanks for the PR. I'll have a look at it.

Could you please check the failed phpcs jobs and fix the code style accordingly.

Also this change would require some new documentation in the docs/

@fabianbees
Copy link
Author

The code style checks should pass now.

Could you please check the failed phpcs jobs and fix the code style accordingly.

You can also use the following Grafana Dashboard Template see this change in action:
icinga_flux.json

@martialblog martialblog self-assigned this Dec 23, 2024
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.

2 participants