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

fluster report: specify data source in the report link #607

Merged
merged 1 commit into from
Nov 12, 2024

Conversation

JenySadadia
Copy link
Collaborator

var-datasource=default in the fluster test summary link leads to playground rather than production on the dashboard. Specify the exact data source instead of relying on the default value.

@spbnick
Copy link
Collaborator

spbnick commented Nov 11, 2024

The default datasource has switched unexpectedly after Grafana upgrade. I fixed it for that dashboard, so there's no need for this change. Let's keep the links in emails as short as possible.

@spbnick spbnick closed this Nov 11, 2024
@spbnick
Copy link
Collaborator

spbnick commented Nov 11, 2024

Actually, you can remove this whole piece of the URLs: orgId=1&var-platform=All&var-datasource=default& Can you try that?

@spbnick
Copy link
Collaborator

spbnick commented Nov 11, 2024

You will need to at least remove var-datasource=default to fix the links, though.

@spbnick
Copy link
Collaborator

spbnick commented Nov 11, 2024

Let's reopen this, as the links are still broken because of that.

@JenySadadia
Copy link
Collaborator Author

I think it's better to specify the exact data source in the URL rather than relying on the default one.
Otherwise, it could break in the future just like it's breaking now.

@spbnick
Copy link
Collaborator

spbnick commented Nov 12, 2024

I think it's better to specify the exact data source in the URL rather than relying on the default one.
Otherwise, it could break in the future just like it's breaking now.

Sure. I'll approve this, if you prefer it this way. Feel free to merge.

However, Grafana is difficult to keep stable in general, since there's a lot manual work involved in setting everything up, and upgrades shift things, so other things may break as well. FWIW, it's better to make as little assumptions as possible about the way it works (like how datasources are identified), in case they change. That's also why I suggest making URLs as short as possible.

spbnick
spbnick previously approved these changes Nov 12, 2024
@JenySadadia
Copy link
Collaborator Author

However, Grafana is difficult to keep stable in general, since there's a lot manual work involved in setting everything up, and upgrades shift things, so other things may break as well. FWIW, it's better to make as little assumptions as possible about the way it works (like how datasources are identified), in case they change. That's also why I suggest making URLs as short as possible.

If shortening the URL is better from Grafana's pov, let's go with that.

Shorten the report link by removing the below part:
`orgId=1&var-platform=All&var-datasource=default`.

Signed-off-by: Jeny Sadadia <[email protected]>
@JenySadadia JenySadadia merged commit f4ac93d into main Nov 12, 2024
5 of 6 checks passed
@JenySadadia JenySadadia deleted the specify-datasource branch November 12, 2024 13:11
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