Maor Lipchuk has submitted this change and it was merged.
Change subject: restapi: Avoid having leftovers when adding NFS SD.
......................................................................
restapi: Avoid having leftovers when adding NFS SD.
If the process, of adding/importing NFS Storage Domain,
fails while validating the storage domain name,
the storage server connection data, which was added in the process,
does not gets deleted from the DB.
The proposed fix validates the name inside the try catch clause.
Change-Id: I7b001468fbbb3a30c08f8efdf0db74db946874d5
Bug-Url:
https://bugzilla.redhat.com/1208057
Signed-off-by: Maor Lipchuk <mlipchuk(a)redhat.com>
---
M
backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BackendStorageDomainsResource.java
1 file changed, 4 insertions(+), 5 deletions(-)
Approvals:
Juan Hernandez: Looks good to me, approved
Jenkins CI: Passed CI tests
Maor Lipchuk: Verified
--
To view, visit
https://gerrit.ovirt.org/43923
To unsubscribe, visit
https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I7b001468fbbb3a30c08f8efdf0db74db946874d5
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Maor Lipchuk <mlipchuk(a)redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini(a)redhat.com>
Gerrit-Reviewer: Daniel Erez <derez(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Juan Hernández
Gerrit-Reviewer: Maor Lipchuk <mlipchuk(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org