5:33 p.m.
On Thu, May 19, 2016 at 4:26 PM, Tal Nisan <tnisan(a)redhat.com> wrote:
No need, already reverted once I've read you message, had it
reverted and
verified so I could keep working on the branch -
https://gerrit.ovirt.org/#/c/57742/
BTW, both jobs ended with a failure:
http://jenkins.ovirt.org/job/ovirt-engine_master_upgrade-from-master_el7_...
: FAILURE
http://jenkins.ovirt.org/job/ovirt-engine_master_upgrade-from-3.6_el7_mer...
: FAILURE
Yes, but those are executed only after patch is merged, check-patch jobs
are return SUCCESS if when build failed, David is working on that.
On Thu, May 19, 2016 at 5:22 PM, David Caro <dcaro(a)redhat.com> wrote:
> On 05/19 16:20, Martin Perina wrote:
> > On Thu, May 19, 2016 at 4:15 PM, David Caro <dcaro(a)redhat.com> wrote:
> >
> > > On 05/19 16:14, Martin Perina wrote:
> > > > Hi,
> > > >
> > > > so I probably found the issue why this broke the build. I have
> checked
> > > the
> > > > CI before merging and it was OK,
> > > > see CI+1 on patch set 20.
> > > >
> > > > Now looking at console outputs of those jobs, the build itself
> failed,
> > > but
> > > > build jobs are marked as SUCCESSFULL.
> > > > So it seems we have a bug in those jobs!!!
> > >
> > >
> > > We will need a bit more info on which project/jenkins job/build, etc,
> by
> > > the
> > > email I can't extract the project that failed
> > >
> >
> > Sorry, here is the problematic patch:
> https://gerrit.ovirt.org/#/c/57052/
> >
> > The problematic patch is build on patch set 20. It's marked as
> successfull,
> > although engine build failed:
> >
> >
> http://jenkins.ovirt.org/job/ovirt-engine_master_check-patch-el7-x86_64/1...
> >
> >
> > Anyway, we are preparing reverting patch ...
>
>
> I see, the issue was introduced by me in the mock_runner.sh script, will
> send a
> patch right away
>
> >
> > Sorry for the issue.
> >
> > Martin
> >
> >
> >
> > >
> > > >
> > > >
> > > >
> > > >
> > > >
> > > > On Thu, May 19, 2016 at 3:53 PM, Tal Nisan <tnisan(a)redhat.com>
> wrote:
> > > >
> > > > > This commit broke the build, missing KernelEnv.
> > > > > Normally I'd revert but since it's feature freeze I'm
more
> forgiving :)
> > > > > Please either revert or send a fix asap.
> > > > >
> > > > > Thanks.
> > > > >
> > > > >
> > >
> > > > _______________________________________________
> > > > Infra mailing list
> > > > Infra(a)ovirt.org
> > > > http://lists.ovirt.org/mailman/listinfo/infra
> > >
> > >
> > > --
> > > David Caro
> > >
> > > Red Hat S.L.
> > > Continuous Integration Engineer - EMEA ENG Virtualization R&D
> > >
> > > Tel.: +420 532 294 605
> > > Email: dcaro(a)redhat.com
> > > IRC: dcaro|dcaroest@{freenode|oftc|redhat}
> > > Web: www.redhat.com
> > > RHT Global #: 82-62605
> > >
>
> --
> David Caro
>
> Red Hat S.L.
> Continuous Integration Engineer - EMEA ENG Virtualization R&D
>
> Tel.: +420 532 294 605
> Email: dcaro(a)redhat.com
> IRC: dcaro|dcaroest@{freenode|oftc|redhat}
> Web: www.redhat.com
> RHT Global #: 82-62605
>