September 2015 - Engine-commits - oVirt List Archives

Change in ovirt-engine[master]: restapi: Add JAXB annotation to "payload" element
by juan.hernandez＠redhat.com 29 Sep '15

29 Sep '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Add JAXB annotation to "payload" element ...................................................................... restapi: Add JAXB annotation to "payload" element This annotation is used to make sure that the name of the corresponding Java getter is "getPayloads", as this, plural, is what is used in the rest of the API. Change-Id: I9244ed33f4071d65bc53008d590512fc33adaf82 Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/schema/api.xsd M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmsResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/validation/VmValidator.java M backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/VmMapperTest.java 5 files changed, 13 insertions(+), 7 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/46768 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9244ed33f4071d65bc53008d590512fc33adaf82 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 JAXB annotation to "event" element
by juan.hernandez＠redhat.com 29 Sep '15

29 Sep '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Add JAXB annotation to "event" element ...................................................................... restapi: Add JAXB annotation to "event" element This annotation is used to make sure that the name of the corresponding Java getter is "getEvents", as this, plural, is what is used in the rest of the API. Change-Id: If48940182475b62618db16c836513ad724f61e9d Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/schema/api.xsd M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendEventsResource.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendEventsResourceTest.java 3 files changed, 9 insertions(+), 8 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/46766 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: If48940182475b62618db16c836513ad724f61e9d 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[ovirt-engine-3.6]: packaging: pythonlib: service: by default redirect to /dev/n...
by piotr.kliczewski＠gmail.com 29 Sep '15

29 Sep '15

Piotr Kliczewski has submitted this change and it was merged. Change subject: packaging: pythonlib: service: by default redirect to /dev/null stdout/stderr ...................................................................... packaging: pythonlib: service: by default redirect to /dev/null stdout/stderr this avoid handle inheritance into the daemon process, enables detaching properly from the init.d parent. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1266881 Change-Id: I621a86647c2df1784e94cada25f8a907b5a2f3f3 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> (cherry picked from commit 931f8d1011ced54c60bcacd1220d99e524206478) --- M packaging/pythonlib/ovirt_engine/service.py M packaging/services/ovirt-fence-kdump-listener/ovirt-fence-kdump-listener.py 2 files changed, 2 insertions(+), 5 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/46773 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I621a86647c2df1784e94cada25f8a907b5a2f3f3 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: ovirt-engine-3.6 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski(a)gmail.com> Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in jenkins[master]: ovirt-engine_standard: added branch 3.6.0
by sbonazzo＠redhat.com 29 Sep '15

29 Sep '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-engine_standard: added branch 3.6.0 ...................................................................... ovirt-engine_standard: added branch 3.6.0 Change-Id: I7a9dde82c7b21c18b4ac9d1ee090bfae26b69ad5 Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M jobs/confs/projects/ovirt-engine/ovirt-engine_standard.yaml 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me; Ready for merge Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/46774 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I7a9dde82c7b21c18b4ac9d1ee090bfae26b69ad5 Gerrit-PatchSet: 1 Gerrit-Project: jenkins Gerrit-Branch: master Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>

1 0

Change in ovirt-engine[master]: packaging: pythonlib: service: by default redirect to /dev/n...
by sbonazzo＠redhat.com 29 Sep '15

29 Sep '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: packaging: pythonlib: service: by default redirect to /dev/null stdout/stderr ...................................................................... packaging: pythonlib: service: by default redirect to /dev/null stdout/stderr this avoid handle inheritance into the daemon process, enables detaching properly from the init.d parent. Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1266881 Change-Id: I621a86647c2df1784e94cada25f8a907b5a2f3f3 Signed-off-by: Alon Bar-Lev <alonbl(a)redhat.com> --- M packaging/pythonlib/ovirt_engine/service.py M packaging/services/ovirt-fence-kdump-listener/ovirt-fence-kdump-listener.py 2 files changed, 2 insertions(+), 5 deletions(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/46769 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I621a86647c2df1784e94cada25f8a907b5a2f3f3 Gerrit-PatchSet: 2 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: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Simone Tiraboschi <stirabos(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: restapi: Use "logical_units" instead of multiple "logical_unit"
by juan.hernandez＠redhat.com 28 Sep '15

