Skip to content

Commit

Permalink
Add e2e test policy creation retry
Browse files Browse the repository at this point in the history
add a retry to the policy create to fix OCP CI flake

also update the ocp version to latest 4.14 rc release

Signed-off-by: Sebastian Sch <sebassch@gmail.com>
  • Loading branch information
SchSeba committed Oct 19, 2023
1 parent bf595e4 commit db39e3c
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 3 deletions.
4 changes: 4 additions & 0 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -145,3 +145,7 @@ jobs:
with:
name: ocp-artifact
path: ./ocp-artifacts

- name: clean cluster
if: always()
run: kcli delete cluster $cluster_name -y && kcli delete network $cluster_name -y
2 changes: 1 addition & 1 deletion hack/run-e2e-conformance-virtual-ocp.sh
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ kcli create network -c 192.168.123.0/24 ocp
kcli create network -c 192.168.${virtual_router_id}.0/24 --nodhcp -i $cluster_name

cat <<EOF > ./${cluster_name}-plan.yaml
tag: 4.14.0-rc.1
tag: 4.14.0-rc.6
ctlplane_memory: 24576
worker_memory: 8192
pool: default
Expand Down
5 changes: 3 additions & 2 deletions test/conformance/tests/test_sriov_operator.go
Original file line number Diff line number Diff line change
Expand Up @@ -2316,8 +2316,9 @@ func createVanillaNetworkPolicy(node string, sriovInfos *cluster.EnabledNodes, n
DeviceType: "netdevice",
},
}
err = clients.Create(context.Background(), config)
Expect(err).ToNot(HaveOccurred())
Eventually(func() error {
return clients.Create(context.Background(), config)
}, 1*time.Minute, 5*time.Second).ShouldNot(HaveOccurred())

Eventually(func() sriovv1.Interfaces {
nodeState, err := clients.SriovNetworkNodeStates(operatorNamespace).Get(context.Background(), node, metav1.GetOptions{})
Expand Down

0 comments on commit db39e3c

Please sign in to comment.