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

Fix Swarm Grafana Dashboard #2640

Merged
merged 1 commit into from
Nov 27, 2023
Merged

Conversation

burdiyan
Copy link
Contributor

When using the existing Grafana setup, the newer versions of Grafana fail to provision the Swarm dashboard. It works when importing it manually, but it fails during the automatic provisioning, complaining about not being able to transform legacy templates.

Apparently the issues is in the format of the templating field of the dashboard JSON. I adapted it from the other dashboards that seem to work just fine with provisioning.

When using the existing Grafana setup, the newer versions of Grafana fail to provision the Swarm dashboard.
It works when importing it manually, but it fails during the automatic provisioning, complaining about not being able to transform legacy templates.
@sukunrt
Copy link
Member

sukunrt commented Nov 14, 2023

Thanks @burdiyan Can you tell me the grafana version it is failing with?

@burdiyan
Copy link
Contributor Author

Actually, I realized that I’m running the Grafana version from the “main” tag on Docker Hub. It’s from the same day I opened the issue. I’m not sure if the latest released version would also reproduce the issue.

But clearly the templating configuration on this dashboard is different from other dashboards, which work fine. So I just adopted the same templating setup from all the other dashboards.

@sukunrt sukunrt merged commit 0d52da2 into libp2p:master Nov 27, 2023
@sukunrt
Copy link
Member

sukunrt commented Nov 27, 2023

Thanks @burdiyan!

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