June 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: restapi: Move Host remove from collection to entity
by juan.hernandez＠redhat.com 02 Jun '15

02 Jun '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Move Host remove from collection to entity ...................................................................... restapi: Move Host remove from collection to entity This patch moves the method that implements the DELETE operation from the collection interface to the entity interface. This is needed to avoid issues with newer versions of Resteasy. Change-Id: I0a2fed67c0edc8c73e96543dc1d8008523d46abe Related: https://gerrit.ovirt.org/41783 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/HostResource.java M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/HostsResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendHostResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendHostsResource.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendHostResourceTest.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendHostsResourceTest.java 6 files changed, 133 insertions(+), 115 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/41823 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I0a2fed67c0edc8c73e96543dc1d8008523d46abe 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: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: Adding system tests
by tlitovsk＠redhat.com 02 Jun '15

02 Jun '15

Anatoly Litovsky has submitted this change and it was merged. Change subject: Adding system tests ...................................................................... Adding system tests Using the new CI test framework Signed-off-by: David Caro <dcaroest(a)redhat.com> Signed-off-by: Dima Kuznetsov <dkuznets(a)redhat.com> Signed-off-by: Tolik Litovsky <tlitovsk(a)redhat.com> Change-Id: If411d80e4213f5d4a5d85defeb6bb1634f9bcae4 --- A jobs/confs/shell-scripts/system_tests.cleanup.sh A jobs/confs/shell-scripts/system_tests.collect_logs.sh A jobs/confs/shell-scripts/system_tests_common.sh A jobs/confs/yaml/builders/system_tests.yaml A jobs/confs/yaml/jobs/ovirt/system-tests.yaml A jobs/confs/yaml/scms/ovirt-engine.yaml M jobs/confs/yaml/scms/vdsm.yaml A jobs/confs/yaml/templates/system_tests.yaml M jobs/confs/yaml/triggers/gerrit.yaml 9 files changed, 234 insertions(+), 0 deletions(-) Approvals: Jenkins CI: Verified; Looks good to me; Passed CI tests Anatoly Litovsky: Ready for merge -- To view, visit https://gerrit.ovirt.org/37069 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If411d80e4213f5d4a5d85defeb6bb1634f9bcae4 Gerrit-PatchSet: 24 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Anatoly Litovsky <tlitovsk(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Dima Kuznetsov <dkuznets(a)redhat.com> Gerrit-Reviewer: Jenkins CI

1 0

Change in ovirt-node-plugin-vdsm[master]: Add a note for a to-be-fixed hack
by fabiand＠redhat.com 02 Jun '15

02 Jun '15

Fabian Deutsch has submitted this change and it was merged. Change subject: Add a note for a to-be-fixed hack ...................................................................... Add a note for a to-be-fixed hack Change-Id: I3527a7d1d963a2076f04edbe118ad1695a486cff Signed-off-by: Fabian Deutsch <fabiand(a)fedoraproject.org> Reviewed-on: https://gerrit.ovirt.org/40649 Reviewed-by: Fabian Deutsch <fabiand(a)redhat.com> Tested-by: Fabian Deutsch <fabiand(a)redhat.com> --- M src/engine_page.py 1 file changed, 28 insertions(+), 13 deletions(-) Approvals: Fabian Deutsch: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/40649 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3527a7d1d963a2076f04edbe118ad1695a486cff Gerrit-PatchSet: 3 Gerrit-Project: ovirt-node-plugin-vdsm Gerrit-Branch: master Gerrit-Owner: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Fabian Deutsch <fabiand(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: core, restapi: Support adding permission using role name
by juan.hernandez＠redhat.com 02 Jun '15

02 Jun '15

