January 2014 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: packaging: setup: prefix database related messages with 'Eng...
by sbonazzo＠redhat.com 08 Jan '14

08 Jan '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: prefix database related messages with 'Engine' ...................................................................... packaging: setup: prefix database related messages with 'Engine' reduce confusion with other databases we going to have. Change-Id: I7c486ecf94e74cece79ba9260c50ac9ab1ba1d05 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/constants.py M packaging/setup/plugins/ovirt-engine-common/base/db/connection.py M packaging/setup/plugins/ovirt-engine-remove/ovirt-engine/db/clear.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-common/db/connection.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/db/schema.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/upgrade/dbvalidations.py 6 files changed, 43 insertions(+), 43 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22985 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7c486ecf94e74cece79ba9260c50ac9ab1ba1d05 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: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: core: Reset emulated machines after architecture changing
by tjelinek＠redhat.com 08 Jan '14

08 Jan '14

Tomas Jelinek has submitted this change and it was merged. Change subject: core: Reset emulated machines after architecture changing ...................................................................... core: Reset emulated machines after architecture changing After PPC patches, cluster received a new parameter, the architecture one. As it affects the parameter "emulated machine", when a change occurs on CPU name, which results in an architecture change, the parameter "emulated machine" must be updated. This patch introduces the change to set the cluster to reset this parameter when adding the first host. Important: It will happen only with empty clusters, since there are validations when changing CPU name for other architecture. Change-Id: I1d400b8d475f63e720ad88ad00116aea9720f7ba Signed-off-by: Leonardo Bianconi <leonardo.bianconi(a)eldorado.org.br> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVdsGroupCommand.java 1 file changed, 6 insertions(+), 0 deletions(-) Approvals: Tomas Jelinek: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22652 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1d400b8d475f63e720ad88ad00116aea9720f7ba Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Leonardo Bianconi <leonardo.bianconi(a)eldorado.org.br> Gerrit-Reviewer: Gustavo Frederico Temple Pedrosa <gustavo.pedrosa(a)eldorado.org.br> Gerrit-Reviewer: Leonardo Bianconi <leonardo.bianconi(a)eldorado.org.br> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Vitor de Lima <vitor.lima(a)eldorado.org.br> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: setup: leave more space for titles at configurati...
by sbonazzo＠redhat.com 08 Jan '14

08 Jan '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: leave more space for titles at configuration preview ...................................................................... packaging: setup: leave more space for titles at configuration preview Change-Id: If3fcdaa9052e0c21af033752d5018594ea7d86bd Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/dialog/preview.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22984 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If3fcdaa9052e0c21af033752d5018594ea7d86bd Gerrit-PatchSet: 2 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: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: setup: print all registered variable at configura...
by sbonazzo＠redhat.com 08 Jan '14

08 Jan '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: print all registered variable at configuration preview ...................................................................... packaging: setup: print all registered variable at configuration preview Change-Id: I1fe43f6d6fc1f7ddd99b0f761dd866f8e5b0f16a Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/dialog/preview.py 1 file changed, 9 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22983 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1fe43f6d6fc1f7ddd99b0f761dd866f8e5b0f16a Gerrit-PatchSet: 2 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: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: setup: db: move create_schema.sh as constant
by sbonazzo＠redhat.com 08 Jan '14

08 Jan '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: db: move create_schema.sh as constant ...................................................................... packaging: setup: db: move create_schema.sh as constant Change-Id: I1861c88b198916c4e4de76458a4bbacc70e02467 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/constants.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/db/schema.py 2 files changed, 5 insertions(+), 4 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22982 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1861c88b198916c4e4de76458a4bbacc70e02467 Gerrit-PatchSet: 2 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: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.3]: core: [Fix] Disabled Balloon in Add Vm
by iheim＠redhat.com 08 Jan '14

08 Jan '14

Itamar Heim has submitted this change and it was merged. Change subject: core: [Fix] Disabled Balloon in Add Vm ...................................................................... core: [Fix] Disabled Balloon in Add Vm Currently, it is impossible to add a VM with balloon disabled (REST/UI), this change fixes this bug. Change-Id: I508ff77c1e309be0b492f46c31311e03f33fc728 Bug-Url : https://bugzilla.redhat.com/show_bug.cgi?id=1049343 Signed-off-by: Gustavo Pedrosa <gustavo.pedrosa(a)eldorado.org.br> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmFromSnapshotCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmTemplateCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/utils/VmDeviceUtils.java 4 files changed, 23 insertions(+), 3 deletions(-) Approvals: Gustavo Frederico Temple Pedrosa: Verified Itamar Heim: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/23039 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7c9526469351a250e2cd8f2261c0dedcfa9c0bad Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Gustavo Frederico Temple Pedrosa <gustavo.pedrosa(a)eldorado.org.br> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(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: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak(a)redhat.com> Gerrit-Reviewer: Leonardo Bianconi <leonardo.bianconi(a)eldorado.org.br> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Vitor de Lima <vitor.lima(a)eldorado.org.br> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: engine : preserve the HA flag when saving/loading the VM ovf...
by dfediuck＠redhat.com 08 Jan '14

08 Jan '14

Doron Fediuck has submitted this change and it was merged. Change subject: engine : preserve the HA flag when saving/loading the VM ovf file ...................................................................... engine : preserve the HA flag when saving/loading the VM ovf file This patch changes the OvfVMReader and OvfVMWriter to preserve the HA flag when exporting or importing the VM Change-Id: I38f55ffc7e7454287121463a90f0437a6b590125 Bug-Url: https://bugzilla.redhat.com/1036746 Signed-off-by: Jiri Moskovcak <jmoskovc(a)redhat.com> --- M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfVmReader.java M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ovf/OvfVmWriter.java 2 files changed, 8 insertions(+), 0 deletions(-) Approvals: Doron Fediuck: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22238 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I38f55ffc7e7454287121463a90f0437a6b590125 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Jiří Moskovčák <jmoskovc(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: setup: db/connection: register transaction at setup
by sbonazzo＠redhat.com 08 Jan '14

08 Jan '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: db/connection: register transaction at setup ...................................................................... packaging: setup: db/connection: register transaction at setup main transaction is not available at init. Change-Id: Ie486a4a3e566b6607e63061d10aa8a11dc4f5815 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine-common/db/connection.py 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22921 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie486a4a3e566b6607e63061d10aa8a11dc4f5815 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: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: setup: config/database: no need to wait for any o...
by sbonazzo＠redhat.com 08 Jan '14

08 Jan '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: config/database: no need to wait for any other stage ...................................................................... packaging: setup: config/database: no need to wait for any other stage Change-Id: Idb87cb10eba322cbbfa1d765ecd6f393645e780e Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/config/database.py 1 file changed, 0 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22916 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idb87cb10eba322cbbfa1d765ecd6f393645e780e 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: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: setup: remove CONFIG_DB_CREDENTIALS stage name
by sbonazzo＠redhat.com 08 Jan '14

08 Jan '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: remove CONFIG_DB_CREDENTIALS stage name ...................................................................... packaging: setup: remove CONFIG_DB_CREDENTIALS stage name apparently it is not required. Change-Id: Ib650f95b16177a7eff3565dc3ebe541d70924df0 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/constants.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/config/database.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/db/schema.py 3 files changed, 0 insertions(+), 5 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/22914 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib650f95b16177a7eff3565dc3ebe541d70924df0 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: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0