Change in ovirt-engine[master]: restapi: Remove the "timezone" element

juan.hernandez at redhat.com juan.hernandez at redhat.com
Tue Sep 15 13:21:43 UTC 2015


Juan Hernandez has submitted this change and it was merged.

Change subject: restapi: Remove the "timezone" element
......................................................................


restapi: Remove the "timezone" element

Currently the VM resource uses "timezone" element to represent the time
zone. This element only allows a string:

  <vm>
    <timezone>Europe/Madrid</timezone>
  </vm>

This doesn't allow extension, and as a it was necessary to add the UTC
offset, it was replaced with a new structured `time_zone` element:

  <vm>
    <time_zone>
      <name>Europe/Madrid</name>
      <utc_offset>GMT+1</utc_offset>
    </time_zone>
  </vm>

The old `timezone` element was preserved for backwards compatibility,
but this patch removes it completely.

Change-Id: Idbd95d7f0f70fb22b604644196b88b071f027060
Signed-off-by: Juan Hernandez <juan.hernandez at 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/VmBaseMapper.java
M backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/TemplateMapperTest.java
M backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/VmMapperTest.java
6 files changed, 24 insertions(+), 25 deletions(-)

Approvals:
  Juan Hernandez: Verified; Looks good to me, approved
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/46034
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Idbd95d7f0f70fb22b604644196b88b071f027060
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list