Skip to content
This repository was archived by the owner on Apr 28, 2025. It is now read-only.

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

@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 subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants