January 2014 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: tools: managed-domains: Fix typo occured->occurred
by yzaslavs＠redhat.com 25 Jan '14

25 Jan '14

Yair Zaslavsky has submitted this change and it was merged. Change subject: tools: managed-domains: Fix typo occured->occurred ...................................................................... tools: managed-domains: Fix typo occured->occurred Fix typo in the usage message and man page. Change-Id: Id4f699e91730844ea09a99c943d9a3f52a68f91f Bug-Url: https://bugzilla.redhat.com/1057176 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M packaging/bin/engine-manage-domains.sh M packaging/man/man8/engine-manage-domains.8 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Yair Zaslavsky: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/23671 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id4f699e91730844ea09a99c943d9a3f52a68f91f Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.3.3]: packaging: setup: use arch limit for shmmax
by sbonazzo＠redhat.com 24 Jan '14

24 Jan '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: use arch limit for shmmax ...................................................................... packaging: setup: use arch limit for shmmax If not already set, set shmmax to arch limit. Change-Id: Idbc9b651d2f9e47e58c75b44bd84418990f59d60 Bug-Url: https://bugzilla.redhat.com/1039616 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> (cherry picked from commit bbecf5779848c12c7a03fe1229017cddcac5a9c8) --- M packaging/setup/ovirt_engine_setup/constants.py 1 file changed, 10 insertions(+), 1 deletion(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/23654 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idbc9b651d2f9e47e58c75b44bd84418990f59d60 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3.3 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.3]: packaging: setup: use arch limit for shmmax
by iheim＠redhat.com 24 Jan '14

24 Jan '14

Itamar Heim has submitted this change and it was merged. Change subject: packaging: setup: use arch limit for shmmax ...................................................................... packaging: setup: use arch limit for shmmax If not already set, set shmmax to arch limit. Change-Id: Idbc9b651d2f9e47e58c75b44bd84418990f59d60 Bug-Url: https://bugzilla.redhat.com/1039616 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> (cherry picked from commit bbecf5779848c12c7a03fe1229017cddcac5a9c8) --- M packaging/setup/ovirt_engine_setup/constants.py 1 file changed, 10 insertions(+), 1 deletion(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/23653 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idbc9b651d2f9e47e58c75b44bd84418990f59d60 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.3 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(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]: packaging: setup: use arch limit for shmmax
by iheim＠redhat.com 24 Jan '14

24 Jan '14

Itamar Heim has submitted this change and it was merged. Change subject: packaging: setup: use arch limit for shmmax ...................................................................... packaging: setup: use arch limit for shmmax If not already set, set shmmax to arch limit. Change-Id: Idbc9b651d2f9e47e58c75b44bd84418990f59d60 Bug-Url: https://bugzilla.redhat.com/1039616 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> (cherry picked from commit bbecf5779848c12c7a03fe1229017cddcac5a9c8) --- M packaging/setup/ovirt_engine_setup/constants.py 1 file changed, 10 insertions(+), 1 deletion(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Sandro Bonazzola: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/23652 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idbc9b651d2f9e47e58c75b44bd84418990f59d60 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: packaging: setup: restart apache if ssl config changed
by sbonazzo＠redhat.com 24 Jan '14

24 Jan '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: restart apache if ssl config changed ...................................................................... packaging: setup: restart apache if ssl config changed we restart it anyway as we overwrite the proxy configuration, but for consistency this should be added. Change-Id: I5aabb084ce5fd9629bfe8a2c086760a6568e6308 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/apache/ssl.py 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Alon Bar-Lev: Verified Sandro Bonazzola: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/23657 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5aabb084ce5fd9629bfe8a2c086760a6568e6308 Gerrit-PatchSet: 1 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>

1 0

Change in ovirt-engine[master]: frontend: Clarify CPU labels in host and cluster views
by tjelinek＠redhat.com 24 Jan '14

24 Jan '14

Tomas Jelinek has submitted this change and it was merged. Change subject: frontend: Clarify CPU labels in host and cluster views ...................................................................... frontend: Clarify CPU labels in host and cluster views This patch clarifies naming of labels and fields related to CPU. In past there were two fields with cpu information: - name - type. The naming of these fields didn't make any sense since "name" field contained value with cpu family whereas "type" field contained actual name of the cpu model. In order to avoid the confusion, this patch renames the fields in this way: type -> model name -> type. Change-Id: I5137dbb61dcc07dc7d5ccced390d94ddc1c9ba4e Signed-off-by: Frantisek Kobzik <fkobzik(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/1056586 --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/clusters/ClusterGeneralModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostGeneralModel.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/cluster/ClusterPopupView.ui.xml M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/HostConfigureLocalStoragePopupView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/host/HostConfigureLocalStoragePopupView.ui.xml M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/MainTabClusterView.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/cluster/ClusterGeneralModelForm.java M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/host/SubTabHostGeneralView.java 10 files changed, 43 insertions(+), 43 deletions(-) Approvals: Tomas Jelinek: Verified; Looks good to me, approved Frank Kobzik: Verified -- To view, visit http://gerrit.ovirt.org/20111 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5137dbb61dcc07dc7d5ccced390d94ddc1c9ba4e Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Frank Kobzik <fkobzik(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Frank Kobzik <fkobzik(a)redhat.com> Gerrit-Reviewer: Gilad Chaplik <gchaplik(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: tools: Fixes empty provider option when editing domain
by yzaslavs＠redhat.com 24 Jan '14

