On 7/23/21 5:06 PM, Mark Gray wrote: > On 23/07/2021 12:21, Dumitru Ceara wrote: >> With the ovn-kubernetes version currently tagged for use within OVN CI >> there were often failures that were unrelated to OVN itself. >> >> Update to the current ovn-kubernetes master branch version for improved >> reliability of the test runs. >> >> Changes in ovn-kubernetes between the two versions: >> https://github.com/ovn-org/ovn-kubernetes/compare/f669ef6d35c86bf46d7a1055072e48ea1506f88c...c9cd438575f11584871309740c78dde29752940b >> >> Signed-off-by: Dumitru Ceara <[email protected]> >> --- >> Successful ovn-kubernetes CI run (also when rerun) with this patch >> applied: >> https://github.com/dceara/ovn/runs/3142362509?check_suite_focus=true >> --- >> .github/workflows/ovn-kubernetes.yml | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/.github/workflows/ovn-kubernetes.yml >> b/.github/workflows/ovn-kubernetes.yml >> index ee77613f6..03721c109 100644 >> --- a/.github/workflows/ovn-kubernetes.yml >> +++ b/.github/workflows/ovn-kubernetes.yml >> @@ -11,7 +11,7 @@ on: >> env: >> GO_VERSION: "1.16.3" >> K8S_VERSION: v1.20.2 >> - OVNKUBE_COMMIT: "f669ef6d35c86bf46d7a1055072e48ea1506f88c" >> + OVNKUBE_COMMIT: "c9cd438575f11584871309740c78dde29752940b" >> KIND_CLUSTER_NAME: ovn >> KIND_INSTALL_INGRESS: true >> KIND_ALLOW_SYSTEM_WRITES: true >> > > Any particular reason why you chose this commit? I wonder what our > policy should be to decide when (and to what commit) we should bump to?
I just chose the current master commit in ovn-kubernetes because tests seem to be less flaky with it. I agree we need a policy for how to bump this version from now on, but for now, I just wanted to try to have the ovn-kubernetes job pass more often (it currently fails on almost every run). > > Acked-by: Mark D. Gray <[email protected]> > Thanks for the review! _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
