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

On Wed, Mar 20, 2019 at 12:44 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. > We don't want to force installation of provider and OvS on the same host/container. But we can add a Conflicts < 2.10 line.

> 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 > > https://copr-be.cloud.fedoraproject.org/results/mdbarroso/openvswitch/fed... > for Fedora? > > > >> >> 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> >