Juan Hernandez has submitted this change and it was merged. Change subject: core, restapi: Support adding permission using role name ...................................................................... core, restapi: Support adding permission using role name Currently the operations to add permissions require the identifier of the role: POST /vms/{vm:id}/permissions <permission> <role id="00000000-0000-0000-0000-000000000001" /> <group id="..."/> </permission> This patch adds the capability to use role names as well: POST /vms/{vm:id}/permissions <permission> <role> <name>UserVmManager</name> </role> <group id="..."/> </permission> Change-Id: I74f22e9bebbc102a45b76e9c62cc44961edadfbc Bug-Url: https://bugzilla.redhat.com/1152989 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/AddPermissionCommand.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/BackendAssignedPermissionsResource.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/PermissionMapper.java 4 files changed, 157 insertions(+), 94 deletions(-) Approvals: Ori Liel: Looks good to me, approved Juan Hernandez: Verified Jenkins CI: Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/40694 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I74f22e9bebbc102a45b76e9c62cc44961edadfbc Gerrit-PatchSet: 4 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: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: restapi: Move Disk Profile remove from collection to entity
by juan.hernandez＠redhat.com 02 Jun '15

02 Jun '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Move Disk Profile remove from collection to entity ...................................................................... restapi: Move Disk Profile remove from collection to entity This patch moves the method that implements the DELETE operation from the collection interface to the entity interface. This is needed to avoid issues with newer versions of Resteasy. Change-Id: Ib304bf8c3d7d4f3d5c948aadc11b00b37c8aa78a Related: https://gerrit.ovirt.org/41783 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/AssignedDiskProfileResource.java M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/AssignedDiskProfilesResource.java M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/DiskProfileResource.java M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/DiskProfilesResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/AbstractBackendDiskProfileResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/AbstractBackendDiskProfilesResource.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/AbstractBackendDiskProfilesResourceTest.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendAssignedDiskProfileResourceTest.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendDiskProfileResourceTest.java 9 files changed, 199 insertions(+), 103 deletions(-) Approvals: Ori Liel: Looks good to me, approved Juan Hernandez: Verified Jenkins CI: Verified; Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/41803 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ib304bf8c3d7d4f3d5c948aadc11b00b37c8aa78a 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: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: restapi: Move CPU Profile remove from collection to entity
by juan.hernandez＠redhat.com 02 Jun '15

02 Jun '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Move CPU Profile remove from collection to entity ...................................................................... restapi: Move CPU Profile remove from collection to entity This patch moves the method that implements the DELETE operation from the collection interface to the entity interface. This is needed to avoid issues with newer versions of Resteasy. Change-Id: I7e230efbc2cd878d192080c268bc6a7097a76804 Related: https://gerrit.ovirt.org/41783 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/AssignedCpuProfileResource.java M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/AssignedCpuProfilesResource.java M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/CpuProfileResource.java M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/CpuProfilesResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/AbstractBackendCpuProfileResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/AbstractBackendCpuProfilesResource.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/AbstractBackendCpuProfilesResourceTest.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendAssignedCpuProfileResourceTest.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendCpuProfileResourceTest.java 9 files changed, 201 insertions(+), 104 deletions(-) Approvals: Ori Liel: Looks good to me, approved Juan Hernandez: Verified Jenkins CI: Verified; Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/41802 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7e230efbc2cd878d192080c268bc6a7097a76804 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: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: restapi: Move Affinity Group VM remove from collection to en...
by juan.hernandez＠redhat.com 02 Jun '15

02 Jun '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Move Affinity Group VM remove from collection to entity ...................................................................... restapi: Move Affinity Group VM remove from collection to entity This patch moves the method that implements the DELETE operation from the collection interface to the entity interface. This is needed to avoid issues with newer versions of Resteasy. Change-Id: Id474459945434e0b44ab17ab389e1c86ee731e9f Related: https://gerrit.ovirt.org/41783 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- A backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/AffinityGroupVmResource.java M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/AffinityGroupVmsResource.java A backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendAffinityGroupVmResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendAffinityGroupVmsResource.java A backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendAffinityGroupVmResourceTest.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendAffinityGroupVmsResourceTest.java 6 files changed, 140 insertions(+), 36 deletions(-) Approvals: Ori Liel: Looks good to me, approved Jenkins CI: Verified; Looks good to me, but someone else must approve -- To view, visit https://gerrit.ovirt.org/41801 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id474459945434e0b44ab17ab389e1c86ee731e9f 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: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: restapi: Move Affinity Group remove from collection to entity
by juan.hernandez＠redhat.com 02 Jun '15

