Change in vdsm[master]: createVolume: fix default values for two args

danken at redhat.com danken at redhat.com
Thu Sep 15 09:06:26 UTC 2011


Dan Kenigsberg has submitted this change and it was merged.

Change subject: createVolume: fix default values for two args
......................................................................


createVolume: fix default values for two args

When createVolume is called without specifying explicit values for
srcImgUUID and srcVolUUID, None value was passed done the chain, running
havoc in async task creation.

This was not noticed since vdsClient and RHEV-M fill these args with
volume.BLANK_UUID. This patch makes Vdsm itself follow suit and have
sane defaults.

Change-Id: If1befbba5e5b4cd649d7e50fe89055c5f3cccce1
---
M vdsm/storage/spm.py
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Dan Kenigsberg: Verified
  Igor Lvovsky: Looks good to me, approved


--
To view, visit http://gerrit.usersys.redhat.com/937
To unsubscribe, visit http://gerrit.usersys.redhat.com/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: If1befbba5e5b4cd649d7e50fe89055c5f3cccce1
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky at redhat.com>


More information about the vdsm-patches mailing list