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

The limit ruler.max-rule-groups-per-tenant-by-namespace looks at the rule grous in all namespace #9130

Open
gotjosh opened this issue Aug 29, 2024 · 0 comments

Comments

@gotjosh
Copy link
Contributor

gotjosh commented Aug 29, 2024

In #8425, we introduced this limit as an override to the value defined in RulerMaxRuleGroupsPerTenant. However, I have realised that it might be causing more confusion because the expectation seems to be that this is a new value that is JUST for the particular namespace.

As worded, the limit might seem to indicate the limit of rule groups will only apply to the rule groups in the defined namespace and not globally.

Allow me to explain. Assuming tenant user-1 has a total of 25 rule groups across all namespaces and is set with a limit of RulerMaxRuleGroupsPerTenant = 50, if we set something like:

ruler_max_rule_groups_per_tenant_by_namespace: {
  newnamespace: 24,
},

This is effectively a downgrade, and we'll be unable to put rules into the new namespace despite the user having plenty of room.

I think the limit should only count rule groups in the given namespace.

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

No branches or pull requests

1 participant