Il giorno mer 20 mar 2019 alle ore 11:55 Dan Kenigsberg <[email protected]> ha scritto:
> > > On Wed, Mar 20, 2019 at 12:44 PM Sandro Bonazzola <[email protected]> > wrote: > >> >> >> Il giorno mer 20 mar 2019 alle ore 11:07 Miguel Duarte de Mora Barroso < >> [email protected]> ha scritto: >> >>> On Wed, Mar 20, 2019 at 9:43 AM Miguel Duarte de Mora Barroso >>> <[email protected]> wrote: >>> > >>> > On Tue, Mar 19, 2019 at 6:51 PM Anton Marchukov <[email protected]> >>> 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. > I think you're kind of abusing the meaning of Conflicts to workaround a RPM limitation regarding containerization but I won't stop you from doing that. I tend to think this won't pass a fedora review session. > > >> 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/fedora-$releasever-$basearch/ >> 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/consoleFull >>> [1] - >>> https://jenkins.ovirt.org/job/ovirt-provider-ovn_master_check-patch-el7-x86_64/611/ >>> [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 <[email protected]> >>> wrote: >>> > > > >>> > > > I retriggered. Will see. >>> > > > >>> > > >> On 19 Mar 2019, at 16:25, Miguel Duarte de Mora Barroso < >>> [email protected]> 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/> >> >> [email protected] >> <https://red.ht/sig> >> > -- SANDRO BONAZZOLA MANAGER, SOFTWARE ENGINEERING, EMEA R&D RHV Red Hat EMEA <https://www.redhat.com/> [email protected] <https://red.ht/sig>
_______________________________________________ Devel mailing list -- [email protected] To unsubscribe send an email to [email protected] 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/[email protected]/message/AFILGICFUXBWLTR63FROXL6I76VGK674/
