- Engine-commits - oVirt List Archives

Change in ovirt-engine[ovirt-engine-3.6]: userportal: Enable contextual alerts in UserPortal
by tnisan＠redhat.com 01 Nov '15

01 Nov '15

Tal Nisan has submitted this change and it was merged. Change subject: userportal: Enable contextual alerts in UserPortal ...................................................................... userportal: Enable contextual alerts in UserPortal Following patch [1], this enables global (contextual) alert messages in UserPortal web application, including the common 'Uncaught exception' alert message triggered by global uncaught exception handler. [1] https://gerrit.ovirt.org/#/c/47074/ Change-Id: If1b3f923e32c1ccefc56ce6b80f48f1e193cb797 Bug-Url: https://bugzilla.redhat.com/1275760 Signed-off-by: Vojtech Szocs <vszocs(a)redhat.com> --- M frontend/webadmin/modules/userportal-gwtp/src/main/java/org/ovirt/engine/ui/userportal/section/main/presenter/MainSectionPresenter.java 1 file changed, 16 insertions(+), 1 deletion(-) Approvals: Alexander Wels: Looks good to me, approved Jenkins CI: Passed CI tests Vojtech Szocs: Verified -- To view, visit https://gerrit.ovirt.org/47907 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If1b3f923e32c1ccefc56ce6b80f48f1e193cb797 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com> Gerrit-Reviewer: Einav Cohen <ecohen(a)redhat.com> Gerrit-Reviewer: Greg Sheremeta <gshereme(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: Vojtech Szocs <vszocs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.6]: webadmin: Enable force remove for host
by tnisan＠redhat.com 01 Nov '15

01 Nov '15

Tal Nisan has submitted this change and it was merged. Change subject: webadmin: Enable force remove for host ...................................................................... webadmin: Enable force remove for host Enable force remove of host if gluster service is enabled, even when virt service is also enabled Change-Id: I14b5aa4ce6349d2d3dbddf4a6e8c068c4c8301ce Bug-Url: https://bugzilla.redhat.com/1244935 Signed-off-by: Sahina Bose <sabose(a)redhat.com> --- M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/hosts/HostListModel.java 1 file changed, 4 insertions(+), 3 deletions(-) Approvals: Jenkins CI: Passed CI tests Sahina Bose: Verified Moti Asayag: Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/47890 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I14b5aa4ce6349d2d3dbddf4a6e8c068c4c8301ce Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Omer Frenkel <d.mosquito(a)gmail.com> Gerrit-Reviewer: Sahina Bose <sabose(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core: StorageDomainToPoolRelationValidator lambda
by amureini＠redhat.com 31 Oct '15

31 Oct '15

