Dan Kenigsberg has posted comments on this change.

Change subject: [WIP] Fix mounts parsing
......................................................................


Patch Set 15: (1 inline comment)

....................................................
File vdsm/storage/mount.py
Line 54:             for suffix in ("\\040(deleted)",):
What does this suffix signifies? Is there a sane way to obtain the state of 
this mountpoint? Current could would result in error if you are mounting 
something on a directory called '/mnt/bla (deleted)'

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I8fdeda5f2900d818912ef5a939b60687462d5344
Gerrit-PatchSet: 15
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Saggi Mizrahi <[email protected]>
_______________________________________________
vdsm-patches mailing list
[email protected]
https://fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to