Change in vdsm[master]: BZ#747917 Don't get information about mountpoints

danken at redhat.com danken at redhat.com
Sun Dec 25 16:29:15 UTC 2011


Dan Kenigsberg has submitted this change and it was merged.

Change subject: BZ#747917 Don't get information about mountpoints
......................................................................


BZ#747917 Don't get information about mountpoints

The regular os.walk() function tries to identify the files present in
the given path. Avoiding to descend into the mountpoint is not enough
to prevent vdsm from getting stuck if a NFS mount is unreachable, we
should also prevent any other operation, eg: os.path.isdir().

Change-Id: I16a9e54586daa766e420fa8571b19a0b744b602d
---
M vdsm/storage/hsm.py
M vdsm/storage/misc.py
2 files changed, 73 insertions(+), 7 deletions(-)

Approvals:
  Dan Kenigsberg: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I16a9e54586daa766e420fa8571b19a0b744b602d
Gerrit-PatchSet: 3
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Douglas Schilling Landgraf <dougsland at redhat.com>
Gerrit-Reviewer: Eduardo <ewarszaw at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>


More information about the vdsm-patches mailing list