Skip to content

Commit f06eec0

Browse files
Merge pull request #30430 from aabughosh/revert-30424-flakeports
CNF-18661: Revert " flake the missing port 9194 and 9193 once the team will fix will remove it"
2 parents e603cd9 + 99cc585 commit f06eec0

File tree

1 file changed

+0
-17
lines changed

1 file changed

+0
-17
lines changed

test/extended/networking/commatrix.go

Lines changed: 0 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,6 @@ import (
2525
listeningsockets "github.com/openshift-kni/commatrix/pkg/listening-sockets"
2626
matrixdiff "github.com/openshift-kni/commatrix/pkg/matrix-diff"
2727
"github.com/openshift-kni/commatrix/pkg/utils"
28-
"github.com/openshift/origin/pkg/test/ginkgo/result"
2928
)
3029

3130
const (
@@ -139,23 +138,7 @@ var _ = Describe("[sig-network][Feature:commatrix][apigroup:config.openshift.io]
139138
}
140139

141140
missingEPSMat := diff.GetUniqueSecondary()
142-
143-
// flake if only any of the temporary approver ports are missing.
144-
// once the approver ports are officially added, this can be removed.
145141
if len(missingEPSMat.Matrix) > 0 {
146-
isOnlyApproverPorts := true
147-
for _, cd := range missingEPSMat.Matrix {
148-
if cd.Port != 9193 && cd.Port != 9194 {
149-
isOnlyApproverPorts = false
150-
break
151-
}
152-
}
153-
154-
if isOnlyApproverPorts {
155-
result.Flakef("temporary approver ports missing endpointslice: %s", missingEPSMat)
156-
return
157-
}
158-
159142
err := fmt.Errorf("the following ports are used but don't have an endpointslice: \n %s", missingEPSMat)
160143
Expect(err).ToNot(HaveOccurred())
161144
}

0 commit comments

Comments
 (0)