Skip to content

Update Grafana dashboards#1339

Merged
Zerpet merged 1 commit intomainfrom
zerpet/update-dashboards
May 4, 2023
Merged

Update Grafana dashboards#1339
Zerpet merged 1 commit intomainfrom
zerpet/update-dashboards

Conversation

@Zerpet
Copy link
Copy Markdown
Member

@Zerpet Zerpet commented Apr 28, 2023

This closes #1334

Note to reviewers: remember to look at the commits in this PR and consider if they can be squashed

Summary Of Changes

Use a perma-link to Grafana Dashboards in rabbitmq-server repo.

Additional Context

Grafana.com is rejecting HTTP requests with basic authentication. Earlier, it used to ignore the basic auth, and simply serve the request. This new behaviour in Grafana API breaks the sidecar import feature of Grafana's Helm Chart. The sidecar from Grafana uses basic authentication in their HTTP request. It then gets a not-authorized response from Grafana API, and dashboard importing fails.

GitHub raw URLs ignore the basic auth and simply serve the file as a static element.

Local Testing

N/A - no code changes

Edit: formatting

Grafana.com is rejecting HTTP requests with basic authentication.
Earlier, it used to ignore the basic auth, and simply serve the request.
This new behaviour in Grafana API breaks the sidecar import feature of
Grafana's Helm Chart. The sidecar from Grafana uses basic authentication
in their HTTP request. It then gets a not-authorized response from
Grafana API, and dashboard importing fails.

GitHub raw URLs ignore the basic auth and simply serve the file as a
static element.

Related to #1334

Signed-off-by: Aitor Perez Cedres <acedres@vmware.com>
@Zerpet Zerpet self-assigned this Apr 28, 2023
@Zerpet Zerpet requested a review from ansd April 28, 2023 16:57
@Zerpet Zerpet merged commit 2f99095 into main May 4, 2023
@Zerpet Zerpet deleted the zerpet/update-dashboards branch May 4, 2023 10:17
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.

Grafana dashboard import process is broken with Grafana Helm Chart

1 participant