Change in vdsm[master]: volume: unify the backing file path computation

danken at redhat.com danken at redhat.com
Thu May 8 17:03:47 UTC 2014


Dan Kenigsberg has submitted this change and it was merged.

Change subject: volume: unify the backing file path computation
......................................................................


volume: unify the backing file path computation

This patch unifies the backing file path computation logic in a single
function (getBackingVolumePath).

Change-Id: I7575f4dd3c6918dfee72ac4c23f27a56a8390b9b
Signed-off-by: Federico Simoncelli <fsimonce at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/26918
Reviewed-by: Francesco Romani <fromani at redhat.com>
Reviewed-by: Allon Mureinik <amureini at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M vdsm/storage/image.py
M vdsm/storage/volume.py
2 files changed, 11 insertions(+), 7 deletions(-)

Approvals:
  Federico Simoncelli: Verified
  Allon Mureinik: Looks good to me, but someone else must approve
  Dan Kenigsberg: Looks good to me, approved
  Francesco Romani: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I7575f4dd3c6918dfee72ac4c23f27a56a8390b9b
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Vinzenz Feenstra <vfeenstr at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list