Skip to content
This repository has been archived by the owner on Jul 19, 2023. It is now read-only.

Datasource: Min step in config page #269

Merged
merged 3 commits into from
Oct 3, 2022
Merged

Conversation

aocenas
Copy link
Member

@aocenas aocenas commented Sep 30, 2022

Fixes: #238
Adds a min step setting in the config page of the data source. This is needed because similar to Prometheus fire has a set scrape interval and it does not make sense to try to query with a step smaller than that as there are no data points.
Screenshot from 2022-09-30 10-43-18

}}
/>
}
tooltip="Minimal step used for metric query. Should be the same or higher as the scrape interval setting in the Fire database."
Copy link
Member Author

Choose a reason for hiding this comment

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

@cyriltovena @simonswine does this make sense? Also in the Prometheus we call this scrape interval but from that, it's not really clear why it is needed and what is grafana using it for. So to me, it seems more clear to say this is Min interval for the query (which also, may not be clear what that actually is) and just add the tooltip saying this should be the same as scrape interval but that is sort of subjective for me so let me know what you think.

Copy link
Collaborator

Choose a reason for hiding this comment

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

That sounds correct to me 👍.

Maybe in the future we can like the config setting in the Fire docs there as well.

Copy link
Collaborator

@simonswine simonswine left a comment

Choose a reason for hiding this comment

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

LGTM

@simonswine
Copy link
Collaborator

I think this is ready to go I have tried it for very short time scales and the graph looks a lot better now 🙂

@aocenas aocenas merged commit 23eae36 into main Oct 3, 2022
@aocenas aocenas deleted the aocenas/datasource/min-step branch October 3, 2022 12:24
simonswine pushed a commit to simonswine/pyroscope that referenced this pull request Jun 30, 2023
Datasource: Min step in config page
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Datasource minimun scraping interval.
3 participants