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

Make route.Key() unique and remove route.ID() #3809

Conversation

grobinson-grafana
Copy link
Contributor

This pull request replaces the code in route.Key() with that of route.ID(), and removes route.ID(). The motivation behind this change is to fix a number of bugs caused by conflicting group keys such as "Different aggregation groups can share the same nflog" (#3808) and also prevent an issue where groups are incorrectly marked as muted when they are not.

This commit replaces the code in route.Key() with that of route.ID(),
and removes route.ID(). The motivation behind this change is to fix
a number of bugs caused by conflicting group keys such as
"Different aggregation groups can share the same nflog" (prometheus#3808)
and also prevent an issue where groups are incorrectly marked as muted
when they are not.

Signed-off-by: George Robinson <george.robinson@grafana.com>
@grobinson-grafana
Copy link
Contributor Author

I'm closing this as @gotjosh found this can have unintended side effects when a configuration is changed. For example, if a route is moved in the configuration file, but not edited, then it's index will change and cause its nflogs to be expired.

@grobinson-grafana grobinson-grafana deleted the grobinson/replace-route-key-with-route-id branch April 24, 2024 14:24
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

1 participant