June 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-3.6.7]: core: HostDevice.isAssignable true by default
by piotr.kliczewski＠gmail.com 03 Jun '16

03 Jun '16

Piotr Kliczewski has submitted this change and it was merged. Change subject: core: HostDevice.isAssignable true by default ...................................................................... core: HostDevice.isAssignable true by default Vdsm versions up to 4.17.28 doesn't report 'is_assignable' property of host devices which breaks the functionality in engine. This patch makes VdsBrokerObjectsBuilder#buildHostDevices assume 'is_assignable' true as originally intended. Change-Id: I1601d72de2756467e08b2d09304538a0d9d17f35 Bug-Url: https://bugzilla.redhat.com/1341299 Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Jakub Niedermertl: Verified Martin Peřina: Looks good to me, approved Jenkins CI: Passed CI tests Martin Polednik: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/58601 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1601d72de2756467e08b2d09304538a0d9d17f35 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6.7 Gerrit-Owner: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6]: core: HostDevice.isAssignable true by default
by piotr.kliczewski＠gmail.com 03 Jun '16

03 Jun '16

Piotr Kliczewski has submitted this change and it was merged. Change subject: core: HostDevice.isAssignable true by default ...................................................................... core: HostDevice.isAssignable true by default Vdsm versions up to 4.17.28 doesn't report 'is_assignable' property of host devices which breaks the functionality in engine. This patch makes VdsBrokerObjectsBuilder#buildHostDevices assume 'is_assignable' true as originally intended. Change-Id: I1601d72de2756467e08b2d09304538a0d9d17f35 Bug-Url: https://bugzilla.redhat.com/1341299 Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Jakub Niedermertl: Verified Martin Peřina: Looks good to me, approved Jenkins CI: Passed CI tests Martin Polednik: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/58583 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1601d72de2756467e08b2d09304538a0d9d17f35 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: core: HostDevice.isAssignable true by default
by piotr.kliczewski＠gmail.com 03 Jun '16

03 Jun '16

Piotr Kliczewski has submitted this change and it was merged. Change subject: core: HostDevice.isAssignable true by default ...................................................................... core: HostDevice.isAssignable true by default Vdsm versions up to 4.17.28 doesn't report 'is_assignable' property of host devices which breaks the functionality in engine. This patch makes VdsBrokerObjectsBuilder#buildHostDevices assume 'is_assignable' true as originally intended. Change-Id: I1601d72de2756467e08b2d09304538a0d9d17f35 Bug-Url: https://bugzilla.redhat.com/1341299 Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Jakub Niedermertl: Verified Martin Peřina: Looks good to me, approved Jenkins CI: Passed CI tests Martin Polednik: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/58564 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1601d72de2756467e08b2d09304538a0d9d17f35 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Martin Polednik <mpolednik(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: restapi: Remove 3.5 capabilities
by piotr.kliczewski＠gmail.com 03 Jun '16

03 Jun '16

Piotr Kliczewski has submitted this change and it was merged. Change subject: restapi: Remove 3.5 capabilities ...................................................................... restapi: Remove 3.5 capabilities Compatibility version 3.5 isn't supported by version 4 of the engine, so it needs to be removed from the capabilities. Change-Id: I2bc1c262cae8842f3c3bc629231dc8fef2e88397 Bug-Url: https://bugzilla.redhat.com/1342226 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit 452b6af3d39c602ac95103365aefaa549905a0d1) --- M backend/manager/modules/restapi/jaxrs/src/main/resources/v3/capabilities.xml 1 file changed, 0 insertions(+), 1,201 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/58596 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2bc1c262cae8842f3c3bc629231dc8fef2e88397 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: restapi: Replace "networklabels" with "labels" in V3
by piotr.kliczewski＠gmail.com 03 Jun '16

03 Jun '16