Allon Mureinik has submitted this change and it was merged. Change subject: core: StorageDomainToPoolRelationValidator lambda ...................................................................... core: StorageDomainToPoolRelationValidator lambda Use a lambda expression to filter the domains in StorageDomainToPoolRelationValidator instead of a cumbersome LinqUtils expression. Note that this patch also provides a slight performance improvement, as LinqUtils.filter would iterate over the entire list, while anyMatch stops as soon as it encounters such a match. Change-Id: If1305d000e2af4917f3e37b9ce404f5812c59d43 Signed-off-by: Allon Mureinik <amureini(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/validator/storage/StorageDomainToPoolRelationValidator.java 1 file changed, 6 insertions(+), 16 deletions(-) Approvals: Tal Nisan: Looks good to me, approved Jenkins CI: Passed CI tests Allon Mureinik: Verified Ala Hino: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/47900 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If1305d000e2af4917f3e37b9ce404f5812c59d43 Gerrit-PatchSet: 2 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: Jenkins CI Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: restapi: Use matrix parameters for force remove storage domain
by juan.hernandez＠redhat.com 31 Oct '15

31 Oct '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Use matrix parameters for force remove storage domain ...................................................................... restapi: Use matrix parameters for force remove storage domain The operation that removes a storage domain supports the "force", "destroy" and "host" parameters. These parameters were passed to the "DELETE" method using the representation of the storage domain as the body: DELETE /storagedomains/{storagedomain:id} <storage_domain> <force>...</force> <destroy>...</destroy> <host id="..."> <name>...</name> </host> </storage_domain> This is problematic, as the HTTP "DELETE" method shouldn't have a body, and the representation of the storage domain shouldn't include things that aren't attributes of the storage domain, rather parameters of the operation. This patch replaces the "force", "delete" and "host" attributes by equivalent matrix parameters, and the operation doesn't now accept a body. For example, after this patch, the way to delete a storage domain with the "force" parameter is the following: DELETE /storagedomain/{storagedomain:id};host=myhost;force=true To delete with the "destroy" parameter: DELETE /storagedomain/{storagedomain:id};host=myhost;destroy=true Change-Id: I87d4a9a29fcd9af794ecbedb1b255e676541ee13 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/README.md M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/StorageDomainResource.java M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendStorageDomainResource.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendStorageDomainResourceTest.java M backend/manager/modules/restapi/model/src/main/java/types/StorageDomain.java 6 files changed, 171 insertions(+), 113 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/47887 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I87d4a9a29fcd9af794ecbedb1b255e676541ee13 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: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: restapi: Remove "UpdatableRoleResource"
by juan.hernandez＠redhat.com 31 Oct '15

31 Oct '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Remove "UpdatableRoleResource" ...................................................................... restapi: Remove "UpdatableRoleResource" This patch merges the "UpdatableRoleResource" and "RoleResource" iterfaces, putting all the methods in "RoleResource" and removing "UpdatableRoleResource". Change-Id: I72507e4513361108547aa07ed0a94a8bde683412 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/RoleResource.java M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/RolesResource.java D backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/UpdatableRoleResource.java M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/utils/LinkHelper.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendRoleResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendRolesResource.java 6 files changed, 14 insertions(+), 39 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/47883 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I72507e4513361108547aa07ed0a94a8bde683412 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: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: restapi: Remove generic quota limit interfaces
by juan.hernandez＠redhat.com 30 Oct '15

30 Oct '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Remove generic quota limit interfaces ...................................................................... restapi: Remove generic quota limit interfaces This kind of interfaces is an obstacle for the introduction of the new service model. Change-Id: Ie8f1cfe16033f71355ac767d103a4ee6b798d01e Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/QuotaClusterLimitResource.java M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/QuotaClusterLimitsResource.java D backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/QuotaLimitResource.java D backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/QuotaLimitsResource.java M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/QuotaStorageLimitResource.java M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/QuotaStorageLimitsResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendQuotaClusterLimitsResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendQuotaLimitResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendQuotaLimitsResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendQuotaStorageLimitsResource.java 10 files changed, 58 insertions(+), 54 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/47879 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie8f1cfe16033f71355ac767d103a4ee6b798d01e 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: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: restapi: Rename "QoSsResource" to "QossResource"
by juan.hernandez＠redhat.com 30 Oct '15

30 Oct '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Rename "QoSsResource" to "QossResource" ...................................................................... restapi: Rename "QoSsResource" to "QossResource" This is in order to stick to the capitalization rule used in the rest of the RESTAPI. Change-Id: I41c3ef58e6082c7ca5617cc2648fa07e8fbca853 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/DataCenterResource.java R backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/QossResource.java M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/utils/LinkHelper.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendDataCenterResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendQossResource.java 5 files changed, 9 insertions(+), 9 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/47878 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I41c3ef58e6082c7ca5617cc2648fa07e8fbca853 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: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: restapi: Use matrix parameter instead of action to force rem...
by juan.hernandez＠redhat.com 30 Oct '15

30 Oct '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Use matrix parameter instead of action to force remove host ...................................................................... restapi: Use matrix parameter instead of action to force remove host The operation that removes a host supports a "force" parameter. In order to use it the "DELETE" operation used to support an optional action parameter: DELETE /hosts/{host:id} <action> <force>true</force> </action> This patch removes the optional action parameter and replaces it with an optional matrix parameter: DELETE /hosts/{host:id};force=true Change-Id: I758d7d9a45b8c3f8bfedb212da0bd90e6b1f01cf Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/README.md M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/HostResource.java M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendHostResource.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendHostResourceTest.java 5 files changed, 51 insertions(+), 47 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/47877 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I758d7d9a45b8c3f8bfedb212da0bd90e6b1f01cf 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: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: restapi: Add missing action sub-resource
by juan.hernandez＠redhat.com 30 Oct '15

30 Oct '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Add missing action sub-resource ...................................................................... restapi: Add missing action sub-resource Some of the resources that have actions are missing the sub-resource that returns information about the asynchronous actions generated by the execution of the action. This patch adds those missing sub-resources. Change-Id: If598290a477a50c3e5b508549c6f5fc9daa406c6 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/EventsResource.java M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/HostNicsResource.java M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/VmGraphicsConsoleResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendEventsResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendHostNicsResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmGraphicsConsoleResource.java 6 files changed, 20 insertions(+), 5 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/47875 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If598290a477a50c3e5b508549c6f5fc9daa406c6 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: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: restapi: Use "export" instead of "doExport"
by juan.hernandez＠redhat.com 30 Oct '15

30 Oct '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Use "export" instead of "doExport" ...................................................................... restapi: Use "export" instead of "doExport" The general rule for the names of the JAX-RS interface methods is that they are named like the action. For example, if the name of the action is "start" then the name of the method is also "start". The only exception is when the result is a Java reserved word, like "import", and in those cases the name is prefixed with "do", resulting in "doImport", for example. But "export" isn't a reserved word, so in order to improve consistency this patch renames the "doExport" method to just "export". Change-Id: I85ecb4c29ff373f9132ef70469170a54d0cadb5c Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/DiskResource.java M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/TemplateDiskResource.java M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/VmDiskResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendDiskResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendTemplateDiskResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmDiskResource.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendDiskResourceTest.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendStorageDomainDiskResourceTest.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendTemplateDiskResourceTest.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendVmDiskResourceTest.java 10 files changed, 18 insertions(+), 18 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests Objections: Jenkins CI: Failed CI tests -- To view, visit https://gerrit.ovirt.org/47874 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I85ecb4c29ff373f9132ef70469170a54d0cadb5c 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: automation(a)ovirt.org

1 0