- Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-3.6.2]: frontend: don't auto format sd when detaching from local dc
by tnisan＠redhat.com 23 Dec '15

23 Dec '15

Tal Nisan has submitted this change and it was merged. Change subject: frontend: don't auto format sd when detaching from local dc ...................................................................... frontend: don't auto format sd when detaching from local dc This patch adds the user the ability to choose whether or not to format the storage domain when detaching it from a local data center. This is done by presenting him a checkbox with the label "Format Domain, i.e. Storage Content will be lost!", like in the "Remove Storage(s)" window. Change-Id: I78526987f2b6df1e76423c2b74b0d0a5be26032e Bug-Url: https://bugzilla.redhat.com/1245249 Signed-off-by: Idan Shaby <ishaby(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterStorageListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageDataCenterListModel.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java 3 files changed, 13 insertions(+), 8 deletions(-) Approvals: Tal Nisan: Looks good to me, approved; Passed CI tests Freddy Rolland: Looks good to me, but someone else must approve Idan Shaby: Verified -- To view, visit https://gerrit.ovirt.org/50985 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I78526987f2b6df1e76423c2b74b0d0a5be26032e Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6.2 Gerrit-Owner: Idan Shaby <ishaby(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(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

Change in ovirt-engine[ovirt-engine-3.6.2]: webadmin: import glance image - fix size value
by tnisan＠redhat.com 23 Dec '15

23 Dec '15

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: import glance image - fix size value ...................................................................... webadmin: import glance image - fix size value Import image from Glance dialog: fix to appropriate size units. Change-Id: Ibe5fdb079d72449cf92a47da7088676573d40e42 Bug-Url: https://bugzilla.redhat.com/1290349 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/ImportExportImagePopupView.java 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Daniel Erez: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/50975 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibe5fdb079d72449cf92a47da7088676573d40e42 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6.2 Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(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

Change in ovirt-engine[ovirt-engine-3.6]: frontend: don't auto format sd when detaching from local dc
by tnisan＠redhat.com 23 Dec '15

23 Dec '15

Tal Nisan has submitted this change and it was merged. Change subject: frontend: don't auto format sd when detaching from local dc ...................................................................... frontend: don't auto format sd when detaching from local dc This patch adds the user the ability to choose whether or not to format the storage domain when detaching it from a local data center. This is done by presenting him a checkbox with the label "Format Domain, i.e. Storage Content will be lost!", like in the "Remove Storage(s)" window. Change-Id: I78526987f2b6df1e76423c2b74b0d0a5be26032e Bug-Url: https://bugzilla.redhat.com/1245249 Signed-off-by: Idan Shaby <ishaby(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/datacenters/DataCenterStorageListModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/StorageDataCenterListModel.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java 3 files changed, 13 insertions(+), 8 deletions(-) Approvals: Jenkins CI: Passed CI tests Daniel Erez: Looks good to me, approved Idan Shaby: Verified -- To view, visit https://gerrit.ovirt.org/50976 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I78526987f2b6df1e76423c2b74b0d0a5be26032e Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Idan Shaby <ishaby(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(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

Change in ovirt-engine[ovirt-engine-3.6]: webadmin: import glance image - fix size value
by tnisan＠redhat.com 23 Dec '15

23 Dec '15

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: import glance image - fix size value ...................................................................... webadmin: import glance image - fix size value Import image from Glance dialog: fix to appropriate size units. Change-Id: Ibe5fdb079d72449cf92a47da7088676573d40e42 Bug-Url: https://bugzilla.redhat.com/1290349 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/ImportExportImagePopupView.java 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Daniel Erez: Verified -- To view, visit https://gerrit.ovirt.org/50974 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ibe5fdb079d72449cf92a47da7088676573d40e42 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(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

Change in ovirt-engine[master]: core: DiskImage.hasRawBlock()
by amureini＠redhat.com 23 Dec '15

23 Dec '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: DiskImage.hasRawBlock() ...................................................................... core: DiskImage.hasRawBlock() isRawBlock() is used to check whether a DiskImage needs refreshing on VDSM's side before an action is performed on it. Theoretically, a DiskImage can have several copies on several domains, and only some of them may be block domains (thus requiring a refresh). This patch removes the assumption that the block domain would necessarily be the first domain returned in getStorageTypes() and instead checks all the domains to see if at least one of them is a block domain. Currently, this situation is only possible for templates, which don't participate in Live Merging, so this patch is more of a future-proofing effort than an actual bugfix. Change-Id: Ief879683c433e52efcf4c2d2dc43db81f0475835 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MergeExtendCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/image/ExtendImageSizeCommand.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/storage/DiskImage.java 3 files changed, 12 insertions(+), 4 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Greg Padgett: Looks good to me, but someone else must approve Allon Mureinik: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/50477 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ief879683c433e52efcf4c2d2dc43db81f0475835 Gerrit-PatchSet: 6 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: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Move ImagesHandler#isDiskImageRawBlock
by amureini＠redhat.com 23 Dec '15

