[ovs-dev] [PATCH ovn] ci: Use a newer version of ovn-kubernetes to improve reliability of the runs.

[Dumitru Ceara]

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 <dceara at redhat.com>
>> ---
>> 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 <mark.d.gray at redhat.com>
> 

Thanks for the review!