May 2014 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: core: add NEXT_RUN snapshot
by ofrenkel＠redhat.com 12 May '14

12 May '14

Omer Frenkel has submitted this change and it was merged. Change subject: core: add NEXT_RUN snapshot ...................................................................... core: add NEXT_RUN snapshot Added new snapshot type to save only vm configuration for the vm next run. added also query to get vm from this configuration, and a field in vm to mark this configuration exists. Feature page: http://www.ovirt.org/Features/Edit_Running_VM Change-Id: I4d68f1d7a2de9e66854985583761f47d6db553bd Signed-off-by: Omer Frenkel <ofrenkel(a)redhat.com> --- A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/GetVmNextRunConfigurationQuery.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/Snapshot.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VM.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/queries/VdcQueryType.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/SnapshotDao.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/SnapshotDaoDbFacadeImpl.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmDAODbFacadeImpl.java M packaging/dbscripts/create_views.sql M packaging/dbscripts/snapshots_sp.sql A packaging/dbscripts/upgrade/03_05_0400_index_for_snapshot_type.sql 10 files changed, 98 insertions(+), 6 deletions(-) Approvals: Omer Frenkel: Verified Arik Hadas: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/26757 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4d68f1d7a2de9e66854985583761f47d6db553bd Gerrit-PatchSet: 9 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: Michal Skrivanek <michal.skrivanek(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 Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core, engine: Prevent memory snapshot in ppc64
by rgolan＠redhat.com 12 May '14

12 May '14

Roy Golan has submitted this change and it was merged. Change subject: core, engine: Prevent memory snapshot in ppc64 ...................................................................... core, engine: Prevent memory snapshot in ppc64 This change disables memory snapshot in the back end for clusters with architecture that doesn't support it. All logic was added in conjunction with the snapshot validation from the class FeatureSupported. Change-Id: I4f201b63ddf441a9bc76a37fda33f54fa766d937 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1061774 Signed-off-by: Leonardo Bianconi <leonardo.bianconi(a)eldorado.org.br> Signed-off-by: Vitor de Lima <vitor.lima(a)eldorado.org.br> Signed-off-by: Gustavo Pedrosa <gustavo.pedrosa(a)eldorado.org.br> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CreateAllSnapshotsFromVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/TryBackToAllSnapshotsOfVmCommand.java 3 files changed, 19 insertions(+), 4 deletions(-) Approvals: Roy Golan: Verified; Looks good to me, approved Vered Volansky: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/21657 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4f201b63ddf441a9bc76a37fda33f54fa766d937 Gerrit-PatchSet: 11 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Leonardo Bianconi <leonardo.bianconi(a)eldorado.org.br> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Eli Mesika <elimesika(a)gmail.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Gustavo Frederico Temple Pedrosa <gustavo.pedrosa(a)eldorado.org.br> Gerrit-Reviewer: Leonardo Bianconi <leonardo.bianconi(a)eldorado.org.br> Gerrit-Reviewer: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: Liron Ar <laravot(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Vered Volansky <vvolansk(a)redhat.com> Gerrit-Reviewer: Vitor de Lima <vitor.lima(a)eldorado.org.br> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: liron aravot <liron.aravot(a)gmail.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: restapi: Mandatory "succeeded" for step end
by juan.hernandez＠redhat.com 12 May '14

12 May '14

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Mandatory "succeeded" for step end ...................................................................... restapi: Mandatory "succeeded" for step end The "succeeded" parameter is mandatory for the action to stop a job step. Change-Id: I55ccc732bce26f903275c280192ee83b225bc195 Bug-Url: https://bugzilla.redhat.com/1087916 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendStepResource.java 2 files changed, 2 insertions(+), 1 deletion(-) Approvals: Eli Mesika: Looks good to me, approved Juan Hernandez: Verified -- To view, visit http://gerrit.ovirt.org/27520 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I55ccc732bce26f903275c280192ee83b225bc195 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine-cli[master]: cli: Don't write integer statistics as float
by juan.hernandez＠redhat.com 12 May '14

12 May '14

Juan Hernandez has submitted this change and it was merged. Change subject: cli: Don't write integer statistics as float ...................................................................... cli: Don't write integer statistics as float Currently when we write a an statistic value that is declared to be integer we use the floating point representation used by default by Python. This happens because all the statistics are represented as as xs:decimal, regardless of their declared type, so the generateDS.py tool converts them to float. This has the inconvenient that integer types are displayed with a .0 suffix. For example: # show statistic memory.total --host-identifier myhost id : 7816602b-c05c-3db7-a4da-3769f7ad8896 name : memory.total description : Total memory host-id : d3560912-f2ad-11e1-8f5f-001a4a169798 type : GAUGE unit : BYTES values-type : INTEGER values-value-datum: 33658241024.0 This patch changes the CLI so that it will convert statistics declared as integer to Python long before rendering, so the .0 suffix will not be present. Change-Id: I21490ec50e3a7433331e981162de800c7d548f27 Bug-Url: https://bugzilla.redhat.com/855724 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M src/ovirtcli/format/text.py 1 file changed, 20 insertions(+), 8 deletions(-) Approvals: Michael Pasternak: Looks good to me, approved Juan Hernandez: Verified -- To view, visit http://gerrit.ovirt.org/26864 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I21490ec50e3a7433331e981162de800c7d548f27 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine-cli Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Michael Pasternak <mishka8520(a)yahoo.com> Gerrit-Reviewer: Ravi Nori <rnori(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.4]: engine: Block changing a used label
by oourfali＠redhat.com 12 May '14

