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

[r282] Backport: Handle TSDB native histogram validation errors as soft errors (#7773) #7787

Merged
merged 1 commit into from
Apr 3, 2024

Conversation

krajorama
Copy link
Contributor

This reverts commit fb7dbaa.

  • Handle TSDB native histogram validation errors are soft errors

  • add all testcases

Signed-off-by: György Krajcsovits gyorgy.krajcsovits@grafana.com

  • Count towards discarded samples metrics with new reason

Signed-off-by: György Krajcsovits gyorgy.krajcsovits@grafana.com


Signed-off-by: György Krajcsovits gyorgy.krajcsovits@grafana.com
(cherry picked from commit 51c4088)

* Revert "Distributor: add bucket count validation to native histograms (#7736)"

This reverts commit fb7dbaa.

* Handle TSDB native histogram validation errors are soft errors

* add all testcases

Signed-off-by: György Krajcsovits <gyorgy.krajcsovits@grafana.com>

* Count towards discarded samples metrics with new reason

Signed-off-by: György Krajcsovits <gyorgy.krajcsovits@grafana.com>

---------

Signed-off-by: György Krajcsovits <gyorgy.krajcsovits@grafana.com>
(cherry picked from commit 51c4088)
@krajorama krajorama requested review from jdbaldry and a team as code owners April 3, 2024 12:10
@krajorama krajorama merged commit 4ca0d07 into r282 Apr 3, 2024
29 checks passed
@krajorama krajorama deleted the backport-7773-to-r282 branch April 3, 2024 12:47
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