[ovirt-devel] Re: master CQ fails due to missing ovirt-host-deploy

On Wed, Feb 12, 2020 at 11:39 AM Dafna Ron <dron(a)redhat.com&gt; wrote: > > this is the current package in tested: ovirt-engine-0:4.2.0-0.0.master.20170712084142.git63b81ef.el7.centos.noarch and since all projects are now failing on this, it means the engine package in tested is a broken package. > what I don't understand is how can CQ mark engine as failed since this patch: https://gerrit.ovirt.org/#/c/104391/ which is 4 weeks ago but I can see a new package in tested since 06-02-2020 with no passing cq test for ovirt-engine. > If anyone is moving packages manually to tested they are breaking all projects by introducing broken packages. > > As the package in tested is now broken, we should start fixing the code on each failing issue. > for now, the main thing that is failing is that we are looking for a package that no longer exists. > Didi, is it possible its the typo Marcin was commenting on in this patch: https://gerrit.ovirt.org/#/c/106797/ I already addressed this, > > Thanks, > Dafna > > > > > > > > On Tue, Feb 11, 2020 at 9:03 AM Yedidyah Bar David <didi(a)redhat.com&gt; wrote: >> >> On Tue, Feb 11, 2020 at 10:22 AM Dusan Fodor <dfodor(a)redhat.com&gt; wrote: >> > >> > Hello, >> > master CQ is failing on initialize_engine due to missing ovirt-host-deploy dependency. >> > >> > [ INFO ] Checking for an update for Setup... >> > [ ERROR ] Yum Cannot queue package ovirt-host-deploy: Package ovirt-host-deploy cannot be found >> > >> > >> > I was under the impression that engine-setup should not require it anymore, but it seems the related patch [1] doesn't work as intended. Perhaps Marcin's comment there explains why? Can you please check this? >> >> How can I know which engine build was used for [2]? >> >> Checking lago.log there, I see: >> >> Installed: >> net-snmp.x86_64 1:5.7.2-43.el7_7.3 >> ntp.x86_64 0:4.2.6p5-29.el7.centos >> otopi-debug-plugins.noarch 0:1.9.0-1.el7 >> ovirt-engine.noarch 0:4.4.0-0.0.master.20200206140618.gitff1f0b4.el7 >> ovirt-engine-extension-aaa-ldap.noarch 0:1.3.11-0.0.master.gitb9a171f.el7 >> ovirt-engine-extension-aaa-ldap-setup.noarch >> 0:1.3.11-0.0.master.gitb9a171f.el7 >> ovirt-log-collector.noarch 0:4.4.0-0.0.master.20191122133512.gitfc29382.el7 >> >> ff1f0b4 is a hash of a not-yet-merged patch: >> >> https://gerrit.ovirt.org/106794 >> >> It should be rebased. >> >> Also not sure how the change-queue-tester uses a not-merged-yet engine. >> >> (And also, [1] is not enough, Marcin Sobczyk later pushed a fix for >> it, https://gerrit.ovirt.org/106809 ). here ^^. Already merged as well. >> >> > Job example [2] >> > Thanks >> > >> > [1] https://gerrit.ovirt.org/#/c/106797/ >> > [2] https://jenkins.ovirt.org/view/Change%20queue%20jobs/job/ovirt-master_cha... >> >> >> >> -- >> Didi >> _______________________________________________ >> 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/I5NOUPCLFXQ... -- Didi