Change in vdsm[master]: readlines - remove redundant pathExists check
abaron at redhat.com
abaron at redhat.com
Sun Apr 7 15:24:03 UTC 2013
Ayal Baron has posted comments on this change.
Change subject: readlines - remove redundant pathExists check
......................................................................
Patch Set 1:
Saggi, try except and do what exactly?
If file doesn't exist reading the metadata should fail.
at most we could translate the exception...
--
To view, visit http://gerrit.ovirt.org/13664
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Id73f4bd177465f253eb4f65403b17ff7375e329c
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron at redhat.com>
Gerrit-Reviewer: Eduardo <ewarszaw at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
More information about the vdsm-patches
mailing list