cortex
cortex copied to clipboard
cortexproject
Clarify documentation and error message for format of alertmanager discovery URL
What this PR does: Edits the error message and documentation for the format of the alertmanager discovery URL for clarity.
Which issue(s) this PR fixes: There's no outstanding issue, but the issue was discussed in Slack here
Checklist
- [ ] Tests updated (n/a)
- [x] Documentation added
- [x]
CHANGELOG.mdupdated - the order of entries should be[CHANGE],[FEATURE],[ENHANCEMENT],[BUGFIX]
cc @bboreham or @ShuzZzle , any chance either of you could chime in on this? We chatted in the cortex slack a couple weeks back. I understand the code doesn't seem to align with the proposed text here, but I've confirmed the results from testing several configs.
cc @bboreham or @ShuzZzle , any chance either of you could chime in on this? We chatted in the cortex slack a couple weeks back. I understand the code doesn't seem to align with the proposed text here, but I've confirmed the results from testing several configs.
Sorry buddy, I am not a maintainer of cortex just the cortex-helm-chart
This issue has been automatically marked as stale because it has not had any activity in the past 60 days. It will be closed in 15 days if no further activity occurs. Thank you for your contributions.
![stale[bot] avatar](https://avatars.githubusercontent.com/in/1724?v=4 "Published by a pub.dev verified publisher"){kind=small}
Hey all! Just wanted to check if there's anything else I should be doing with this commit. I've seen a number of other people have issues with alertmanager config in the Cortex Slack, I think it would be great to get this in to help people who run into this misconfiguration!
This issue has been automatically marked as stale because it has not had any activity in the past 60 days. It will be closed in 15 days if no further activity occurs. Thank you for your contributions.