Build broken cause of commit aeb98b5a5ac807d23008ed26293244efab103bde, please fix ASAP

[Martin Perina]

On Thu, May 19, 2016 at 4:26 PM, Tal Nisan <tnisan at 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_merged/310/
> : FAILURE
>
>
> http://jenkins.ovirt.org/job/ovirt-engine_master_upgrade-from-3.6_el7_merged/307/
> : 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 at redhat.com> wrote:
>
>> On 05/19 16:20, Martin Perina wrote:
>> > On Thu, May 19, 2016 at 4:15 PM, David Caro <dcaro at 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 at 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 at 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 at 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 at redhat.com
>> IRC: dcaro|dcaroest@{freenode|oftc|redhat}
>> Web: www.redhat.com
>> RHT Global #: 82-62605
>>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.ovirt.org/pipermail/infra/attachments/20160519/50368515/attachment.html>