OCPEDGE-1306: feat: change minimum for arbiter ha deployments #734
Conversation
openshift-ci
bot
added
the
do-not-merge/work-in-progress
eggfoobar
force-pushed
the
add-arbiter-node-minimum
branch
from
073ca2e to
83754c1
Compare
eggfoobar
changed the title
openshift-ci-robot
added
the
jira/valid-reference
|
@eggfoobar: This pull request references OCPEDGE-1306 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.18.0" version, but no target version was set. In response to this: Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
eggfoobar
force-pushed
the
add-arbiter-node-minimum
branch
from
83754c1 to
0515f14
Compare
|
@eggfoobar: This pull request references OCPEDGE-1306 which is a valid jira issue. In response to this:
Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the openshift-eng/jira-lifecycle-plugin repository. |
eggfoobar
changed the title
openshift-ci
bot
removed
the
do-not-merge/work-in-progress
eggfoobar
force-pushed
the
add-arbiter-node-minimum
branch
from
0515f14 to
a7f5995
Compare
|
/test okd-scos-e2e-aws-ovn |
|
/lgtm |
|
/approve |
openshift-ci
bot
added
the
approved
Signed-off-by: ehila <ehila@redhat.com>
eggfoobar
force-pushed
the
add-arbiter-node-minimum
branch
from
a7f5995 to
c461a43
Compare
|
Hey @ibihim when you get a chance, can I get another lgtm on this PR, I needed to rebase the dependencies. |
|
@eggfoobar: The following tests failed, say
Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
|
/retest-required |
|
/lgtm This code hasn't changed, it was just rebased some after some dependency updates. |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: deads2k, eggfoobar, ibihim, jaypoulz 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:
Approvers can indicate their approval by writing |
|
/label acknowledge-critical-fixes-only Payloads seem to have stabilized, merging in change. |
openshift-ci
bot
added
the
acknowledge-critical-fixes-only
openshift-merge-bot
bot
merged commit c36b165
into
openshift:master
|
[ART PR BUILD NOTIFIER] Distgit: ose-cluster-authentication-operator |
6 participants
Per EP: openshift/enhancements#1674
The EP supports a new cluster topology called
HighlyAvailableArbiter, this topology means that there will be 2 regular nodes and 1 arbiter less powerful node to help maintain HA. This PR updates the minimum pod count to 2 forHighlyAvailableArbitertopologies since the pods will only be running on the full sized master nodes and not in the arbiter node.