March 2016 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: setup: fixing setup when engine option is not enabled
by sbonazzo＠redhat.com 23 Mar '16

23 Mar '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: setup: fixing setup when engine option is not enabled ...................................................................... setup: fixing setup when engine option is not enabled The user can also install engine-setup rpms and respond No to Configure Engine on this host (Yes, No). Supporting that path. Change-Id: I38dcd682946485f97f704f99ca6d51c5c4efb0d2 Bug-Url: https://bugzilla.redhat.com/1318986 Signed-off-by: Simone Tiraboschi <stirabos(a)redhat.com> --- M packaging/setup/ovirt_engine_setup/engine_common/constants.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/config/storage.py M packaging/setup/plugins/ovirt-engine-setup/ovirt-engine/pki/ca.py M packaging/setup/plugins/ovirt-engine-setup/vmconsole_proxy_helper/config.py 4 files changed, 40 insertions(+), 20 deletions(-) Approvals: Sandro Bonazzola: Looks good to me, approved Simone Tiraboschi: Verified Jenkins CI: Passed CI tests Yedidyah Bar David: Looks good to me, but someone else must approve Francesco Romani: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/54912 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I38dcd682946485f97f704f99ca6d51c5c4efb0d2 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: Francesco Romani <fromani(a)redhat.com> Gerrit-Reviewer: Jenkins CI 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> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[ovirt-4.0]: The glusterfs key was changed to rsa
by sbonazzo＠redhat.com 23 Mar '16

23 Mar '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: The glusterfs key was changed to rsa ...................................................................... The glusterfs key was changed to rsa https://download.gluster.org/pub/gluster/glusterfs/LATEST/NEW_PUBLIC_KEY.RE… Change-Id: If34c154f955dae83b16a69b1e2fedaab62f21c2f Signed-off-by: Tolik Litovsky <tlitovsk(a)redhat.com> --- M ovirt-el7-deps.repo.in 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/55121 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If34c154f955dae83b16a69b1e2fedaab62f21c2f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-4.0 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[ovirt-3.6]: build: post ovirt-release36-006
by sbonazzo＠redhat.com 23 Mar '16

23 Mar '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: post ovirt-release36-006 ...................................................................... build: post ovirt-release36-006 Change-Id: Ia5ca69378c34e6ece7890a66de4bd9fe9cb35e23 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M automation/build-artifacts.sh M automation/check-patch.sh M configure.ac 3 files changed, 9 insertions(+), 3 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/55118 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia5ca69378c34e6ece7890a66de4bd9fe9cb35e23 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-3.6 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[master]: build: bump to 0.3 due to glusterfs key changes
by sbonazzo＠redhat.com 23 Mar '16

23 Mar '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: bump to 0.3 due to glusterfs key changes ...................................................................... build: bump to 0.3 due to glusterfs key changes Change-Id: I8d028b3fa536ec32379db6b6847fda4a57bdb612 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M configure.ac M ovirt-release-master.spec.in 2 files changed, 4 insertions(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/55120 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8d028b3fa536ec32379db6b6847fda4a57bdb612 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: webadmin: Avoid UpdateVmDiskCommand for hosted engine Edit VM
by tjelinek＠redhat.com 23 Mar '16

23 Mar '16

Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin: Avoid UpdateVmDiskCommand for hosted engine Edit VM ...................................................................... webadmin: Avoid UpdateVmDiskCommand for hosted engine Edit VM After OK in Edit VM Dialog of a hosted engine: - The UpdateVmDiskCommand is not called - Disk-edit buttons are disabled Change-Id: I4ffaa7725039d58f55e88ef22e586349323d70a9 Bug-Url: https://bugzilla.redhat.com/1311908 Signed-off-by: Marek Libra <mlibra(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/ExistingVmModelBehavior.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/InstanceImageLineModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/InstanceImagesModel.java 3 files changed, 24 insertions(+), 2 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Jenkins CI: Passed CI tests Marek Libra: Verified -- To view, visit https://gerrit.ovirt.org/54755 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4ffaa7725039d58f55e88ef22e586349323d70a9 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Marek Libra <mlibra(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Marek Libra <mlibra(a)redhat.com> Gerrit-Reviewer: Martin Betak <mbetak(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Shmuel Leib Melamud <smelamud(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: HE: Set default priority and timezone when importing hosted ...
by rgolan＠redhat.com 23 Mar '16

23 Mar '16