12 May '14

Oved Ourfali has submitted this change and it was merged. Change subject: engine: Block changing a used label ...................................................................... engine: Block changing a used label Changing a used label isn't supported and should be blocked until it does. Change-Id: I473ff703c35be305abec7b78c44e983c58d4cb03 Bug-Url: https://bugzilla.redhat.com/1091714 Signed-off-by: Moti Asayag <masayag(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/network/dc/UpdateNetworkCommand.java 1 file changed, 27 insertions(+), 5 deletions(-) Approvals: Mike Kolesnik: Looks good to me, approved Moti Asayag: Verified -- To view, visit http://gerrit.ovirt.org/27571 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I473ff703c35be305abec7b78c44e983c58d4cb03 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: webadmin: Fix NPE in edit network popup.
by lvernia＠redhat.com 12 May '14

12 May '14

Lior Vernia has submitted this change and it was merged. Change subject: webadmin: Fix NPE in edit network popup. ...................................................................... webadmin: Fix NPE in edit network popup. Set network.selectedItem for both HostManagementNetworkModel and HostInterfaceModel. Change-Id: I2188dd71037777eb0099d24e0d244589551986d5 Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostSetupNetworksModel.java 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Lior Vernia: Looks good to me, approved Yevgeny Zaspitsky: Verified -- To view, visit http://gerrit.ovirt.org/27565 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2188dd71037777eb0099d24e0d244589551986d5 Gerrit-PatchSet: 1 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: Lior Vernia <lvernia(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: engine: Update spec openstack SDK to 3.0.4
by mkolesni＠redhat.com 12 May '14

12 May '14

Mike Kolesnik has submitted this change and it was merged. Change subject: engine: Update spec openstack SDK to 3.0.4 ...................................................................... engine: Update spec openstack SDK to 3.0.4 Network creation code updated to use the non-deprecated entities. Change-Id: I4bd7373006a996fd1bb6c9efc5927fabcada81e3 Related-To: https://bugzilla.redhat.com/1064749 Signed-off-by: Mike Kolesnik <mkolesni(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/provider/network/openstack/OpenstackNetworkProviderProxy.java M ovirt-engine.spec.in 2 files changed, 5 insertions(+), 6 deletions(-) Approvals: Mike Kolesnik: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/27347 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4bd7373006a996fd1bb6c9efc5927fabcada81e3 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Mike Kolesnik <mkolesni(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-dwh[master]: packaging: setup: remove unused constant
by ydary＠redhat.com 12 May '14

12 May '14

Yaniv Dary has submitted this change and it was merged. Change subject: packaging: setup: remove unused constant ...................................................................... packaging: setup: remove unused constant we do not use jboss any more. Change-Id: Ie32f824b8d166270c822d7253df66cee43688b79 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/dwhconstants.py 1 file changed, 0 insertions(+), 4 deletions(-) Approvals: Alon Bar-Lev: Verified Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/27562 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie32f824b8d166270c822d7253df66cee43688b79 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-dwh Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.4]: core: Use configuration to represent internal domain
by oourfali＠redhat.com 12 May '14

12 May '14

Oved Ourfali has submitted this change and it was merged. Change subject: core: Use configuration to represent internal domain ...................................................................... core: Use configuration to represent internal domain Change-Id: Ie9b4c042feb7243621f119ffc6c4c86ad88d09ad Bug-Url: https://bugzilla.redhat.com/1095420 Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/adbroker/InternalBrokerUtils.java 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/27576 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie9b4c042feb7243621f119ffc6c4c86ad88d09ad Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.4.1]: core: Fix admin@internal login
by oourfali＠redhat.com 12 May '14

12 May '14

Oved Ourfali has submitted this change and it was merged. Change subject: core: Fix admin@internal login ...................................................................... core: Fix admin@internal login This fixes InternalBrokerUtils to query for users not only by user name but also domain. This will allow proper login in case there are several users with the same name, but in different domains Bug-Url: https://bugzilla.redhat.com/1095420 Change-Id: Ifb2455380242a7b409dd8ea0376e049127996d09 Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/adbroker/InternalBrokerUtils.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/DbUserDAO.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/DbUserDAODbFacadeImpl.java M packaging/dbscripts/user_sp.sql 4 files changed, 29 insertions(+), 1 deletion(-) Approvals: Oved Ourfali: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/27575 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifb2455380242a7b409dd8ea0376e049127996d09 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4.1 Gerrit-Owner: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0