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