Change in ovirt-engine[master]: restapi: Remove unneeded calls in SD extend

amureini at redhat.com amureini at redhat.com
Thu Aug 6 12:04:01 UTC 2015


Allon Mureinik has submitted this change and it was merged.

Change subject: restapi: Remove unneeded calls in SD extend
......................................................................


restapi: Remove unneeded calls in SD extend

Removing unneeded code when extending a storage domain.
The same logic already exists in ExtendSANStorageDomainCommand.

Now, the 'host' parameter is not needed and can be removed from the update
storage domain definition in the rsdl.

Backward compatibility is preserved as the 'host' parameter is not
removed from XML schema.
If the parameter is present, it will be ignored.

Change-Id: I45128d928421e27f9eac2b73ac6fc03f1682f545
Bug-Url: https://bugzilla.redhat.com/1213961
Signed-off-by: Fred Rolland <frolland at redhat.com>
---
M backend/manager/modules/restapi/interface/definition/src/main/resources/rsdl_metadata.yaml
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendStorageDomainResource.java
2 files changed, 4 insertions(+), 36 deletions(-)

Approvals:
  Ori Liel: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Allon Mureinik: Looks good to me, but someone else must approve
  Freddy Rolland: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I45128d928421e27f9eac2b73ac6fc03f1682f545
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Fred Rolland <frolland at redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland 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