Skip to content

Unable to edit alerts config if a fallback routing config is in use #591

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

Closed
leth opened this issue Nov 9, 2017 · 1 comment
Closed

Unable to edit alerts config if a fallback routing config is in use #591

leth opened this issue Nov 9, 2017 · 1 comment
Assignees
Labels

Comments

@leth
Copy link
Contributor

leth commented Nov 9, 2017

Identified by debugging a customer issue. The issue is somewhere around https://github.com/weaveworks/cortex/blob/master/pkg/alertmanager/multitenant.go#L440

@bboreham
Copy link
Contributor

The context was that we saw messages like these when the user saved alerts:

time="2017-11-08T22:53:31Z" level=warning msg="MultitenantAlertmanager: invalid Cortex configuration for 5570: error parsing Alertmanager config: no route provided in config" source="multitenant.go:377"
time="2017-11-08T22:53:37Z" level=warning msg="MultitenantAlertmanager: invalid Cortex configuration for 5570: error parsing Alertmanager config: no route provided in config" source="multitenant.go:377"
time="2017-11-08T23:03:29Z" level=warning msg="MultitenantAlertmanager: invalid Cortex configuration for 5570: error parsing Alertmanager config: no route provided in config" source="multitenant.go:377"
time="2017-11-08T23:03:31Z" level=warning msg="MultitenantAlertmanager: invalid Cortex configuration for 5570: error parsing Alertmanager config: no route provided in config" source="multitenant.go:377"
time="2017-11-08T23:03:37Z" level=warning msg="MultitenantAlertmanager: invalid Cortex configuration for 5570: error parsing Alertmanager config: no route provided in config" source="multitenant.go:377"

But alerts are generated by ruler, which doesn't go through that code path.
I don't think MultitenantAlertmanager uses the RulesFiles member.
So this is an unfortunate log message but not actually a problem.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants