- Engine-commits - oVirt List Archives

Change in jenkins[master]: Fix wrong jenkins repo conf in coverity job
by eedri＠redhat.com 18 Jul '16

18 Jul '16

Eyal Edri has submitted this change and it was merged. Change subject: Fix wrong jenkins repo conf in coverity job ...................................................................... Fix wrong jenkins repo conf in coverity job Change-Id: Ib73beca81e30ab5d40f4686c1a8ea984303d5098 Signed-off-by: Sharon Naftaly <snaftaly(a)redhat.com> --- M jobs/confs/yaml/jobs/ovirt-engine_master_coverity-analysis.yaml 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sharon Naftaly: Verified; Ready for review Eyal Edri: Ready for merge Barak Korren: Looks good to me Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60890 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib73beca81e30ab5d40f4686c1a8ea984303d5098 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sharon Naftaly <snaftaly(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sharon Naftaly <snaftaly(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: webadmin: Fix disable of the entire edit disk popup when VM ...
by tnisan＠redhat.com 18 Jul '16

18 Jul '16

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Fix disable of the entire edit disk popup when VM is running ...................................................................... webadmin: Fix disable of the entire edit disk popup when VM is running Change I14dcbfb50a6249de6380346469baa055f7130662 (937704c5) introduced the ReadOnlyDiskModel used for image upload purposes, to decide whether to disable the entire dialog it used the isEditEnabled() methods from the parent model which although the misleading does not mean that the entire panel should be disabled but whether properties that are no editable when the VM is up should be disabled, this cause the edit dialog to be disabled at all times. This fixes it by calling getIsModelDisabled() after the initial load of the data centers Change-Id: I274635fbf97f0af45c1b1170dce887d480424b74 Bug-Url: https://bugzilla.redhat.com/1349498 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/vm/VmDiskPopupWidget.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/ReadOnlyDiskModel.java 2 files changed, 3 insertions(+), 10 deletions(-) Approvals: Tal Nisan: Verified Jenkins CI: Passed CI tests Daniel Erez: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/60891 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I274635fbf97f0af45c1b1170dce887d480424b74 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Tal Nisan <tnisan(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: VmDeviceUtils#addImportedOtherDevices local var
by amureini＠redhat.com 18 Jul '16

18 Jul '16

Allon Mureinik has submitted this change and it was merged. Change subject: core: VmDeviceUtils#addImportedOtherDevices local var ...................................................................... core: VmDeviceUtils#addImportedOtherDevices local var Removed the unused hasSound local variable. Change-Id: I27f98080f6a4d8d48370d2bd4094e511f8b3db9d Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/utils/VmDeviceUtils.java 1 file changed, 0 insertions(+), 5 deletions(-) Approvals: Allon Mureinik: Verified; Passed CI tests Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/60878 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I27f98080f6a4d8d48370d2bd4094e511f8b3db9d 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: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: VmDeviceUtils type inference
by amureini＠redhat.com 18 Jul '16

18 Jul '16

Allon Mureinik has submitted this change and it was merged. Change subject: core: VmDeviceUtils type inference ...................................................................... core: VmDeviceUtils type inference Change-Id: Ie155a036aa1a6c04ede8de56070afa777e5ce744 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/utils/VmDeviceUtils.java 1 file changed, 3 insertions(+), 3 deletions(-) Approvals: Allon Mureinik: Verified; Passed CI tests Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/60877 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie155a036aa1a6c04ede8de56070afa777e5ce744 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: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: VmDeviceUtils - use addAll
by amureini＠redhat.com 18 Jul '16

18 Jul '16

Allon Mureinik has submitted this change and it was merged. Change subject: core: VmDeviceUtils - use addAll ...................................................................... core: VmDeviceUtils - use addAll Use List's addAll method instead of looping over add methods. This reduces some boilerplate code from the class and makes these calls slightly neater. Change-Id: I93c3e2d9a057094dc11b3b7d3e64020d50c45fc9 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/utils/VmDeviceUtils.java 1 file changed, 2 insertions(+), 6 deletions(-) Approvals: Allon Mureinik: Verified; Passed CI tests Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/60880 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I93c3e2d9a057094dc11b3b7d3e64020d50c45fc9 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: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[master]: core: Remove VmDeviceUtils#EMPTY_SPEC_PARAMS
by amureini＠redhat.com 18 Jul '16

18 Jul '16

Allon Mureinik has submitted this change and it was merged. Change subject: core: Remove VmDeviceUtils#EMPTY_SPEC_PARAMS ...................................................................... core: Remove VmDeviceUtils#EMPTY_SPEC_PARAMS The constant conveys no extra information other than being an empty map, so there's no value in keeping it instead of just calling Collections#emptyMap() instead. Change-Id: I67dab16eb66df90f8d1689b65ef8bebd11d9082a Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/utils/VmDeviceUtils.java 1 file changed, 7 insertions(+), 8 deletions(-) Approvals: Allon Mureinik: Verified; Passed CI tests Arik Hadas: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/60879 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I67dab16eb66df90f8d1689b65ef8bebd11d9082a 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: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: core: image upload - propagate validation messages
by tnisan＠redhat.com 18 Jul '16

18 Jul '16

Tal Nisan has submitted this change and it was merged. Change subject: core: image upload - propagate validation messages ...................................................................... core: image upload - propagate validation messages Propagate validation messages in UploadDiskImageCommand. Change-Id: I1d408a4393953a7ba6ba4aa34b30f69686eb754c Bug-Url: https://bugzilla.redhat.com/1352857 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/disk/image/UploadDiskImageCommand.java 1 file changed, 5 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/60884 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1d408a4393953a7ba6ba4aa34b30f69686eb754c Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 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-4.0]: webadmin: disk tab - storage domain selection on upload
by tnisan＠redhat.com 18 Jul '16

18 Jul '16

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: disk tab - storage domain selection on upload ...................................................................... webadmin: disk tab - storage domain selection on upload Disk tab -> upload: Data Center and Storage Domains select-boxes should be changeable. Change-Id: I4a999af0b8c6c9b9a509ec1bfac13371cdcace62 Bug-Url: https://bugzilla.redhat.com/1352928 Bug-Url: https://bugzilla.redhat.com/1352855 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/UploadImageModel.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/60885 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4a999af0b8c6c9b9a509ec1bfac13371cdcace62 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 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-4.0]: webadmin: Disk upload dialog - update image type
by tnisan＠redhat.com 18 Jul '16

18 Jul '16

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Disk upload dialog - update image type ...................................................................... webadmin: Disk upload dialog - update image type Disk Upload dialog: 'Image Type' select-box - format should be selected according to the StorageType and VolumeType. Change-Id: I76c3bc4584cb917d908c9109b89f4d6ba337b47f Bug-Url: https://bugzilla.redhat.com/1353229 Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/UploadImageModel.java 1 file changed, 15 insertions(+), 19 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Daniel Erez: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/60886 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I76c3bc4584cb917d908c9109b89f4d6ba337b47f Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 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]: webadmin: Fix disable of the entire edit disk popup when VM ...
by tnisan＠redhat.com 18 Jul '16

18 Jul '16

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Fix disable of the entire edit disk popup when VM is running ...................................................................... webadmin: Fix disable of the entire edit disk popup when VM is running Change I14dcbfb50a6249de6380346469baa055f7130662 (937704c5) introduced the ReadOnlyDiskModel used for image upload purposes, to decide whether to disable the entire dialog it used the isEditEnabled() methods from the parent model which although the misleading does not mean that the entire panel should be disabled but whether properties that are no editable when the VM is up should be disabled, this cause the edit dialog to be disabled at all times. This fixes it by calling getIsModelDisabled() after the initial load of the data centers Change-Id: I274635fbf97f0af45c1b1170dce887d480424b74 Bug-Url: https://bugzilla.redhat.com/1349498 Signed-off-by: Tal Nisan <tnisan(a)redhat.com> --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/vm/VmDiskPopupWidget.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/vms/ReadOnlyDiskModel.java 2 files changed, 3 insertions(+), 10 deletions(-) Approvals: Tal Nisan: Verified Daniel Erez: Looks good to me, approved Jenkins CI: Passed CI tests Allon Mureinik: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/60861 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I274635fbf97f0af45c1b1170dce887d480424b74 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Tal Nisan <tnisan(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: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0