Roy Golan has submitted this change and it was merged. Change subject: HE: Set default priority and timezone when importing hosted engine VM ...................................................................... HE: Set default priority and timezone when importing hosted engine VM Also fix the fields to defaults if missing in already imported HE VM. Bug-Url: https://bugzilla.redhat.com/1160094 Change-Id: I40d6396fcc767e041466ace758d2a8fe0631aeef Signed-off-by: Martin Sivak <msivak(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/HostedEngineImporter.java A packaging/dbscripts/upgrade/04_00_0350_set_timezone_priority_for_he.sql 2 files changed, 14 insertions(+), 0 deletions(-) Approvals: Martin Sivák: Verified Jenkins CI: Passed CI tests Roy Golan: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/54803 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I40d6396fcc767e041466ace758d2a8fe0631aeef Gerrit-PatchSet: 13 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Eli Mesika <emesika(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com> Gerrit-Reviewer: Yaniv Kaul <ykaul(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-release[ovirt-3.6]: build: ovirt-release36-006
by sbonazzo＠redhat.com 23 Mar '16

23 Mar '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: ovirt-release36-006 ...................................................................... build: ovirt-release36-006 Change-Id: Ibc7b1dbeeab0681c212f13653e059c26acc98917 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M automation/build-artifacts.sh M automation/check-patch.sh M configure.ac M ovirt-release36.spec.in 4 files changed, 5 insertions(+), 8 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/55115 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibc7b1dbeeab0681c212f13653e059c26acc98917 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-release Gerrit-Branch: ovirt-3.6 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: HE: Fix next run need detection bug
by rgolan＠redhat.com 23 Mar '16

23 Mar '16

Roy Golan has submitted this change and it was merged. Change subject: HE: Fix next run need detection bug ...................................................................... HE: Fix next run need detection bug Missing parentheses in the condition caused the logic to be wrong. Since the condition was too complicated I decided to split it in two. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1160094 Change-Id: Ia0144801dbe367191416d9ebc3c30607f0bf6c8f Signed-off-by: Martin Sivak <msivak(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmCommand.java 1 file changed, 6 insertions(+), 2 deletions(-) Approvals: Martin Sivák: Verified Jenkins CI: Passed CI tests Roy Golan: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/54919 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia0144801dbe367191416d9ebc3c30607f0bf6c8f Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Jenny Tokar <jtokar(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-live[ovirt-live-3.6]: build: oVirt Live 3.6.4 rc2
by sbonazzo＠redhat.com 22 Mar '16

22 Mar '16

Sandro Bonazzola has submitted this change and it was merged. Change subject: build: oVirt Live 3.6.4 rc2 ...................................................................... build: oVirt Live 3.6.4 rc2 Change-Id: Ib2fc06a644debc552432b94dda98230c727ee3d2 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M centos-7/Makefile 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/55081 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib2fc06a644debc552432b94dda98230c727ee3d2 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: ovirt-live-3.6 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: No StorageType in VersionStorageFormatUtil
by amureini＠redhat.com 22 Mar '16

22 Mar '16

Allon Mureinik has submitted this change and it was merged. Change subject: core: No StorageType in VersionStorageFormatUtil ...................................................................... core: No StorageType in VersionStorageFormatUtil VersionStorageFormatUtil is supposed to map between a logical DC level (e.g., 3.6) to a StorageFormatType constant (e.g. V3). The entire point of having format versions is to transcend the differences of various storage types. DC level 3.1 failed to do so, and mixed V1 (for file based storage )and V2 (for block storage) in the DC level. With their removal, the interface can now be cleaned up, and simply map a version to a format. Change-Id: Icd9250fc0ff88cc3776b8c5ef5621768e26bd2a3 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/domain/AddLocalStorageDomainCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/domain/AddStorageDomainCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/pool/AddStoragePoolWithStoragesCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/pool/UpdateStoragePoolCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/storage/StorageDomainToPoolRelationValidator.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/utils/VersionStorageFormatUtil.java M backend/manager/modules/common/src/test/java/org/ovirt/engine/core/common/utils/VersionStorageFormatUtilTest.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterListModel.java 8 files changed, 22 insertions(+), 68 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Allon Mureinik: Verified; Passed CI tests Freddy Rolland: Looks good to me, but someone else must approve Idan Shaby: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/54851 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icd9250fc0ff88cc3776b8c5ef5621768e26bd2a3 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Ala Hino <ahino(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Amit Aviram <aaviram(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com> Gerrit-Reviewer: Idan Shaby <ishaby(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0