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

piotr.kliczewski at gmail.com piotr.kliczewski at gmail.com
Mon Aug 10 07:14:45 UTC 2015


Piotr Kliczewski 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/44594
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I45128d928421e27f9eac2b73ac6fc03f1682f545
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.6
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: Ori Liel <oliel at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list