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