Skip to content

Commit 758a627

Browse files
committed
fix: add exception for two node fencing
add two node fencing exception to the etcd operator state transition during upgrade, in two node fencing the etcd operator will go unavailable as the two pods are updated and etcd fencing job is running via pacemaker, this is expected behavior due to the limitations of two node deployments Signed-off-by: ehila <[email protected]>
1 parent 2af38a7 commit 758a627

File tree

1 file changed

+7
-0
lines changed
  • pkg/monitortests/clusterversionoperator/legacycvomonitortests

1 file changed

+7
-0
lines changed

pkg/monitortests/clusterversionoperator/legacycvomonitortests/operators.go

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -274,6 +274,13 @@ func testUpgradeOperatorStateTransitions(events monitorapi.Intervals, clientConf
274274
strings.Contains(condition.Message, `Waiting for Deployment`) {
275275
return "csi snapshot controller is allowed to have Available=False due to CSI webhook test on two node", nil
276276
}
277+
case "etcd":
278+
if condition.Type == configv1.OperatorAvailable && condition.Status == configv1.ConditionFalse &&
279+
(strings.Contains(condition.Reason, `tnf-auth-job`) ||
280+
strings.Contains(condition.Reason, `fencing-job_JobRunning`) ||
281+
strings.Contains(condition.Message, `1 of 2 members are available`)) {
282+
return "etcd is allowed to have Available=False during two node fencing upgrade", nil
283+
}
277284
}
278285
}
279286

0 commit comments

Comments
 (0)