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

distributor: allow skipping label count validation #9576

Merged
merged 13 commits into from
Oct 11, 2024

Conversation

ortuman
Copy link
Contributor

@ortuman ortuman commented Oct 10, 2024

What this PR does

This PR introduces a new feature that enables users to bypass label count validation via the X-Mimir-SkipLabelCountValidation header.

A new configuration option, -api.skip-label-count-validation-header-enabled has been added to control this behavior. If enabled, users can set the header to true to skip label count validation.

In principle, this feature is not intended to be used by Mimir itself, but rather by projects that depend on it.

Which issue(s) this PR fixes or relates to

Fixes n/a

Checklist

  • Tests updated.
  • Documentation added.
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX].
  • about-versioning.md updated with experimental features.

@ortuman ortuman changed the title mimirpb: added new (*WriteRequest).SkipLabelCountValidation field distributor: allow skipping label count validation Oct 10, 2024
@ortuman ortuman force-pushed the ortuman/allow-skipping-label-count-validation branch 6 times, most recently from 889afb6 to 23e4d0b Compare October 10, 2024 13:04
@ortuman ortuman added enhancement New feature or request component/distributor labels Oct 10, 2024
@ortuman ortuman marked this pull request as ready for review October 10, 2024 14:56
@ortuman ortuman requested review from tacole02 and a team as code owners October 10, 2024 14:56
Copy link
Contributor

@tacole02 tacole02 left a comment

Choose a reason for hiding this comment

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

LGTM! I left a few minor docs suggestions. Thank you!

CHANGELOG.md Show resolved Hide resolved
pkg/api/api.go Outdated Show resolved Hide resolved
Copy link
Contributor

@replay replay left a comment

Choose a reason for hiding this comment

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

LGTM, thanks

Signed-off-by: Miguel Ángel Ortuño <[email protected]>
@ortuman ortuman force-pushed the ortuman/allow-skipping-label-count-validation branch from 94d04e6 to 21bda1d Compare October 11, 2024 13:28
Signed-off-by: Miguel Ángel Ortuño <[email protected]>
Signed-off-by: Miguel Ángel Ortuño <[email protected]>
@ortuman ortuman merged commit 7ee1cf5 into main Oct 11, 2024
29 checks passed
@ortuman ortuman deleted the ortuman/allow-skipping-label-count-validation branch October 11, 2024 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants