Change in ovirt-engine[master]: restapi: Remove deprecated power management elements

juan.hernandez at redhat.com juan.hernandez at redhat.com
Sat Sep 26 17:50:29 UTC 2015


Juan Hernandez has submitted this change and it was merged.

Change subject: restapi: Remove deprecated power management elements
......................................................................


restapi: Remove deprecated power management elements

Currently the host power management configuration is part of the host
resource, using embedded configuration elements:

  <power_management type="apc">
    <enabled>true</enabled>
    <address>myaddress</address>
    <username>myaddress</username>
    <options>
      <option name="port" value="22/>
      </option name="slot" value="5/>
    </options>
    ...
  </power_management>

This has been changed some time ago, in order to support multiple power
management agents, introducing a new "/hosts/{host:id}/fenceagents"
collection.

The old "type" attribute, the old "address", "username" and "password"
elements, and the inner "agents" element directly inside
"power_management" were preserved for backwards compatibility. This
patch completely remove all these elements, so the only way to query or
modify the power management agents is now the
"/hosts/{host:id}/fenceagents" sub-collection.

Change-Id: I8c57bfd49e2ceaffb4f1c8f1c371a25f0bed68ef
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/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendCapabilitiesResource.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/main/java/org/ovirt/engine/api/restapi/util/FenceOptionsParser.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/jaxrs/src/test/java/org/ovirt/engine/api/restapi/util/FenceOptionsParserTest.java
D backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/DeprecatedPowerManagementMapper.java
M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/HostMapper.java
D backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/DeprecatedPowerManagementMapperTest.java
M backend/manager/modules/restapi/types/src/test/java/org/ovirt/engine/api/restapi/types/HostMapperTest.java
14 files changed, 150 insertions(+), 670 deletions(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I8c57bfd49e2ceaffb4f1c8f1c371a25f0bed68ef
Gerrit-PatchSet: 4
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: Ori Liel <oliel at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list