integr8ly / application-monitoring-operator

Operator for installing the Application Monitoring Stack on OpenShift (Prometheus, AlertManager, Grafana)
Apache License 2.0
30 stars 45 forks source link

Prometheus and Alertmanager Pod anti-affinity #152

Closed obrienrobert closed 3 years ago

obrienrobert commented 3 years ago

Description

MGDAPI-187

Added Affinity to the CR spec to allow HA deployments on multi-AZ clusters. For HA deployments, 3 Alertmanager and Prometheus pods are deployed.

A follow-on PR in the RHMI operator is required to enable Affinity via the AMO CR on installation when this has been verified.

Verification Steps

sergioifg94 commented 3 years ago

/lgtm

obrienrobert commented 3 years ago

@pb82 @grdryn @david-martin as approvers, would it be possible for one of you to take a look over and approve these changes, please?

openshift-ci-robot commented 3 years ago

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: pb82

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files: - ~~[OWNERS](https://github.com/integr8ly/application-monitoring-operator/blob/master/OWNERS)~~ [pb82] Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
sergioifg94 commented 3 years ago

/lgtm