-
Notifications
You must be signed in to change notification settings - Fork 598
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
Adding rulebook_url also affects external alerts #40
Comments
Hi @serathius! Thanks for the report. Can confirm this is not the intention - I was under the impression the merge ordering would fix this but looks like we've got the same problem. I'll give it some thought. |
Your right, my lack of jsonnet knowledge here. Order is important here. |
Sorry my comment was not clear: I was under the impression the merge ordering was important, but we have the same problem, and we're merging in the "right" order. So it looks like merge ordering isn't important. I need to do some more investigation. |
This issue has not had any activity in the past 30 days, so the
Thank you for your contributions! |
Automatic rulebook_url generation runs as mixin without any condition.
So when vendoring
kubernetes-mixin
, my own alerts are also affected by this.The text was updated successfully, but these errors were encountered: