[ovirt-devel] Re: Check-Merged Fails for ovirt-provider-ovn

On Wed, Mar 20, 2019 at 1:31 PM Sandro Bonazzola <sbonazzo(a)redhat.com&gt; wrote: > > > Il giorno mer 20 mar 2019 alle ore 12:33 Eyal Edri <eedri(a)redhat.com&gt; ha > scritto: > >> >> >> On Wed, Mar 20, 2019 at 12:45 PM Sandro Bonazzola <sbonazzo(a)redhat.com&gt; >> wrote: >> >>> >>> >>> Il giorno mer 20 mar 2019 alle ore 11:07 Miguel Duarte de Mora Barroso < >>> mdbarroso(a)redhat.com&gt; ha scritto: >>> >>>> On Wed, Mar 20, 2019 at 9:43 AM Miguel Duarte de Mora Barroso >>>> <mdbarroso(a)redhat.com&gt; wrote: >>>> > >>>> > On Tue, Mar 19, 2019 at 6:51 PM Anton Marchukov <amarchuk(a)redhat.com&gt; >>>> wrote: >>>> > > >>>> > > Retrigger failed, seems to be the same error [1]? >>>> > >>>> > OK, taking a look now. >>>> >>>> I noticed that it is installing a very old openvswitch version in >>>> fedora - 2.8.1 - as you can check in the logs [0]. >>>> >>>> 17:21:51 openvswitch.x86_64 2.8.1-2.fc28 >>>> 17:21:51 openvswitch-ovn-central.x86_64 2.8.1-2.fc28 >>>> 17:21:51 openvswitch-ovn-common.x86_64 2.8.1-2.fc28 >>>> 17:21:51 openvswitch-ovn-host.x86_64 2.8.1-2.fc28 >>>> >>>> The 'check-patch' scripts have also begun failing for all my patches >>>> on all targets *except* 4.2 - check [1]. >>>> >>>> On el7, it is grabbing version openvswitch.x86_64 >>>> 1:2.9.0-4.el7 from centos-ovirt-4.2-el7. >>>> >>>> ovirt-provider-ovn *requires* openvswitch 2.10 on 4.3 *and* master. >>>> >>>> Sandro, this is just a theory, but to me it looks like >>>> ovirt-provider-ovn master doesn't "know" it needs to get the packages >>>> from the virt7-ovirt-43-testing tag or something alike. >>>> >>> >>> I would start with adding >>> Requires: openvswitch >= 2.10 >>> to ovirt-provider-ovn spec file. >>> >>> Then yes, looks like OST is taking openvswitch from the wrong repo. >>> Dafna, can you fix OST repo config for taking openvswitch from >>> https://buildlogs.centos.org/centos/7/virt/$basearch/ovirt-4.3/ on EL7 >>> and >>> >> This failure was unrelated to OST - it happened on the ovirt-provider-ovn integration test environment. We use the tripleo-ovn containers to check the data plane connectivity that ovirt-provider-ovn configures. The controller container was updated, 2 days ago, and for whatever reason, it breaks our CI. By selecting a different container image we managed to get CI to pass in patch [0], which is already merged. I'll look at the root cause when I can find the time. @Anton Marchukov <amarchuk(a)redhat.com&gt; I'll keep an eye out for the check-merged target of the aforementioned patch; I'll update you once it finishes.

[0] - https://gerrit.ovirt.org/#/c/98698/ >> Dafna is on sick leave, >> In order to simplify the reposync file we dropped inlude list for all >> centos small repos so you won't need to handle whitelisting everything a >> new pkgs is out. >> If a package is in multiple repos, YUM should bring always the highest >> version which is normal behavior for YUM afaik. >> If there are specific repos you know for a fact you don't want pkgs >> taken from there, please add exclude from that repo. >> >> I don't even see >> https://buildlogs.centos.org/centos/7/virt/$basearch/ovirt-4.3/ on >> reposync, is that a new repo? >> > > No, it's just the testing repo generated by the sum of > http://cbs.centos.org/repos/virt7-ovirt-common-testing/x86_64/os/ > and > http://cbs.centos.org/repos/virt7-ovirt-43-testing/x86_64/os/ > > > >> >> >> >>> >>> https://copr-be.cloud.fedoraproject.org/results/mdbarroso/openvswitch/fed... >>> for Fedora? >>> >> >> If its needed for the project itself (check-patch) the project owner >> should update it in their automation/check-patch.fc28 file: >> >> cat check-merged.repos.fc28 >> ovirt-snapshot-static, >> http://resources.ovirt.org/pub/ovirt-master-snapshot-static/rpm/$distro >> mdbarroso-python-ovsdbapp, >> https://copr-be.cloud.fedoraproject.org/results/mdbarroso/ovsdbapp/fedora... >> tested,http://resources.ovirt.org/repos/ovirt/tested/master/rpm/$distro >> virt-preview, >> http://fedorapeople.org/groups/virt/virt-preview/fedora-$releasever/$base... >> >> >> >>> >>> >>> >>>> >>>> I've tried locally to revert patch [2] that added the 4.3 branch to >>>> the provider; it fixed this, but I think that's just masking the >>>> problem. >>>> >>>> [0] - >>>> https://jenkins.ovirt.org/job/ovirt-provider-ovn_standard-on-merge/264/co... >>>> [1] - >>>> https://jenkins.ovirt.org/job/ovirt-provider-ovn_master_check-patch-el7-x... >>>> [2] - https://gerrit.ovirt.org/#/c/98565/ >>>> >>>> > >>>> > > >>>> > > [1] >>>> https://jenkins.ovirt.org/job/ovirt-provider-ovn_standard-on-merge/264/ >>>> > > >>>> > > > On 19 Mar 2019, at 17:10, Anton Marchukov <amarchuk(a)redhat.com&gt; >>>> wrote: >>>> > > > >>>> > > > I retriggered. Will see. >>>> > > > >>>> > > >> On 19 Mar 2019, at 16:25, Miguel Duarte de Mora Barroso < >>>> mdbarroso(a)redhat.com&gt; wrote: >>>> > > >> >>>> > > >>> >>>> https://jenkins.ovirt.org/job/ovirt-provider-ovn_standard-on-merge/263/ >>>> > > > >>>> > > > -- >>>> > > > Anton Marchukov >>>> > > > Associate Manager - RHV DevOps - Red Hat >>>> > > > >>>> > > > >>>> > > > >>>> > > > >>>> > > > >>>> > > > >>>> > > >>>> > > -- >>>> > > Anton Marchukov >>>> > > Associate Manager - RHV DevOps - Red Hat >>>> > > >>>> > > >>>> > > >>>> > > >>>> > > >>>> > > >>>> >>> >>> >>> -- >>> >>> SANDRO BONAZZOLA >>> >>> MANAGER, SOFTWARE ENGINEERING, EMEA R&D RHV >>> >>> Red Hat EMEA <https://www.redhat.com/> >>> >>> sbonazzo(a)redhat.com >>> <https://red.ht/sig> >>> _______________________________________________ >>> Devel mailing list -- devel(a)ovirt.org >>> To unsubscribe send an email to devel-leave(a)ovirt.org >>> Privacy Statement: https://www.ovirt.org/site/privacy-policy/ >>> oVirt Code of Conduct: >>> https://www.ovirt.org/community/about/community-guidelines/ >>> List Archives: >>> https://lists.ovirt.org/archives/list/devel@ovirt.org/message/4GWFMJTRWF6... >>> >> >> >> -- >> >> Eyal edri >> >> >> MANAGER >> >> RHV/CNV DevOps >> >> EMEA VIRTUALIZATION R&D >> >> >> Red Hat EMEA <https://www.redhat.com/> >> <https://red.ht/sig> TRIED. TESTED. TRUSTED. >> <https://redhat.com/trusted> >> phone: +972-9-7692018 >> irc: eedri (on #tlv #rhev-dev #rhev-integ) >> > > > -- > > SANDRO BONAZZOLA > > MANAGER, SOFTWARE ENGINEERING, EMEA R&D RHV > > Red Hat EMEA <https://www.redhat.com/> > > sbonazzo(a)redhat.com > <https://red.ht/sig> >