Skip to content

Conversation

danielblando
Copy link
Contributor

@danielblando danielblando commented Mar 14, 2025

What this PR does:
Ingester are expected to receive labels ordered. This test is ingesting labels not ordered. Nit fixing it.

Which issue(s) this PR fixes:
Fixes #

Checklist

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

Signed-off-by: Daniel Deluiggi <[email protected]>
@danielblando danielblando marked this pull request as ready for review March 14, 2025 16:13
Copy link
Member

@alanprot alanprot left a comment

Choose a reason for hiding this comment

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

thanks

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Mar 14, 2025
@danielblando danielblando merged commit 0d78ead into cortexproject:master Mar 14, 2025
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/ingester lgtm This PR has been approved by a maintainer size/XS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants