July 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: Revert "webadmin: fix assert error in GroupedListModelListBox"
by gshereme＠redhat.com 30 Jul '15

30 Jul '15

Greg Sheremeta has submitted this change and it was merged. Change subject: Revert "webadmin: fix assert error in GroupedListModelListBox" ...................................................................... Revert "webadmin: fix assert error in GroupedListModelListBox" This reverts commit 531dac99cd8c725ae5a7ef45fb2a885e628a704a. Change-Id: I5885dd1c756f09f58addda3670e034cb1b4e4ee5 Signed-off-by: Oved Ourfali <oourfali(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/editor/GroupedListModelListBox.java 1 file changed, 2 insertions(+), 10 deletions(-) Approvals: Alexander Wels: Looks good to me, approved Jenkins CI: Passed CI tests Greg Sheremeta: Verified Oved Ourfali: Verified -- To view, visit https://gerrit.ovirt.org/44190 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5885dd1c756f09f58addda3670e034cb1b4e4ee5 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: packaging: dbutils: Restore taskcleaner_sp_3_5.sql
by sbonazzo＠redhat.com 30 Jul '15

30 Jul '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: dbutils: Restore taskcleaner_sp_3_5.sql ...................................................................... packaging: dbutils: Restore taskcleaner_sp_3_5.sql Was wrongly removed in commit a1adf1576697d14752ae2e0f73d32e70bb8d93a4 Change-Id: Ieb794b8f8c563d1f87c3dbf9374d081ae6ff95b9 Bug-Url: https://bugzilla.redhat.com/1243125 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- M packaging/setup/dbutils/taskcleaner.sh A packaging/setup/dbutils/taskcleaner_sp_3_5.sql 2 files changed, 71 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Jenkins CI: Passed CI tests Yedidyah Bar David: Verified Oved Ourfali: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/44171 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ieb794b8f8c563d1f87c3dbf9374d081ae6ff95b9 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: Add initial files for 3.5 to master upgrade
by sbonazzo＠redhat.com 30 Jul '15

30 Jul '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: Add initial files for 3.5 to master upgrade ...................................................................... Add initial files for 3.5 to master upgrade Required for http://jenkins.ovirt.org/job/ovirt-engine_master_upgrade-from-3.5_merged Change-Id: I76d33fdbb1e61f264229557a9a812ae00d2d1db9 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- A jobs/ovirt-engine_upgrade_3.5_to_master/build_engine_rpm.sh A jobs/ovirt-engine_upgrade_3.5_to_master/cleanup.file.otopi A jobs/ovirt-engine_upgrade_3.5_to_master/cleanup.sh A jobs/ovirt-engine_upgrade_3.5_to_master/setup.file.otopi A jobs/ovirt-engine_upgrade_3.5_to_master/upgrade.sh 5 files changed, 749 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Passed CI tests; Ready for merge -- To view, visit https://gerrit.ovirt.org/44160 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I76d33fdbb1e61f264229557a9a812ae00d2d1db9 Gerrit-PatchSet: 3 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Anton Marchukov <amarchuk(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Max Kovgan <mvk(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: build: Makefile: sort generated
by sbonazzo＠redhat.com 30 Jul '15

30 Jul '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: Makefile: sort generated ...................................................................... build: Makefile: sort generated Change-Id: If0daddf0552256da7e63beceb004763b059ea341 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M Makefile 1 file changed, 4 insertions(+), 4 deletions(-) Approvals: Alon Bar-Lev: Verified Sandro Bonazzola: Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/44173 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If0daddf0552256da7e63beceb004763b059ea341 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: wgt: publish wgt dependencies rpms
by sbonazzo＠redhat.com 30 Jul '15

30 Jul '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: wgt: publish wgt dependencies rpms ...................................................................... wgt: publish wgt dependencies rpms Change-Id: I9cb8ccf65ee1f1aa00ca727da2a3ef2420bde985 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/yaml/jobs/ovirt/publish-rpms-nightly-master.yaml 1 file changed, 11 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/44178 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9cb8ccf65ee1f1aa00ca727da2a3ef2420bde985 Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: run VDS commands with factory
by ofrenkel＠redhat.com 30 Jul '15

30 Jul '15

Omer Frenkel has submitted this change and it was merged. Change subject: core: run VDS commands with factory ...................................................................... core: run VDS commands with factory Commands should not be directly created and called. Change-Id: I95b5825907aa15880a147d9957533af51907e080 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1245088 Signed-off-by: Omer Frenkel <ofrenkel(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/vdscommands/VDSCommandType.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/CreateVmVDSCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/DestroyVmVDSCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/HibernateVDSCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/MigrateVDSCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/ResumeVDSCommand.java M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VDSCommandBase.java 7 files changed, 60 insertions(+), 70 deletions(-) Approvals: Jenkins CI: Passed CI tests Omer Frenkel: Verified Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/44162 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I95b5825907aa15880a147d9957533af51907e080 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marcin Mirecki <mmirecki(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: webadmin: set empty mgmt network when cluster is detached
by alkaplan＠redhat.com 30 Jul '15

30 Jul '15

Alona Kaplan has submitted this change and it was merged. Change subject: webadmin: set empty mgmt network when cluster is detached ...................................................................... webadmin: set empty mgmt network when cluster is detached Set empty mgmt network when cluster is detached. Change-Id: Ife77c59fd4b91046882dd4451b8b149cf17bf7a1 Bug-Url: https://bugzilla.redhat.com/1206891 Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterModel.java 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Alona Kaplan: Looks good to me, approved; Passed CI tests Yevgeny Zaspitsky: Verified -- To view, visit https://gerrit.ovirt.org/44039 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ife77c59fd4b91046882dd4451b8b149cf17bf7a1 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: jenkins: more gwt dependencies building
by sbonazzo＠redhat.com 30 Jul '15

30 Jul '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: jenkins: more gwt dependencies building ...................................................................... jenkins: more gwt dependencies building Add rpm building for these gwt dependencies: * py2exe-py2.7 * pywin32-py2.7 * spice-qxl * vcredist-x86 Change-Id: I5a22f0d07931f75af7203ea306b8bf3098b71ef7 Signed-off-by: Yedidyah Bar David <didi(a)redhat.com> --- A jobs/confs/yaml/jobs/py2exe-py2.7/py2exe-py2.7.yaml A jobs/confs/yaml/jobs/pywin32-py2.7/pywin32-py2.7.yaml A jobs/confs/yaml/jobs/spice-qxl/spice-qxl.yaml A jobs/confs/yaml/jobs/vcredist-x86/vcredist-x86.yaml 4 files changed, 72 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/43753 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5a22f0d07931f75af7203ea306b8bf3098b71ef7 Gerrit-PatchSet: 3 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Lev Veyde <lveyde(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: fix tab ordering on providers popup
by ahadas＠redhat.com 30 Jul '15

30 Jul '15

Arik Hadas has submitted this change and it was merged. Change subject: webadmin: fix tab ordering on providers popup ...................................................................... webadmin: fix tab ordering on providers popup Fix tab traversal ordering for external providers by assigning the tab indexes properly. Change-Id: I9d2a814247b04fdc1b7706e67e345c426be07e6b Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/provider/ProviderPopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/widget/provider/VmwarePropertiesWidget.java 2 files changed, 26 insertions(+), 0 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/44127 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9d2a814247b04fdc1b7706e67e345c426be07e6b Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: fix the description of import from vmware job
by ahadas＠redhat.com 30 Jul '15

30 Jul '15

Arik Hadas has submitted this change and it was merged. Change subject: core: fix the description of import from vmware job ...................................................................... core: fix the description of import from vmware job The job description for ImportVmFromExternalProvider was missing. This patch adds it with the same description as the one that is used for regular import (from export domain). Change-Id: Iee427d555f3f623f04adfa51733e28db4900553b Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ImportVmCommandBase.java M backend/manager/modules/dal/src/main/resources/bundles/ExecutionMessages.properties 3 files changed, 12 insertions(+), 11 deletions(-) Approvals: Jenkins CI: Passed CI tests Omer Frenkel: Looks good to me, approved Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/44145 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Iee427d555f3f623f04adfa51733e28db4900553b Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0