24 Jan '14

Yair Zaslavsky has submitted this change and it was merged. Change subject: tools: Fixes empty provider option when editing domain ...................................................................... tools: Fixes empty provider option when editing domain Fixes error during engine-manage-domains edit action when provider is not entered. Change-Id: I79effa66047579b0bc5aa8772a6e90dc849a4cd3 Bug-Url: https://bugzilla.redhat.com/1055417 Signed-off-by: Martin Perina <mperina(a)redhat.com> --- M backend/manager/tools/src/main/java/org/ovirt/engine/core/domains/ManageDomains.java 1 file changed, 4 insertions(+), 1 deletion(-) Approvals: Martin Peřina: Verified Yair Zaslavsky: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/23655 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I79effa66047579b0bc5aa8772a6e90dc849a4cd3 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: setup: use arch limit for shmmax
by sbonazzo＠redhat.com 24 Jan '14

24 Jan '14

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: setup: use arch limit for shmmax ...................................................................... packaging: setup: use arch limit for shmmax If not already set, set shmmax to arch limit. Change-Id: Idbc9b651d2f9e47e58c75b44bd84418990f59d60 Bug-Url: https://bugzilla.redhat.com/1039616 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/constants.py 1 file changed, 10 insertions(+), 1 deletion(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Sandro Bonazzola: Verified; Looks good to me, approved Yaniv Dary: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/23641 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idbc9b651d2f9e47e58c75b44bd84418990f59d60 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: Yedidyah Bar David <didi(a)redhat.com>

1 0

Change in ovirt-engine[ovirt-engine-3.4]: core: add template version related fields to vm and template
by iheim＠redhat.com 24 Jan '14

24 Jan '14

Itamar Heim has submitted this change and it was merged. Change subject: core: add template version related fields to vm and template ...................................................................... core: add template version related fields to vm and template for templates, added templateVersionNumber, templateVersionName and baseTemplateId fot VMs, added useLatestVersion these fields are used for supporting templates versions, as documented in the Template Versions feature page http://www.ovirt.org/index.php?title=Features/Template_Versions Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1037478 Change-Id: I08f13eee5609525880a92fc81c9fd3cb1d2ba62d Signed-off-by: Omer Frenkel <ofrenkel(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddVmTemplateCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/AddVmTemplateParameters.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/businessentities/VmStatic.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VmTemplate.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmDAODbFacadeImpl.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmStaticDAODbFacadeImpl.java M backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/VmTemplateDAODbFacadeImpl.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/FixturesTool.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VmDAOTest.java M backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/VmTemplateDAOTest.java M backend/manager/modules/dal/src/test/resources/fixtures.xml M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/Cloner.java M packaging/dbscripts/create_views.sql M packaging/dbscripts/storages_sp.sql A packaging/dbscripts/upgrade/03_04_0540_add_template_version_to_vm_static.sql M packaging/dbscripts/vm_templates_sp.sql M packaging/dbscripts/vms_sp.sql 18 files changed, 468 insertions(+), 31 deletions(-) Approvals: Itamar Heim: Verified; Looks good to me, approved Omer Frenkel: Verified -- To view, visit http://gerrit.ovirt.org/23606 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I08f13eee5609525880a92fc81c9fd3cb1d2ba62d Gerrit-PatchSet: 6 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Omer Frenkel <ofrenkel(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Martin Betak <mbetak(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[ovirt-engine-3.4]: userportal: Update Vm.comment field on edit VM
by iheim＠redhat.com 24 Jan '14

24 Jan '14

Itamar Heim has submitted this change and it was merged. Change subject: userportal: Update Vm.comment field on edit VM ...................................................................... userportal: Update Vm.comment field on edit VM Change-Id: I125478ef32de5cefa16d8a5568b468c361f520bd Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1056567 Signed-off-by: Martin Betak <mbetak(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/userportal/UserPortalListModel.java 1 file changed, 1 insertion(+), 0 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Martin Betak: Verified -- To view, visit http://gerrit.ovirt.org/23637 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I125478ef32de5cefa16d8a5568b468c361f520bd Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Itamar Heim <iheim(a)redhat.com> Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0