Change in vdsm[ovirt-3.5]: storage domain detach with force fails

danken at redhat.com danken at redhat.com
Thu Sep 4 20:47:42 UTC 2014


Dan Kenigsberg has submitted this change and it was merged.

Change subject: storage domain detach with force fails
......................................................................


storage domain detach with force fails

In jsonrpc schema masterSdUUID and masterVersion are marked as optional.
We need to provide default values for both arguments.


Bug-Url: https://bugzilla.redhat.com/1110831
Change-Id: I17058f86448c0019520ea9315da5524c5b3f2b6c
Signed-off-by: pkliczewski <piotr.kliczewski at gmail.com>
Reviewed-on: http://gerrit.ovirt.org/32503
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M vdsm/API.py
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Piotr Kliczewski: Verified
  Dan Kenigsberg: Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/32503
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I17058f86448c0019520ea9315da5524c5b3f2b6c
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipchuk at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list