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

Add rule to check default time and refresh interval #76

Closed
wants to merge 3 commits into from

Conversation

mshahzeb
Copy link
Contributor

@mshahzeb mshahzeb commented Jun 20, 2022

Reference issue:
#52

This rule checks if each dashboard has:

  • A default time interval of 1h
  • A default refresh interval of 5m

image

@CLAassistant
Copy link

CLAassistant commented Jun 20, 2022

CLA assistant check
All committers have signed the CLA.

Copy link
Collaborator

@rgeyer rgeyer left a comment

Choose a reason for hiding this comment

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

The code looks good, however..

This is very opinionated, and does not necessarily improve dashboards in any tangible way.

In fact, for some dashboards with slow moving data (SCM releases which happen over hours, days, or weeks) these defaults wouldn't make any sense.

We use these defaults across all of the mixins/integrations for Grafana Cloud for the sake of uniformity, but we needn't necessarily impose this on all dashboards for all users.

What do you think?
@gaantunes @v-zhuravlev

@mshahzeb
Copy link
Contributor Author

I guess this is more of an integration specific rule rather than a generic dashboard rule.

I think we are already doing that here in utils.libsonnet:
https://github.com/grafana/cloud-onboarding/blob/7473d9783afbf1a0a205b58c70ed8fe6be3ca8a1/integrations/util.libsonnet#L19

But its setting it to 30m and 30s refresh not 1h and 5m interval.

@rgeyer
Copy link
Collaborator

rgeyer commented Jun 27, 2022

I guess this is more of an integration specific rule rather than a generic dashboard rule.

I think we are already doing that here in utils.libsonnet: https://github.com/grafana/cloud-onboarding/blob/7473d9783afbf1a0a205b58c70ed8fe6be3ca8a1/integrations/util.libsonnet#L19

But its setting it to 30m and 30s refresh not 1h and 5m interval.

Right. We should discuss what the expectation is here, and resolve in cloud-onboarding. This one PR can be closed tho.

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.

None yet

3 participants