July 2014 - Engine-commits - oVirt List Archives

Change in ovirt-engine-sdk[sdk_3.5]: sdk: Add contributors to AUTHORS.md
by juan.hernandez＠redhat.com 23 Jul '14

23 Jul '14

Juan Hernandez has submitted this change and it was merged. Change subject: sdk: Add contributors to AUTHORS.md ...................................................................... sdk: Add contributors to AUTHORS.md This patch adds the names of contributors to the AUTHORS.md file. The list has been extracted from the output of git log. Change-Id: Ia8e709d5faa77691c717e977ce974bee50318ef1 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit 2502a8fda6f54af3c63c7506dd2675cd9135bbde) --- M AUTHORS.md 1 file changed, 10 insertions(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/30626 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia8e709d5faa77691c717e977ce974bee50318ef1 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: sdk_3.5 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine-sdk[master]: sdk: Add contributors to AUTHORS.md
by juan.hernandez＠redhat.com 23 Jul '14

23 Jul '14

Juan Hernandez has submitted this change and it was merged. Change subject: sdk: Add contributors to AUTHORS.md ...................................................................... sdk: Add contributors to AUTHORS.md This patch adds the names of contributors to the AUTHORS.md file. The list has been extracted from the output of git log. Change-Id: Ia8e709d5faa77691c717e977ce974bee50318ef1 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M AUTHORS.md 1 file changed, 10 insertions(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/30625 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ia8e709d5faa77691c717e977ce974bee50318ef1 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: aaa: fixing more context issues
by oourfali＠redhat.com 23 Jul '14

23 Jul '14

Oved Ourfali has submitted this change and it was merged. Change subject: aaa: fixing more context issues ...................................................................... aaa: fixing more context issues Change-Id: I793344d07b1a60eac50df81ed3fdca04e5a7851a Topic: AAA Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ActivateVdsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/ApproveVdsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/MigrateVmToServerCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RemoveVmPoolCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RestartVdsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/StartVdsCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsNotRespondingTreatmentCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsPowerDownCommand.java M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/storage/SetStoragePoolStatusCommand.java 9 files changed, 58 insertions(+), 10 deletions(-) Approvals: Alon Bar-Lev: Looks good to me, but someone else must approve Yair Zaslavsky: Verified Oved Ourfali: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/30620 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I793344d07b1a60eac50df81ed3fdca04e5a7851a Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine-sdk[sdk_3.5]: sdk: Fix the names of the documentation files
by juan.hernandez＠redhat.com 23 Jul '14

23 Jul '14

Juan Hernandez has submitted this change and it was merged. Change subject: sdk: Fix the names of the documentation files ...................................................................... sdk: Fix the names of the documentation files The use now markdown, so the names have the .md extension, and this needs to be updated in the RPM spec. Change-Id: Id0eb77f3fc2c8f78d4632d50bf9b05d9648457b0 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M ovirt-engine-sdk-python.spec.in 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/30623 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id0eb77f3fc2c8f78d4632d50bf9b05d9648457b0 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: sdk_3.5 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[ovirt-engine-3.4]: restapi: Ignore blank domains
by oourfali＠redhat.com 23 Jul '14

23 Jul '14

Oved Ourfali has submitted this change and it was merged. Change subject: restapi: Ignore blank domains ...................................................................... restapi: Ignore blank domains Currently the the value of the "domain" column of the "vm_init" table can be null or an empty string, both meaning that the VM doesn't have a domain assigned. But when the value of the column is an empty string it will be included in the RESTAPI representation as follows: <domain> <name></name> </domain> This is confusing for callers, and in particular it triggers wrong behaviour in the Python SDK. To avoid this issue this patch changes the RESTAPI so that it won't generate the "domain" element if the domain name is a blank string. Change-Id: Ic2dc13b1dabc08e38def31297148f135ec299ec9 Bug-Url: https://bugzilla.redhat.com/1113571 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit ac78f2b7687df6f8e0ed40cf783729f9eda8c9c0) --- M backend/manager/modules/restapi/types/pom.xml M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/TemplateMapper.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java 3 files changed, 8 insertions(+), 5 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/30617 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic2dc13b1dabc08e38def31297148f135ec299ec9 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.4 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine-sdk[master]: sdk: Fix the names of the documentation files
by juan.hernandez＠redhat.com 23 Jul '14

23 Jul '14

Juan Hernandez has submitted this change and it was merged. Change subject: sdk: Fix the names of the documentation files ...................................................................... sdk: Fix the names of the documentation files The use now markdown, so the names have the .md extension, and this needs to be updated in the RPM spec. Change-Id: Id0eb77f3fc2c8f78d4632d50bf9b05d9648457b0 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M Makefile M ovirt-engine-sdk-python.spec.in 2 files changed, 2 insertions(+), 2 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/30622 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id0eb77f3fc2c8f78d4632d50bf9b05d9648457b0 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: restapi: move aaa artifact to own subpackage
by juan.hernandez＠redhat.com 23 Jul '14

23 Jul '14

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: move aaa artifact to own subpackage ...................................................................... restapi: move aaa artifact to own subpackage Topic: AAA Change-Id: Idc08ae7bcf8d506569a30c97887bc2534d66cc61 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- R backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/aaa/DomainGroupResource.java R backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/aaa/DomainGroupsResource.java R backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/aaa/DomainResource.java R backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/aaa/DomainUserResource.java R backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/aaa/DomainUsersResource.java R backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/aaa/DomainsResource.java R backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/aaa/GroupResource.java R backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/aaa/GroupsResource.java R backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/aaa/UserResource.java R backend/manager/modules/restapi/interface/definition/src/main/java/org/ovirt/engine/api/resource/aaa/UsersResource.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/BackendApplication.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendAssignedRolesResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendGroupTagsResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendUserTagsResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmSessionsResource.java R backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/aaa/BackendDomainGroupResource.java R backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/aaa/BackendDomainGroupsResource.java R backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/aaa/BackendDomainResource.java R backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/aaa/BackendDomainUserResource.java R backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/aaa/BackendDomainUsersResource.java R backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/aaa/BackendDomainsResource.java R backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/aaa/BackendGroupResource.java R backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/aaa/BackendGroupsResource.java R backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/aaa/BackendUserResource.java R backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/aaa/BackendUsersResource.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendVmSessionsResourceTest.java R backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/aaa/BackendDomainGroupResourceTest.java R backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/aaa/BackendDomainUserResourceTest.java R backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/aaa/BackendGroupResourceTest.java R backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/aaa/BackendGroupsResourceTest.java R backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/aaa/BackendUserResourceTest.java R backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/aaa/BackendUsersResourceTest.java 33 files changed, 111 insertions(+), 65 deletions(-) Approvals: Alon Bar-Lev: Verified Juan Hernandez: Looks good to me, approved Yair Zaslavsky: Looks good to me, but someone else must approve -- To view, visit http://gerrit.ovirt.org/30592 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idc08ae7bcf8d506569a30c97887bc2534d66cc61 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[ovirt-engine-3.5]: restapi: Ignore blank domains
by piotr.kliczewski＠gmail.com 23 Jul '14

23 Jul '14

Piotr Kliczewski has submitted this change and it was merged. Change subject: restapi: Ignore blank domains ...................................................................... restapi: Ignore blank domains Currently the the value of the "domain" column of the "vm_init" table can be null or an empty string, both meaning that the VM doesn't have a domain assigned. But when the value of the column is an empty string it will be included in the RESTAPI representation as follows: <domain> <name></name> </domain> This is confusing for callers, and in particular it triggers wrong behaviour in the Python SDK. To avoid this issue this patch changes the RESTAPI so that it won't generate the "domain" element if the domain name is a blank string. Change-Id: Ic2dc13b1dabc08e38def31297148f135ec299ec9 Bug-Url: https://bugzilla.redhat.com/1113571 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> (cherry picked from commit ac78f2b7687df6f8e0ed40cf783729f9eda8c9c0) --- M backend/manager/modules/restapi/types/pom.xml M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/TemplateMapper.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java 3 files changed, 8 insertions(+), 5 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/30616 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic2dc13b1dabc08e38def31297148f135ec299ec9 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.5 Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-engine[master]: restapi: Ignore blank domains
by juan.hernandez＠redhat.com 23 Jul '14

23 Jul '14

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Ignore blank domains ...................................................................... restapi: Ignore blank domains Currently the the value of the "domain" column of the "vm_init" table can be null or an empty string, both meaning that the VM doesn't have a domain assigned. But when the value of the column is an empty string it will be included in the RESTAPI representation as follows: <domain> <name></name> </domain> This is confusing for callers, and in particular it triggers wrong behaviour in the Python SDK. To avoid this issue this patch changes the RESTAPI so that it won't generate the "domain" element if the domain name is a blank string. Change-Id: Ic2dc13b1dabc08e38def31297148f135ec299ec9 Bug-Url: https://bugzilla.redhat.com/1113571 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/types/pom.xml M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/TemplateMapper.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java 3 files changed, 8 insertions(+), 5 deletions(-) Approvals: Shahar Havivi: Looks good to me, approved Juan Hernandez: Verified -- To view, visit http://gerrit.ovirt.org/30548 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ic2dc13b1dabc08e38def31297148f135ec299ec9 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com> Gerrit-Reviewer: Shahar Havivi <shavivi(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server

1 0

Change in ovirt-reports[master]: build: post ovirt-engine-reports-3.5_beta2
by ydary＠redhat.com 23 Jul '14

23 Jul '14

Yaniv Dary has submitted this change and it was merged. Change subject: build: post ovirt-engine-reports-3.5_beta2 ...................................................................... build: post ovirt-engine-reports-3.5_beta2 Change-Id: I9017e682ed3ce39edaa38360a1ec3b4dd66009f3 Signed-off-by: Shirly Radco <sradco(a)redhat.com> --- M version.mak 1 file changed, 2 insertions(+), 2 deletions(-) Approvals: Shirly Radco: Verified Yaniv Dary: Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/30600 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9017e682ed3ce39edaa38360a1ec3b4dd66009f3 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-reports Gerrit-Branch: master Gerrit-Owner: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Shirly Radco <sradco(a)redhat.com> Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0