Change in ovirt-engine[master]: restapi: Avoid having leftovers when adding NFS SD.

mlipchuk at redhat.com mlipchuk at redhat.com
Thu Jul 23 08:01:14 UTC 2015


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 at 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 at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Juan Hernández
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list