02 Jun '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Move Affinity Group remove from collection to entity ...................................................................... restapi: Move Affinity Group remove from collection to entity This patch moves the method that implements the DELETE operation from the collection interface to the entity interface. This is needed to avoid issues with newer versions of Resteasy. Change-Id: I5eae61aa07eac428dd4de1fae7be28e656c32b5b Related: https://gerrit.ovirt.org/41783 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/AffinityGroupResource.java M backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/AffinityGroupsResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendAffinityGroupResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendAffinityGroupsResource.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendAffinityGroupResourceTest.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendAffinityGroupsResourceTest.java 6 files changed, 44 insertions(+), 48 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit https://gerrit.ovirt.org/41790 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I5eae61aa07eac428dd4de1fae7be28e656c32b5b Gerrit-PatchSet: 4 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: Ori Liel <oliel(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: Added git review file
by dcaroest＠redhat.com 02 Jun '15

02 Jun '15

David Caro has submitted this change and it was merged. Change subject: Added git review file ...................................................................... Added git review file Change-Id: Ica08fc260bc5fad34a83cbabb10fedab3ce8d99b Signed-off-by: David Caro <dcaroest(a)redhat.com> --- A .gitreview 1 file changed, 5 insertions(+), 0 deletions(-) Approvals: David Caro: Verified; Looks good to me; Ready for merge; Passed CI tests -- To view, visit https://gerrit.ovirt.org/40702 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ica08fc260bc5fad34a83cbabb10fedab3ce8d99b Gerrit-PatchSet: 2 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Barak Korren <bkorren(a)redhat.com> Gerrit-Reviewer: David Caro <dcaroest(a)redhat.com> Gerrit-Reviewer: Eyal Edri <eedri(a)redhat.com> Gerrit-Reviewer: Max Kovgan <mvk(a)redhat.com> Gerrit-Reviewer: Paz Dangur <pdangur(a)redhat.com>

1 0

Change in ovirt-engine[master]: core: Add JPA UserTypes
by masayag＠redhat.com 02 Jun '15

02 Jun '15

Moti Asayag has submitted this change and it was merged. Change subject: core: Add JPA UserTypes ...................................................................... core: Add JPA UserTypes The VDS patch requires some UserTypes. To simplify things, those UserTypes are added as a separate patch Change-Id: I30f76493ec72e3c2540e43bef2c095a7ed1c38aa Bug-Url: https://bugzilla.redhat.com/1159621 Signed-off-by: lzelkha(a)redhat.com <lzelkha(a)redhat.com> --- A backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/jpa/HbaUserType.java A backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/jpa/RpmVersionUserType.java A backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dao/jpa/VmRngDeviceSourceUserType.java 3 files changed, 251 insertions(+), 0 deletions(-) Approvals: Martin Peřina: Looks good to me, but someone else must approve Jenkins CI: Verified; Looks good to me, but someone else must approve Moti Asayag: Looks good to me, approved Liran Zelkha: Verified -- To view, visit https://gerrit.ovirt.org/40515 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I30f76493ec72e3c2540e43bef2c095a7ed1c38aa Gerrit-PatchSet: 15 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Liran Zelkha <lzelkha(a)redhat.com> Gerrit-Reviewer: Martin Peřina <mperina(a)redhat.com> Gerrit-Reviewer: Moti Asayag <masayag(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yevgeny Zaspitsky <yzaspits(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0