Change in vdsm[master]: BZ#726105 - Check that qemu can also access file mounts

Saggi Mizrahi smizrahi at redhat.com
Tue Aug 2 14:11:58 UTC 2011


Saggi Mizrahi has posted comments on this change.

Change subject: BZ#726105 - Check that qemu can also access file mounts
......................................................................


Patch Set 1: (2 inline comments)

....................................................
File vdsm/storage/storage_connection.py
Line 27:             constants.DISKIMAGE_GROUP, dirPath,
You're kidding

....................................................
File vdsm/supervdsmServer.py
Line 81:                 gid = resolveGid(group)
Not here, maybe the caller of validateAccess.

--
To view, visit http://gerrit.usersys.redhat.com/776
To unsubscribe, visit http://gerrit.usersys.redhat.com/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I9367410e0172569b4ccb83d5cea1064771f66bb7
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Saggi Mizrahi <smizrahi at redhat.com>
Gerrit-Reviewer: Ayal Baron
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Eduardo Warszawski <ewarszaw at redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi at redhat.com>


More information about the vdsm-patches mailing list