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(a)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(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Fred Rolland <frolland(a)redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Ori Liel <oliel(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org