Skip to content
This repository has been archived by the owner on Jan 15, 2025. It is now read-only.

Using a dashboard with spaces in between words #108

Closed
codingwatermelon opened this issue Dec 15, 2017 · 3 comments
Closed

Using a dashboard with spaces in between words #108

codingwatermelon opened this issue Dec 15, 2017 · 3 comments
Assignees
Milestone

Comments

@codingwatermelon
Copy link

Hello,

I recently installed your Grafana module for Icingaweb2 (v2.5.0) (side note, awesome job!!). I'm using v1.1.10 of the module, but noticed that when configuring a graph to use a dashboard with spaces in between words (e.g "My Dashboard") in the configuration menu of the module, it doesn't seem to recognize the space(s). Therefore, it won't find the dashboard and it will give the "First parameter must either be an object or the name of an existing class" error, which refers to the dashboard name.
I also tried manually inputting spaces using the "%20" encoding, but I came across a separate error: "Cannot fetch graph with curl: 'Operation timed out after 5001 milliseconds with 0 out of -1 bytes received'."
The only workaround for this is to not use spaces (which is fine, but it may be more user-friendly if you were to implement this functionality).
I can provide more details if necessary. Also, I have a thread about this issue on monitoring-portal here: https://monitoring-portal.org/index.php?thread/42635-solved-first-parameter-must-be-error-using-mikesch-s-grafana-module-with-icingaw/&postID=260206#post260206
Let me know what you think!

@Mikesch-mp Mikesch-mp self-assigned this Dec 16, 2017
@Mikesch-mp Mikesch-mp added this to the 1.1.10 milestone Dec 16, 2017
@Mikesch-mp
Copy link
Owner

Hi,

this is normaly for Grafana. In Grafana you can create a dashboard with spaces in the name, also Grafana will Display them with a space in the name, BUT its stored inside Grafana with '-' sign. I will add code to replace also the space with - in dashboard names. Thanks for reporting this.

Regards,

Carsten

@Mikesch-mp
Copy link
Owner

Fixed it in latest Master, please test if it works now for you also.

@Mikesch-mp
Copy link
Owner

Fixed with 1.1.10 release

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants