- Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: webadmin: resize upload image dialog
by derez＠redhat.com 27 Jul '16

27 Jul '16

Daniel Erez has submitted this change and it was merged. Change subject: webadmin: resize upload image dialog ...................................................................... webadmin: resize upload image dialog Resized 'Upload Image' dialog to accommodate its widgets more properly. Before: http://imgur.com/a/Pyecu After: http://imgur.com/a/KX8ht Change-Id: I516d982cf39c94bf1bd4448ef34a32c1dd9f7fcc 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/UploadImagePopupView.ui.xml 1 file changed, 4 insertions(+), 3 deletions(-) Approvals: Jenkins CI: Passed CI tests Daniel Erez: Verified Allon Mureinik: Looks good to me, approved Amit Aviram: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/61332 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I516d982cf39c94bf1bd4448ef34a32c1dd9f7fcc Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(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

Change in ovirt-engine[master]: webadmin: rename getTemplateVolumeType method
by derez＠redhat.com 27 Jul '16

27 Jul '16

Daniel Erez has submitted this change and it was merged. Change subject: webadmin: rename getTemplateVolumeType method ...................................................................... webadmin: rename getTemplateVolumeType method Renamed getTemplateVolumeType to a clearer one by its functionality: getVolumeType. Change-Id: I13912345e1c4180b5197e0c486315088e59115dc Signed-off-by: Daniel Erez <derez(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/dataprovider/AsyncDataProvider.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/DisksAllocationModel.java M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/storage/UploadImageModel.java 3 files changed, 3 insertions(+), 3 deletions(-) Approvals: Jenkins CI: Passed CI tests Daniel Erez: Verified Allon Mureinik: Looks good to me, approved Amit Aviram: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/61323 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I13912345e1c4180b5197e0c486315088e59115dc Gerrit-PatchSet: 4 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(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

Change in ovirt-engine[master]: webadmin: image upload - align selection button
by derez＠redhat.com 27 Jul '16

27 Jul '16

Daniel Erez has submitted this change and it was merged. Change subject: webadmin: image upload - align selection button ...................................................................... webadmin: image upload - align selection button Image Upload dialog: align 'Choose File' button properly. Screenshot: http://imgur.com/a/7jRwK Change-Id: I6a6ccf5c877f9c6a587523a523cd751c00633df3 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/UploadImagePopupView.ui.xml 1 file changed, 2 insertions(+), 0 deletions(-) Approvals: Jenkins CI: Passed CI tests Daniel Erez: Verified Allon Mureinik: Looks good to me, approved Amit Aviram: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/61416 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6a6ccf5c877f9c6a587523a523cd751c00633df3 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Daniel Erez <derez(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

Change in ovirt-engine[master]: webadmin: Fix of FE error while opening HTML5 console
by tjelinek＠redhat.com 27 Jul '16

27 Jul '16

Tomas Jelinek has submitted this change and it was merged. Change subject: webadmin: Fix of FE error while opening HTML5 console ...................................................................... webadmin: Fix of FE error while opening HTML5 console Frontend error was thrown when HTML5 console was opened and the browser blocked pop-up windows. User is now notified using a dialog popup that opening console failed and the reason might be the browser blocking opening new windows.. Change-Id: I8e28390f34b8a921ead66a715064f087d9252b30 Signed-off-by: Jakub Niedermertl <jniederm(a)redhat.com> Bug-Url: https://bugzilla.redhat.com/1327869 --- M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/uicommon/WebClientConsoleInvoker.java M frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java M frontend/webadmin/modules/uicompat/src/main/resources/org/ovirt/engine/ui/uicompat/UIConstants.properties 3 files changed, 33 insertions(+), 15 deletions(-) Approvals: Tomas Jelinek: Looks good to me, approved Jakub Niedermertl: Verified Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61351 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8e28390f34b8a921ead66a715064f087d9252b30 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jakub Niedermertl <jniederm(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Betak <mbetak(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]: restapi: Improve affinity label resources
by juan.hernandez＠redhat.com 27 Jul '16

27 Jul '16

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Improve affinity label resources ...................................................................... restapi: Improve affinity label resources Currently the resources that manage the virtual machines and hosts that are part of an affinity label are implemented using abstract generic base classes. This doesn't work correctly with some versions of Resteasy, as generic methods aren't recognized as implementations of the methods annotated with @GET. In addition the methods to add virtual machines and hosts aren't implemented correctly, as they return "action" instead of the expected "vm" or "host" elements. To address these issues this patch removes the abstract generic base classes, and moves the reusable code to a new helper class. The methods that add virtual machines and hosts are modified so that they return "vm" or "host", as expected. Change-Id: I274750970f283bb9ce813f301a28a5cbc544eb60 Bug-Url: https://bugzilla.redhat.com/1350228 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- D backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/AbstractBackendAffinityLabelledEntitiesResource.java D backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/AbstractBackendAffinityLabelledEntityResource.java A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendAffinityLabelHelper.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendAffinityLabelHostResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendAffinityLabelHostsResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendAffinityLabelVmResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendAffinityLabelVmsResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendAffinityLabelsResource.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/AffinityLabelMapper.java 9 files changed, 248 insertions(+), 199 deletions(-) Approvals: Martin Sivák: Looks good to me, but someone else must approve Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/61388 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I274750970f283bb9ce813f301a28a5cbc544eb60 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Martin Sivák <msivak(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0.2]: webadmin: Hide hosted engine tab when editing host
by piotr.kliczewski＠gmail.com 27 Jul '16

27 Jul '16

Piotr Kliczewski has submitted this change and it was merged. Change subject: webadmin: Hide hosted engine tab when editing host ...................................................................... webadmin: Hide hosted engine tab when editing host The hosted engine tab in host editing or adding dialog in the ui is only relevant when adding a new host. This patch hides the tab when a host is edited. To achieve this the initialization of the popup window was moved after the host model initialization. This allows setting the visibility for hostedEngineTab in HostPopupView by using the getIsNew method (was already implemented but didn't work since the relevant fields weren't initialized). Change-Id: I4746dda9fa0a06f0dec1d179639a20e5941e8ec9 Bug-Url: https://bugzilla.redhat.com/1350423 Signed-off-by: Jenny Tokar <jtokar(a)redhat.com> (cherry picked from commit bb20845) --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostListModel.java 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Jenkins CI: Passed CI tests Daniel Erez: Looks good to me, approved Jenny Tokar: Verified -- To view, visit https://gerrit.ovirt.org/61478 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4746dda9fa0a06f0dec1d179639a20e5941e8ec9 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0.2 Gerrit-Owner: Jenny Tokar <jtokar(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Jenny Tokar <jtokar(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Roy Golan <rgolan(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[ovirt-engine-4.0]: core: protection from NPE in the initialization of RunVm
by piotr.kliczewski＠gmail.com 27 Jul '16

27 Jul '16

Piotr Kliczewski has submitted this change and it was merged. Change subject: core: protection from NPE in the initialization of RunVm ...................................................................... core: protection from NPE in the initialization of RunVm Need to verify that the VM to run exists before accessing it. Change-Id: I2e3e7ff30777077cb1e96581574c484d7f71d65b Bug-Url: https://bugzilla.redhat.com/1360265 Signed-off-by: Arik Hadas <ahadas(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommand.java 1 file changed, 5 insertions(+), 6 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Jenkins CI: Passed CI tests Arik Hadas: Verified -- To view, visit https://gerrit.ovirt.org/61447 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2e3e7ff30777077cb1e96581574c484d7f71d65b Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0

Change in ovirt-engine[ovirt-engine-4.0]: webadmin: Hide hosted engine tab when editing host
by piotr.kliczewski＠gmail.com 27 Jul '16

27 Jul '16

Piotr Kliczewski has submitted this change and it was merged. Change subject: webadmin: Hide hosted engine tab when editing host ...................................................................... webadmin: Hide hosted engine tab when editing host The hosted engine tab in host editing or adding dialog in the ui is only relevant when adding a new host. This patch hides the tab when a host is edited. To achieve this the initialization of the popup window was moved after the host model initialization. This allows setting the visibility for hostedEngineTab in HostPopupView by using the getIsNew method (was already implemented but didn't work since the relevant fields weren't initialized). Change-Id: I4746dda9fa0a06f0dec1d179639a20e5941e8ec9 Bug-Url: https://bugzilla.redhat.com/1350423 Signed-off-by: Jenny Tokar <jtokar(a)redhat.com> (cherry picked from commit bb20845) --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostListModel.java 1 file changed, 2 insertions(+), 1 deletion(-) Approvals: Jenkins CI: Passed CI tests Daniel Erez: Looks good to me, approved Jenny Tokar: Verified -- To view, visit https://gerrit.ovirt.org/61479 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I4746dda9fa0a06f0dec1d179639a20e5941e8ec9 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Jenny Tokar <jtokar(a)redhat.com> Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Jenny Tokar <jtokar(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Roy Golan <rgolan(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[ovirt-engine-4.0]: engine: use cluster.dcId if params.dcId isn't set
by piotr.kliczewski＠gmail.com 27 Jul '16

27 Jul '16

Piotr Kliczewski has submitted this change and it was merged. Change subject: engine: use cluster.dcId if params.dcId isn't set ...................................................................... engine: use cluster.dcId if params.dcId isn't set Retrive dcId through VM's cluster if the first isn't supplied in parameters of ImportVmCommand. And make sure that it isn't overriden along th import flow. Change-Id: I055bf8767650006a9baeca22a504c9aaa8bb050d Bug-Url: https://bugzilla.redhat.com/1359668 Signed-off-by: Yevgeny Zaspitsky <yzaspits(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/exportimport/ImportVmCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/exportimport/ImportVmCommandBase.java 2 files changed, 29 insertions(+), 10 deletions(-) Approvals: Jenkins CI: Passed CI tests Arik Hadas: Looks good to me, approved Yevgeny Zaspitsky: Verified -- To view, visit https://gerrit.ovirt.org/61325 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I055bf8767650006a9baeca22a504c9aaa8bb050d Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-4.0 Gerrit-Owner: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Mucha <mmucha(a)redhat.com> Gerrit-Reviewer: Michal Skrivanek <michal.skrivanek(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com>

1 0

Change in ovirt-engine[master]: webadmin: Make sure used is always <= total
by awels＠redhat.com 27 Jul '16

27 Jul '16

Alexander Wels has submitted this change and it was merged. Change subject: webadmin: Make sure used is always <= total ...................................................................... webadmin: Make sure used is always <= total - In certain situations it is possible that the DWH is reporting used > 0 while total = 0. This patch makes sure that used can never be higher than total. Change-Id: I792c3bebee465bc2a0d80aae7a12a76e7c34ad88 Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1360403 Signed-off-by: Alexander Wels <awels(a)redhat.com> --- M frontend/webadmin/modules/frontend/src/main/java/org/ovirt/engine/ui/frontend/server/dashboard/GlobalUtilizationResourceSummary.java 1 file changed, 5 insertions(+), 1 deletion(-) Approvals: Alexander Wels: Verified Jenkins CI: Passed CI tests Oved Ourfali: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/61419 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I792c3bebee465bc2a0d80aae7a12a76e7c34ad88 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>

1 0