Piotr Kliczewski has submitted this change and it was merged. Change subject: restapi: Replace "networklabels" with "labels" in V3 ...................................................................... restapi: Replace "networklabels" with "labels" in V3 In version 4 of the API the "labels" URL segment was renamed to "labels", but we need to preserve it in version 3. This patch changes the network label out adapter so that it performs the required replacement. Change-Id: Ifeec86e2e11062c9c5a261595e37e0ea87e7371a Bug-Url: https://bugzilla.redhat.com/1338522 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit baa9730f86f358755b03c9199480e8f54ded5cdf) --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/adapters/V3NetworkLabelOutAdapter.java 1 file changed, 8 insertions(+), 2 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/58595 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifeec86e2e11062c9c5a261595e37e0ea87e7371a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-3.6.7]: api: api doesnt pass the vminit when increasing vm in pool
by piotr.kliczewski＠gmail.com 03 Jun '16

03 Jun '16

Piotr Kliczewski has submitted this change and it was merged. Change subject: api: api doesnt pass the vminit when increasing vm in pool ...................................................................... api: api doesnt pass the vminit when increasing vm in pool Change-Id: Iecdda54e5e99faad195984f98f47f517267f7e75 Bug-Url: https://bugzilla.redhat.com/1339287 Signed-off-by: Shahar Havivi <shaharh(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmPoolResource.java 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Juan Hernandez: Looks good to me, approved Shahar Havivi: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/58528 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iecdda54e5e99faad195984f98f47f517267f7e75 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6.7 Gerrit-Owner: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in jenkins[master]: engine: Add -pre and -snapshot branches and update publisher
by sbonazzo＠redhat.com 03 Jun '16

03 Jun '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: engine: Add -pre and -snapshot branches and update publisher ...................................................................... engine: Add -pre and -snapshot branches and update publisher Change-Id: Ie99806056942b01145dfd15aaac7c037d6ae6180 Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> --- M jobs/confs/projects/ovirt-appliance/ovirt-appliance.yaml M jobs/confs/projects/ovirt/publish-rpms-nightly-4.0.yaml 2 files changed, 9 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/58599 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie99806056942b01145dfd15aaac7c037d6ae6180 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in gerrit-admin[master]: Fixed issue wit bugzilla login
by dcaro＠redhat.com 03 Jun '16

03 Jun '16

David Caro has submitted this change and it was merged. Change subject: Fixed issue wit bugzilla login ...................................................................... Fixed issue wit bugzilla login We were extracting the user and pass before parsing the options Change-Id: Ica814c967dba33ff39172d0801ccb1d7917f53a0 Signed-off-by: David Caro <dcaroest(a)redhat.com> --- M hooks/lib/bz.sh 1 file changed, 4 insertions(+), 2 deletions(-) Approvals: Jenkins CI: Passed CI tests David Caro: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/58432 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ica814c967dba33ff39172d0801ccb1d7917f53a0 Gerrit-PatchSet: 1 Gerrit-Project: gerrit-admin Gerrit-Branch: master Gerrit-Owner: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: David Caro <dcaro(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Remove 3.5 capabilities
by juan.hernandez＠redhat.com 03 Jun '16

03 Jun '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Remove 3.5 capabilities ...................................................................... restapi: Remove 3.5 capabilities Compatibility version 3.5 isn't supported by version 4 of the engine, so it needs to be removed from the capabilities. Change-Id: I2bc1c262cae8842f3c3bc629231dc8fef2e88397 Bug-Url: https://bugzilla.redhat.com/1342226 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/resources/v3/capabilities.xml 1 file changed, 0 insertions(+), 1,201 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/58570 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2bc1c262cae8842f3c3bc629231dc8fef2e88397 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Meni Yakove <myakove(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: restapi: Replace "networklabels" with "labels" in V3
by juan.hernandez＠redhat.com 03 Jun '16

03 Jun '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Replace "networklabels" with "labels" in V3 ...................................................................... restapi: Replace "networklabels" with "labels" in V3 In version 4 of the API the "labels" URL segment was renamed to "labels", but we need to preserve it in version 3. This patch changes the network label out adapter so that it performs the required replacement. Change-Id: Ifeec86e2e11062c9c5a261595e37e0ea87e7371a Bug-Url: https://bugzilla.redhat.com/1338522 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/v3/adapters/V3NetworkLabelOutAdapter.java 1 file changed, 8 insertions(+), 2 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/58540 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifeec86e2e11062c9c5a261595e37e0ea87e7371a Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0