Skip to content

Fix dashboard example #86

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

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

Fix dashboard example #86

wants to merge 2 commits into from

Conversation

jnweiger
Copy link

@jnweiger jnweiger commented May 15, 2025

This fixes two small issues with the example dashboard.

a) the pattern '^my*' does not really filter for anything starting with 'my', it accepts everything with 'm' -> use '^my' instead, without the trailing '*'.

b) the graph 'Time usage by phase' was renamed to 'Size and used space'

I am not sure how to fix the [.guihint]#Time usage by phase.#
In my 2.3.0 html rendering it is shown as "Filesystem size and used space".
This should say just "Size and used space".

Searching for this string in the checkmk-docs repo did not find anything.

The issue was seen in 2.3.0 but also seems to be in 2.4.0 and master. Please backport as appropriate.

jnweiger added 2 commits May 15, 2025 22:01
The graph is now called "Size and used space".
We need to say either '^my.*' or simply '^my' -- I suggest the latter
as this makes it clear, how these patterns work.
This one is rooted as a prefix with '^', and thus effectively a prefix match.
Copy link
Contributor

@DaniW42 DaniW42 left a comment

Choose a reason for hiding this comment

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

Had the same problem. Should be changed.

@jnweiger
Copy link
Author

I have read the CLA Document and I hereby sign the CLA or my organization already has a signed CLA.

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