Skip to content

Commit

Permalink
OCPBUGS-39315: remove ingress operator exclusion
Browse files Browse the repository at this point in the history
  • Loading branch information
kannon92 committed Jan 7, 2025
1 parent 9ef4f33 commit f44f437
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 23 deletions.
6 changes: 0 additions & 6 deletions pkg/monitortests/node/legacynodemonitortests/exclusions.go
Original file line number Diff line number Diff line change
Expand Up @@ -22,12 +22,6 @@ func isThisContainerRestartExcluded(locator string, exclusion Exclusion) bool {
topologyToExclude string
}
exceptions := []exceptionVariants{
{
// ingress operator seems to only fail on the single topology.
// platform did not matter.
containerName: "container/ingress-operator", // https://issues.redhat.com/browse/OCPBUGS-39315
topologyToExclude: "single",
},
{
// snapshot controller operator seems to fail on SNO during kube api upgrades
// the error from the pod is the inability to connect to the kas to get volumesnapshots on startup.
Expand Down
17 changes: 0 additions & 17 deletions pkg/monitortests/node/legacynodemonitortests/exclusions_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -22,23 +22,6 @@ func Test_test_exclusions(t *testing.T) {
event: "namespace/openshift-container ... container/dummy restarted 4 times at",
expected: false,
},

{
name: "ingress-operator; single; exclude",
mockJobData: platformidentification.JobType{
Topology: "single",
},
event: "namespace/openshift-ingress-operator node/ip-10-0-113-2.us-west-1.compute.internal pod/ingress-operator-59d86c7754-br88c uid/acaf41b6-0c87-42fa-a157-b6d56e2edcb5 container/ingress-operator restarted 4 times at",
expected: true,
},
{
name: "ingress-operator; ha; include for failures",
mockJobData: platformidentification.JobType{
Topology: "HA",
},
event: "namespace/openshift-ingress-operator node/ip-10-0-113-2.us-west-1.compute.internal pod/ingress-operator-59d86c7754-br88c uid/acaf41b6-0c87-42fa-a157-b6d56e2edcb5 container/ingress-operator restarted 4 times at",
expected: false,
},
{
name: "multus; exclude for all platforms",
mockJobData: platformidentification.JobType{},
Expand Down

0 comments on commit f44f437

Please sign in to comment.