Skip to content

Commit 6fd4527

Browse files
Merge pull request #30424 from aabughosh/flakeports
OCPBUGS-63519: flake the missing port 9194 and 9193 once the team will fix will remove it
2 parents 60c17c5 + d414f12 commit 6fd4527

File tree

1 file changed

+17
-0
lines changed

1 file changed

+17
-0
lines changed

test/extended/networking/commatrix.go

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@ 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"
2829
)
2930

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

140141
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.
141145
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+
142159
err := fmt.Errorf("the following ports are used but don't have an endpointslice: \n %s", missingEPSMat)
143160
Expect(err).ToNot(HaveOccurred())
144161
}

0 commit comments

Comments
 (0)