28 Sep '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Use "logical_units" instead of multiple "logical_unit" ...................................................................... restapi: Use "logical_units" instead of multiple "logical_unit" Currently the logical units that are part of a volume group are reported as an unbounded number of "logical_unit" elements. For example, when reporting the details of a storage domain: GET /storagedomains/{storagedomain:id} <storage_domain> ... <storage> ... <volume_group> <logical_unit>  </logical_unit> <logical_unit>  </logical_unit> ... </volume_group> </storage> </storage_domain> This is contrary to the usual practice in the API, as list of elements are always wrapped with an element. This has been fixed now, so the list of logical units will be wrapped with the "logical_units" element: GET /storagedomains/{storagedomain:id} <storage_domain> ... <storage> ... <volume_group> <logical_units> <logical_unit>  </logical_unit> <logical_unit>  </logical_unit> ... </logical_units> </volume_group> </storage> </storage_domain> Change-Id: I08a71b3ca7873cfee6dcd0d7b881ce6bae935581 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/resources/rsdl_metadata.yaml M backend/manager/modules/restapi/interface/definition/src/main/schema/api.xsd M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendDisksResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendHostStorageResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendStorageDomainResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendStorageDomainsResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendVmDisksResource.java M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/utils/DiskResourceUtils.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendHostStorageResourceTest.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendStorageDomainsResourceTest.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendVmDisksResourceTest.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/StorageLogicalUnitMapper.java M backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/StorageLogicalUnitMapperTest.java 14 files changed, 315 insertions(+), 135 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/46764 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I08a71b3ca7873cfee6dcd0d7b881ce6bae935581 Gerrit-PatchSet: 3 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 complex type "SupportedVersions" to "Versions"
by juan.hernandez＠redhat.com 28 Sep '15

28 Sep '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Rename complex type "SupportedVersions" to "Versions" ...................................................................... restapi: Rename complex type "SupportedVersions" to "Versions" The "supported" part of the name is an indication of what it is used for, not of what is its structure, and "Versions" is more consistent with the existig "Version" complex type. Change-Id: I3b7943dd4e041ccf14c15df9a9b5644d036ad11e 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/schema/api.xsd M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendDataCentersResource.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VersionMapper.java 4 files changed, 10 insertions(+), 9 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/46762 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I3b7943dd4e041ccf14c15df9a9b5644d036ad11e 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-live[ovirt-live-3.6]: ovirt-live-3.6.0-rc1
by sbonazzo＠redhat.com 28 Sep '15

28 Sep '15

Sandro Bonazzola has submitted this change and it was merged. Change subject: ovirt-live-3.6.0-rc1 ...................................................................... ovirt-live-3.6.0-rc1 Change-Id: Ifb19786190659e266c2ef1c30d4e58e507b0654d Signed-off-by: Sandro Bonazzola <sbonazzo(a)redhat.com> --- M centos-7/Makefile 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Sandro Bonazzola: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/46763 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifb19786190659e266c2ef1c30d4e58e507b0654d Gerrit-PatchSet: 1 Gerrit-Project: ovirt-live Gerrit-Branch: ovirt-live-3.6 Gerrit-Owner: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com> Gerrit-Reviewer: automation(a)ovirt.org

1 0

Change in ovirt-engine[master]: restapi: Add JAXB annotation to "custom_property" element
by juan.hernandez＠redhat.com 28 Sep '15

28 Sep '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Add JAXB annotation to "custom_property" element ...................................................................... restapi: Add JAXB annotation to "custom_property" element This annotation is used to make sure that the name of the corresponding Java getter is "getCustomProperties", as this, plural, is what is used in the rest of the API. Change-Id: I6779bec2b34466b3a5d1e9e3372d348d1dca343e Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com> --- M backend/manager/modules/restapi/interface/definition/src/main/schema/api.xsd M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendCapabilitiesResource.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendCapabilitiesResourceTest.java M backend/manager/modules/restapi/jaxrs/src/test/java/org/ovirt/engine/api/restapi/resource/BackendCapabilityResourceTest.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmBaseMapper.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/VnicProfileMapper.java M backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/VnicProfileMapperTest.java 8 files changed, 19 insertions(+), 13 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests -- To view, visit https://gerrit.ovirt.org/46760 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6779bec2b34466b3a5d1e9e3372d348d1dca343e Gerrit-PatchSet: 3 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 complex type "SPM" to "Spm"
by juan.hernandez＠redhat.com 28 Sep '15

28 Sep '15

Juan Hernandez has submitted this change and it was merged. Change subject: restapi: Rename complex type "SPM" to "Spm" ...................................................................... restapi: Rename complex type "SPM" to "Spm" This is required to make sure that there is only one rule for the names of complex types: all words, including acronyms, start with one upper case character and all the rest are lower case. Change-Id: I40cbc1803d80890ad7957373851820db823d0d47 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/schema/api.xsd M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/HostMapper.java 3 files changed, 7 insertions(+), 6 deletions(-) Approvals: Juan Hernandez: Verified; Looks good to me, approved Jenkins CI: Passed CI tests -- To view, visit https://gerrit.ovirt.org/46756 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I40cbc1803d80890ad7957373851820db823d0d47 Gerrit-PatchSet: 1 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