Ayal Baron has posted comments on this change.
Change subject: Extracted image related functinality to another class ......................................................................
Patch Set 19: Looks good to me, approved
Approving assuming: 1. You made no changes in the code you transferred from sp.py to the new module 2. You will fix the unsecured issue
-- To view, visit http://gerrit.ovirt.org/191 To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment Gerrit-Change-Id: Ic29dc8d964f0c2e56329e7044a3b2f48a9f2e193 Gerrit-PatchSet: 19 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Saggi Mizrahi smizrahi@redhat.com Gerrit-Reviewer: Ayal Baron abaron@redhat.com Gerrit-Reviewer: Dan Kenigsberg danken@redhat.com Gerrit-Reviewer: Eduardo ewarszaw@redhat.com Gerrit-Reviewer: Federico Simoncelli fsimonce@redhat.com Gerrit-Reviewer: Igor Lvovsky ilvovsky@redhat.com Gerrit-Reviewer: Saggi Mizrahi smizrahi@redhat.com
vdsm-patches@lists.fedorahosted.org