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

Change Error Messages in TSDB Status Page #8659

Merged
merged 2 commits into from Mar 28, 2021

Conversation

LeviHarrison
Copy link
Member

This PR changes the wording when there are no datapoints to No datapoints left instead of the blanket Error parsing time. Another test is also added and both are moved into the correct testing suite.

Follow up to #8654 as proposed here.

Signed-off-by: Levi Harrison <git@leviharrison.dev>
Signed-off-by: Levi Harrison <git@leviharrison.dev>
@juliusv
Copy link
Member

juliusv commented Mar 28, 2021

👍 Thanks!

@juliusv juliusv merged commit 054e9d1 into prometheus:main Mar 28, 2021
@LeviHarrison LeviHarrison deleted the blank-tsdb-status-2 branch March 28, 2021 13:07
vatine pushed a commit to vatine/prometheus that referenced this pull request Mar 31, 2021
* Changed error wording

Signed-off-by: Levi Harrison <git@leviharrison.dev>

* Added another test and moved both into the correct suite

Signed-off-by: Levi Harrison <git@leviharrison.dev>
Signed-off-by: Ingvar <imagineaclevernamehere@gmail.com>
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

2 participants