On 7/23/21 6:05 PM, Numan Siddique wrote: > On Fri, Jul 23, 2021, 11:47 AM Dumitru Ceara <[email protected]> wrote: > >> 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! >> > > > Thanks. I applied this patch. > > Numan >
+ Antonio, Dan I stand corrected, we just got one more failure with this version too, sorry for the noise: https://github.com/ovn-org/ovn/actions/runs/1060150002 I *think* those are test issues but we probably need to investigate some more. I wonder if it makes sense to skip the ones that have issues for now. Thoughts? Regards, Dumitru _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
