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

grafana: replace broken heap usage gauge #2073

Merged

Conversation

valentijnscholten
Copy link
Contributor

The current heap usage gauge is not working correctly. This is because Grafana doesn't support variables in data transformations.
And we're using a transformation to dynamically set the Max value of the gauge, but this fails when using the $instance variable. It only works if the instance name is hardcoded (or left out). It currently is set hardcoded to apiserver:8080 which probably won't work for most realworld deployments.

This PR switches from a gauge to two stat panels giving similar information. The colours are always green as again if we wanted thresholds and a max value, we would run into the same problems.

image

If you want colours for heap usage, you can use the micrometer dashboard which shows the percentage of heap used in green - orange - red styles.

Signed-off-by: Valentijn Scholten [email protected]

Signed-off-by: Valentijn Scholten <[email protected]>
@valentijnscholten valentijnscholten changed the title grafana: replace heap usage gauage grafana: replace broken heap usage gauge Oct 21, 2022
@sonatype-lift
Copy link
Contributor

sonatype-lift bot commented Oct 21, 2022

⚠️ 16 God Classes were detected by Lift in this project. Visit the Lift web console for more details.

Copy link
Member

@nscuro nscuro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@nscuro nscuro added this to the 4.7 milestone Oct 21, 2022
@nscuro nscuro added the documentation Improvements or additions to documentation label Oct 21, 2022
@nscuro nscuro merged commit ecbe957 into DependencyTrack:master Oct 21, 2022
@valentijnscholten
Copy link
Contributor Author

For future reference: This is latest state of variable support in Transformations: grafana/grafana#46428

@valentijnscholten valentijnscholten deleted the grafana-replace-heap-gauge branch November 5, 2022 13:22
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 6, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants