On Thu, May 19, 2016 at 4:26 PM, Tal Nisan <tnisan@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: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@redhat.com> wrote:On 05/19 16:20, Martin Perina wrote:
> On Thu, May 19, 2016 at 4:15 PM, David Caro <dcaro@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/1250/console
>
>
> 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@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@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@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@redhat.com
IRC: dcaro|dcaroest@{freenode|oftc|redhat}
Web: www.redhat.com
RHT Global #: 82-62605