Skip to content

Conversation

GiedriusS
Copy link
Contributor

Ensure that the labels are in order because we are iterating over map[string]string which has a random sorting order. I think the consequence of not sorting this is that when using a frontend directly, the write requests might have unsorted labels in them.

Ensure that the labels are in order because we are iterating over
map[string]string which has a random sorting order. I think the
consequence of not sorting this is that when using a frontend directly,
the write requests might have unsorted labels in them.

Signed-off-by: Giedrius Statkevičius <[email protected]>
@dosubot dosubot bot added the component/rules Bits & bobs todo with rules and alerts: the ruler, config service etc. label Apr 1, 2025
@SungJin1212
Copy link
Member

Thanks

Copy link
Contributor

@yeya24 yeya24 left a comment

Choose a reason for hiding this comment

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

Thanks. Make sense to me!

@yeya24 yeya24 merged commit 3c6be6d into cortexproject:master Apr 3, 2025
17 checks passed
wilguo pushed a commit to wilguo/cortex that referenced this pull request Apr 10, 2025
Ensure that the labels are in order because we are iterating over
map[string]string which has a random sorting order. I think the
consequence of not sorting this is that when using a frontend directly,
the write requests might have unsorted labels in them.

Signed-off-by: Giedrius Statkevičius <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/rules Bits & bobs todo with rules and alerts: the ruler, config service etc. size/S
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants