Change in vdsm[master]: clientIF: prepareVolumePath payload cleanup

danken at redhat.com danken at redhat.com
Fri Mar 28 15:07:11 UTC 2014


Dan Kenigsberg has submitted this change and it was merged.

Change subject: clientIF: prepareVolumePath payload cleanup
......................................................................


clientIF: prepareVolumePath payload cleanup

the prepareVolumePath code path for cdrom/floppy
images is complicated due the fact the code has to
deal with both regular and payload-created
images (vmPayload, used by cloud-init).

The code has been patched various times, and in the end the
code became more complex than is needed.

This patch refactors the existing code in order to simplify
it, without changes of behaviour.

Change-Id: I058206b7506ddbb5ec087c9ea0963a10ed57affb
Relates-To: http://gerrit.ovirt.org/22928/
Relates-To: https://bugzilla.redhat.com/1047356
Signed-off-by: Francesco Romani <fromani at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/24636
Reviewed-by: Nir Soffer <nsoffer at redhat.com>
Reviewed-by: Vinzenz Feenstra <vfeenstr at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M vdsm/clientIF.py
1 file changed, 30 insertions(+), 30 deletions(-)

Approvals:
  Nir Soffer: Looks good to me, but someone else must approve
  Vinzenz Feenstra: Looks good to me, but someone else must approve
  Dan Kenigsberg: Looks good to me, approved
  Francesco Romani: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I058206b7506ddbb5ec087c9ea0963a10ed57affb
Gerrit-PatchSet: 8
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Antoni Segura Puimedon <asegurap at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Eduardo <ewarszaw at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Martin Polednik <mpoledni at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer 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