October 2015 - Engine-commits - oVirt List Archives

Change in otopi[master]: packaging: otopi-1.4.0
by Alon Bar-Lev 15 Oct '15

15 Oct '15

Alon Bar-Lev has submitted this change and it was merged. Change subject: packaging: otopi-1.4.0 ...................................................................... packaging: otopi-1.4.0 Change-Id: I26f441ed00440491432da0323fc5dddbcffbdf0c Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M ChangeLog M configure.ac M otopi.spec.in 3 files changed, 6 insertions(+), 3 deletions(-) Approvals: Alon Bar-Lev: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/47377 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I26f441ed00440491432da0323fc5dddbcffbdf0c Gerrit-PatchSet: 1 Gerrit-Project: otopi Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-node-plugin-vdsm[ovirt-3.6]: engine_page: fix parms name to update /etc/default/ovirt
by dougsland＠redhat.com 15 Oct '15

15 Oct '15

Douglas Schilling Landgraf has submitted this change and it was merged. Change subject: engine_page: fix parms name to update /etc/default/ovirt ...................................................................... engine_page: fix parms name to update /etc/default/ovirt Currently the params are incorrect and updating /etc/default/ovirt is broken. This patch fix the parms name. Change-Id: I311083b4c74d752e201ceedffa923f450346eedc Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1271111 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> Reviewed-on: https://gerrit.ovirt.org/47339 Continuous-Integration: Jenkins CI Reviewed-by: Ryan Barry <rbarry(a)redhat.com> (cherry picked from commit 1a0f42aed0437affd2bd5d6ff4d8ad432551a251) Reviewed-on: https://gerrit.ovirt.org/47368 --- M src/engine_page.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Douglas Schilling Landgraf: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/47368 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I311083b4c74d752e201ceedffa923f450346eedc Gerrit-PatchSet: 2 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: ovirt-3.6 Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-node-plugin-vdsm[master]: engine_page: fix parms name to update /etc/default/ovirt
by dougsland＠redhat.com 15 Oct '15

15 Oct '15

Douglas Schilling Landgraf has submitted this change and it was merged. Change subject: engine_page: fix parms name to update /etc/default/ovirt ...................................................................... engine_page: fix parms name to update /etc/default/ovirt Currently the params are incorrect and updating /etc/default/ovirt is broken. This patch fix the parms name. Change-Id: I311083b4c74d752e201ceedffa923f450346eedc Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1271111 Signed-off-by: Douglas Schilling Landgraf <dougsland(a)redhat.com> Reviewed-on: https://gerrit.ovirt.org/47339 Continuous-Integration: Jenkins CI Reviewed-by: Ryan Barry <rbarry(a)redhat.com> --- M src/engine_page.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Douglas Schilling Landgraf: Verified; Looks good to me, approved Jenkins CI: Passed CI tests Ryan Barry: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/47339 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I311083b4c74d752e201ceedffa923f450346eedc Gerrit-PatchSet: 2 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: master Gerrit-Owner: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Ryan Barry <rbarry(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: webadmin: Allow to set prestarted VMs for a new VM pool
by ahadas＠redhat.com 15 Oct '15

15 Oct '15

