Change in ovirt-engine[master]: restapi: Use multiple "vcpu_pins.vcpu_pin" instead of multip...
Juan Hernandez has submitted this change and it was merged.
Change subject: restapi: Use multiple "vcpu_pins.vcpu_pin" instead of multiple
"vcpu_pin"
......................................................................
restapi: Use multiple "vcpu_pins.vcpu_pin" instead of multiple
"vcpu_pin"
Currently the way to specify the virtual to physical CPU pinning of a
virtual machie is to use multiple "vcpu_pin" elements:
<vm>
<cpu>
<cpu_tune>
<vcpu_pin>...</vcpu_pin>
<vcpu_pin>...</vcpu_pin>
...
</cpu_tune>
</cpu>
</vm>
In order to conform to the common practice in other parts of the API
this has been changed to use a wrapper element, in this case
"vcpu_pins":
<vm>
<cpu>
<cpu_tune>
<vcpu_pins>
<vcpu_pin>...</vcpu_pin>
<vcpu_pin>...</vcpu_pin>
...
</vcpu_pins>
</cpu_tune>
</cpu>
</vm>
Change-Id: Ia87c261c4c3ba0a668b04a2b7cf80ddd2131de01
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/types/src/main/java/org/ovirt/engine/api/restapi/types/VmMapper.java
M
backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/VmMapperTest.java
5 files changed, 85 insertions(+), 20 deletions(-)
Approvals:
Juan Hernandez: Verified; Looks good to me, approved; Passed CI tests
--
To view, visit https://gerrit.ovirt.org/46853
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ia87c261c4c3ba0a668b04a2b7cf80ddd2131de01
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