Change in ovirt-engine[master]: packaging: setup: Require latest otopi

Yedidyah Bar David didi at redhat.com
Thu Dec 3 10:03:46 UTC 2015


On Thu, Dec 3, 2015 at 10:43 AM, Sandro Bonazzola <sbonazzo at redhat.com> wrote:
>
>
> On Thu, Dec 3, 2015 at 7:41 AM, Yedidyah Bar David <didi at redhat.com> wrote:
>>
>> On Thu, Dec 3, 2015 at 3:50 AM, Jenkins CI <gerrit2 at gerrit.ovirt.org>
>> wrote:
>> > Jenkins CI has posted comments on this change.
>> >
>> > Change subject: packaging: setup: Require latest otopi
>> > ......................................................................
>> >
>> >
>> > Patch Set 3:
>> >
>> > Build Failed
>> >
>> >
>> > http://jenkins.ovirt.org/job/ovirt-engine_master_upgrade-from-master_el7_merged/1178/
>> > : FAILURE
>> >
>> >
>> > http://jenkins.ovirt.org/job/ovirt-engine_master_upgrade-from-3.6_el7_merged/1088/
>> > : FAILURE
>>
>> Broken, intentionally, by my patch - it requires latest available
>> otopi to be installed.
>> Not sure how to fix in jenkins. I understand that adding 'otopi' to
>> automation/*.packages
>> won't help, because it will require it for the build chroot, not for
>> the one used for
>> running tests. Clues are welcome :-)
>
>
>
> Adding Sharon, I think something like this should do the work:
> https://gerrit.ovirt.org/49606
>

Above is ok, it's for master.

On a second thought, we do not want that for stable. We want there
to Require the correct version of otopi, not fail and let the user
upgrade for us.

>>
>>
>> >
>> >
>> > http://jenkins.ovirt.org/job/ovirt-engine_master_unit-tests_merged/13904/ :
>> > SUCCESS
>> >
>> >
>> > http://jenkins.ovirt.org/job/ovirt-engine_master_check-merged-fc22-x86_64/779/
>> > : SUCCESS
>> >
>> >
>> > http://jenkins.ovirt.org/job/ovirt-engine_master_check-merged-fc23-x86_64/371/
>> > : SUCCESS
>> >
>> >
>> > http://jenkins.ovirt.org/job/ovirt-engine_master_check-merged-el7-x86_64/777/
>> > : SUCCESS
>> >
>> >
>> > http://jenkins.ovirt.org/job/ovirt-engine_master_dao-unit-tests_merged/14023/
>> > : SUCCESS
>> >
>> > --
>> > To view, visit https://gerrit.ovirt.org/49105
>> > To unsubscribe, visit https://gerrit.ovirt.org/settings
>> >
>> > Gerrit-MessageType: comment
>> > Gerrit-Change-Id: Icf2a5c3af3845527b75c2b0a4cdd42d5bd89f4a3
>> > Gerrit-PatchSet: 3
>> > Gerrit-Project: ovirt-engine
>> > Gerrit-Branch: master
>> > Gerrit-Owner: Yedidyah Bar David <didi at redhat.com>
>> > Gerrit-Reviewer: Jenkins CI
>> > Gerrit-Reviewer: Sandro Bonazzola <sbonazzo at redhat.com>
>> > Gerrit-Reviewer: Simone Tiraboschi <stirabos at redhat.com>
>> > Gerrit-Reviewer: Yedidyah Bar David <didi at redhat.com>
>> > Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>
>> > Gerrit-HasComments: No
>>
>>
>>
>> --
>> Didi
>
>
>
>
> --
> Sandro Bonazzola
> Better technology. Faster innovation. Powered by community collaboration.
> See how it works at redhat.com



-- 
Didi



More information about the Infra mailing list