Arik Hadas has submitted this change and it was merged. Change subject: webadmin: Allow to set prestarted VMs for a new VM pool ...................................................................... webadmin: Allow to set prestarted VMs for a new VM pool Enable 'Prestarted VMs' field in the new VM pool creation dialog. Allow to prestart VMs that will be added to the pool as result of pool creation or update. Change-Id: Ic3c74489f3f038190a6685f027172a633929b86b Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1258380 Signed-off-by: Shmuel Melamud <smelamud(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommonVmPoolWithVmsCommand.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/AbstractVmPopupWidget.java M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/AbstractVmPopupWidget.ui.xml M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/pool/PoolNewPopupWidget.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/PoolModelBehaviorBase.java 5 files changed, 7 insertions(+), 27 deletions(-) Approvals: Tomas Jelinek: Looks good to me, but someone else must approve Jenkins CI: Passed CI tests Shmuel Leib Melamud: Verified Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/46703 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic3c74489f3f038190a6685f027172a633929b86b Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Shmuel Leib Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Michal Skrivanek <mskrivan(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6.0]: core: Abort SD activation when connection to VDS fails
by tnisan＠redhat.com 15 Oct '15

15 Oct '15

Tal Nisan has submitted this change and it was merged. Change subject: core: Abort SD activation when connection to VDS fails ...................................................................... core: Abort SD activation when connection to VDS fails Prior to this change SD activation was attempted and failed, leaving the SD in inactive status. Now the activation is not attempted and the SD maintains its status. Change-Id: Ie99cdbe34c15a608a5777d4881fc2cd255d2c512 Bug-Url: https://bugzilla.redhat.com/1164938 Signed-off-by: Vered Volansky <vvolansk(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ActivateStorageDomainCommand.java 1 file changed, 14 insertions(+), 0 deletions(-) Approvals: Jenkins CI: Passed CI tests Vered Volansky: Verified Allon Mureinik: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/47357 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie99cdbe34c15a608a5777d4881fc2cd255d2c512 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6.0 Gerrit-Owner: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6]: core: Abort SD activation when connection to VDS fails
by tnisan＠redhat.com 15 Oct '15

15 Oct '15

Tal Nisan has submitted this change and it was merged. Change subject: core: Abort SD activation when connection to VDS fails ...................................................................... core: Abort SD activation when connection to VDS fails Prior to this change SD activation was attempted and failed, leaving the SD in inactive status. Now the activation is not attempted and the SD maintains its status. Change-Id: Ie99cdbe34c15a608a5777d4881fc2cd255d2c512 Bug-Url: https://bugzilla.redhat.com/1164938 Signed-off-by: Vered Volansky <vvolansk(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ActivateStorageDomainCommand.java 1 file changed, 14 insertions(+), 0 deletions(-) Approvals: Jenkins CI: Passed CI tests Vered Volansky: Verified Allon Mureinik: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/47356 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie99cdbe34c15a608a5777d4881fc2cd255d2c512 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: Abort SD activation when connection to VDS fails
by amureini＠redhat.com 15 Oct '15

15 Oct '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: Abort SD activation when connection to VDS fails ...................................................................... core: Abort SD activation when connection to VDS fails Prior to this change SD activation was attempted and failed, leaving the SD in inactive status. Now the activation is not attempted and the SD maintains its status. Change-Id: Ie99cdbe34c15a608a5777d4881fc2cd255d2c512 Bug-Url: https://bugzilla.redhat.com/1164938 Signed-off-by: Vered Volansky <vvolansk(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/ActivateStorageDomainCommand.java 1 file changed, 14 insertions(+), 0 deletions(-) Approvals: Jenkins CI: Passed CI tests Vered Volansky: Verified Allon Mureinik: Looks good to me, approved Maor Lipchuk: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/47134 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie99cdbe34c15a608a5777d4881fc2cd255d2c512 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine-sdk[master]: sdk: Regenerate against the latest API
by juan.hernandez＠redhat.com 15 Oct '15

15 Oct '15

Juan Hernandez has submitted this change and it was merged. Change subject: sdk: Regenerate against the latest API ...................................................................... sdk: Regenerate against the latest API This patch regenerates the SDK using the latest API obtained from the engine built from commit ae25257. Change-Id: Ie73046251886962fc34c8efd65831d41e0f150b1 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M Makefile M generator/pom.xml M generator/src/main/resources/api.xsd M generator/src/main/resources/rsdl.xml M src/ovirtsdk/api.py M src/ovirtsdk/infrastructure/brokers.py M src/ovirtsdk/xml/params.py 7 files changed, 7,359 insertions(+), 7,354 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/47355 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie73046251886962fc34c8efd65831d41e0f150b1 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk 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: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6.0]: engine: Use UUID as content host id on Satellite
by tnisan＠redhat.com 15 Oct '15

15 Oct '15

Tal Nisan has submitted this change and it was merged. Change subject: engine: Use UUID as content host id on Satellite ...................................................................... engine: Use UUID as content host id on Satellite There is a difference in the semantic of the ID field under: /katello/api/v2/systems/:id resource between Katello to Satellite. The :id field in Katello represents the UUID of the content host, however under satellite it is a sequence number, which causes the requests sent from the engine to fail with unknown resource identifier. On Satellite (and also on Katello) the value of the UUID field of the content host can be used to identify it @candlepin. The solution will be to use the UUID field of the system resource instead of the ID. Change-Id: I44ee45ac9770238c8d353ae30bf18a280b92b7ea Bug-Url: https://bugzilla.redhat.com/1271340 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/host/provider/foreman/ContentHost.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/host/provider/foreman/ForemanHostProviderProxy.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/host/provider/foreman/SystemProviderFinder.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/BaseProviderProxy.java 4 files changed, 14 insertions(+), 9 deletions(-) Approvals: Jenkins CI: Passed CI tests Moti Asayag: Verified Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/47352 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I44ee45ac9770238c8d353ae30bf18a280b92b7ea Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6.0 Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6]: engine: Use UUID as content host id on Satellite
by oourfali＠redhat.com 15 Oct '15

15 Oct '15

Oved Ourfali has submitted this change and it was merged. Change subject: engine: Use UUID as content host id on Satellite ...................................................................... engine: Use UUID as content host id on Satellite There is a difference in the semantic of the ID field under: /katello/api/v2/systems/:id resource between Katello to Satellite. The :id field in Katello represents the UUID of the content host, however under satellite it is a sequence number, which causes the requests sent from the engine to fail with unknown resource identifier. On Satellite (and also on Katello) the value of the UUID field of the content host can be used to identify it @candlepin. The solution will be to use the UUID field of the system resource instead of the ID. Change-Id: I44ee45ac9770238c8d353ae30bf18a280b92b7ea Bug-Url: https://bugzilla.redhat.com/1271340 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/host/provider/foreman/ContentHost.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/host/provider/foreman/ForemanHostProviderProxy.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/host/provider/foreman/SystemProviderFinder.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/BaseProviderProxy.java 4 files changed, 14 insertions(+), 9 deletions(-) Approvals: Jenkins CI: Passed CI tests Moti Asayag: Verified Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/47350 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I44ee45ac9770238c8d353ae30bf18a280b92b7ea Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0