Change in vdsm[master]: BZ#841863 Change exception thrown by the fileUtils module

danken at redhat.com danken at redhat.com
Sat Aug 4 18:34:00 UTC 2012


Dan Kenigsberg has submitted this change and it was merged.

Change subject: BZ#841863 Change exception thrown by the fileUtils module
......................................................................


BZ#841863 Change exception thrown by the fileUtils module

Serializing obejcts that aren't canonical might cause issue with
pickling. In anycase, the gobal exceptions should not be used so low in
the stack.

Change-Id: I0c6a4fded368e89f057d244665be0e37d71f2d08
Signed-off-by: Saggi Mizrahi <smizrahi at redhat.com>
---
M vdsm/storage/fileSD.py
M vdsm/storage/fileUtils.py
M vdsm/storage/localFsSD.py
M vdsm/storage/nfsSD.py
M vdsm/storage/sp.py
5 files changed, 32 insertions(+), 9 deletions(-)

Approvals:
  Alon Bar-Lev: Verified
  Federico Simoncelli: Looks good to me, approved
  Dan Kenigsberg: 


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I0c6a4fded368e89f057d244665be0e37d71f2d08
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Asaf Shakarchi <asaf at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Igor Lvovsky <ilvovsky at redhat.com>
Gerrit-Reviewer: Mark Wu <wudxw at linux.vnet.ibm.com>
Gerrit-Reviewer: Ofer Schreiber <oschreib at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Xu He Jie <xuhj at linux.vnet.ibm.com>


More information about the vdsm-patches mailing list