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

Fix typo in Alertmanager configuration and more tweaks #179

Merged
merged 3 commits into from
Sep 4, 2020
Merged

Conversation

gotjosh
Copy link
Contributor

@gotjosh gotjosh commented Sep 4, 2020

This PR introduces three main things for our Alertmanager configuration:

  • A small typo fix in the Alertmanager configuration alertmanger.gossip_port instead of alertmanager_gossip_port
  • A new configuration options fallback_config to allow a fallback configuration to be specified.
  • Uses a different name for the headless-service. The ClusterIP metadata field is immutable so we can't really update the spec of the service which makes it hard to switch between 1 and multiple replicas.

Checklist

  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

@gotjosh gotjosh requested a review from a team as a code owner September 4, 2020 13:50
Copy link
Contributor

@jtlisi jtlisi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@gotjosh gotjosh changed the title Fix typo in Alertmanager configuration Fix typo in Alertmanager configuration and more tweaks Sep 4, 2020
@gotjosh gotjosh force-pushed the fix-am branch 4 times, most recently from fbbb58b to 97a290d Compare September 4, 2020 17:45
- Introduces the `fallback_config` option to allow an Alertmanager to
  have a fallback config.
- Given the headless service a different name to allow seamless
  switching between 1 or multiple replicas. The cluster field in the
service metadata is immutable which made it impossible to create the new
service unless you delete the previous one.
@gotjosh gotjosh requested review from jtlisi and rfratto September 4, 2020 18:01
Copy link
Member

@rfratto rfratto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just one question

cortex/alertmanager.libsonnet Show resolved Hide resolved
@gotjosh gotjosh merged commit 83c3753 into master Sep 4, 2020
@gotjosh gotjosh deleted the fix-am branch September 4, 2020 18:15
simonswine pushed a commit to grafana/mimir that referenced this pull request Dec 20, 2021
Fix typo in Alertmanager configuration and more tweaks
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.

3 participants