23 Dec '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: Move ImagesHandler#isDiskImageRawBlock ...................................................................... core: Move ImagesHandler#isDiskImageRawBlock Since this method is just a convenient accessor to the DiskImage's methods it makes more sense to just put it there instead of having it in ImagesHandler. Change-Id: Ie66e7a85ed159f04fdfa04079d2669057a8cbcb4 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MergeExtendCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/image/ExtendImageSizeCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/image/ImagesHandler.java M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/storage/DiskImage.java 4 files changed, 6 insertions(+), 8 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Greg Padgett: Looks good to me, but someone else must approve Allon Mureinik: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/50476 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie66e7a85ed159f04fdfa04079d2669057a8cbcb4 Gerrit-PatchSet: 4 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: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: ImagesHandler#isDiskImageRawBlock performance
by amureini＠redhat.com 23 Dec '15

23 Dec '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: ImagesHandler#isDiskImageRawBlock performance ...................................................................... core: ImagesHandler#isDiskImageRawBlock performance Since a DiskImage object has the getStorageTypes() methods, there's no need to re-read them from the database, and the in-memory values could be used to improve performance. Change-Id: Id031970a8ef5a79c185f5dd94f496384c8032a6a Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/image/ImagesHandler.java 1 file changed, 1 insertion(+), 2 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Greg Padgett: Looks good to me, but someone else must approve Allon Mureinik: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/50475 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id031970a8ef5a79c185f5dd94f496384c8032a6a 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: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Remove VM#runOnce
by amureini＠redhat.com 23 Dec '15

23 Dec '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: Remove VM#runOnce ...................................................................... core: Remove VM#runOnce The private field VM#runOnce is never set, and is only read in the hashCode() method. Hence, it could savely be removed. It's worth noting that a VM actually does have a runOnce property exposed by the isRunOnce() and setRunOnce(boolean) methods, but it's value is stored by the VmDynamic object the VM holds. Change-Id: Icbf21832633d0b0ff86abde7af3fae4388c8180b Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VM.java 1 file changed, 0 insertions(+), 2 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Allon Mureinik: Verified; Passed CI tests -- To view, visit https://gerrit.ovirt.org/50956 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icbf21832633d0b0ff86abde7af3fae4388c8180b Gerrit-PatchSet: 3 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: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: backend: avoid cast exception in VdsBrokerObjectsBuilder
by alkaplan＠redhat.com 23 Dec '15

23 Dec '15

Alona Kaplan has submitted this change and it was merged. Change subject: backend: avoid cast exception in VdsBrokerObjectsBuilder ...................................................................... backend: avoid cast exception in VdsBrokerObjectsBuilder VdsBrokerObjectsBuilder.updateCommonInterfaceData can return a String or an Integer (depends on the vdsm version). This patch enables us to work with both return types. Change-Id: I4932f924288b15b11000e6e3b1d4cee83f7cf50b Bug-Url: https://bugzilla.redhat.com/1293881 Signed-off-by: Idan Shaby <ishaby(a)redhat.com> --- M backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/vdsbroker/VdsBrokerObjectsBuilder.java 1 file changed, 5 insertions(+), 3 deletions(-) Approvals: Alona Kaplan: Looks good to me, approved; Passed CI tests Idan Shaby: Verified -- To view, visit https://gerrit.ovirt.org/50968 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4932f924288b15b11000e6e3b1d4cee83f7cf50b Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Idan Shaby <ishaby(a)redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com> Gerrit-Reviewer: Alona Kaplan <alkaplan(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: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: change description of hibernation disks
by ahadas＠redhat.com 23 Dec '15

23 Dec '15

Arik Hadas has submitted this change and it was merged. Change subject: core: change description of hibernation disks ...................................................................... core: change description of hibernation disks Change-Id: If5bcefb621d4bc5b41dd61ac3d9b55e6780f2cc0 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/memory/MemoryUtils.java 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/50979 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If5bcefb621d4bc5b41dd61ac3d9b55e6780f2cc0 Gerrit-PatchSet: 2 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: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0