Yaniv Bronhaim has posted comments on this change.
Change subject: Remove redundant supervsdm.validateAccess.
......................................................................
Patch Set 1: Code-Review-1
sorry, I leaded you wrong. you should not remove the verb from supervdsm , and actually
use it
what you are trying to validate is if you have access as qemu process, which runs as qemu
user. if you use oop for that, you actually check access for vdsm user... validateAccess
in supervdsm let you check access for any user you want
--
To view, visit
http://gerrit.ovirt.org/20838
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Id37865225ea2d29361d6e588f9bb0acf7bf71cec
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Eduardo <ewarszaw(a)redhat.com>
Gerrit-Reviewer: Ayal Baron <abaron(a)redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken(a)redhat.com>
Gerrit-Reviewer: Saggi Mizrahi <smizrahi(a)redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei(a)redhat.com>
Gerrit-Reviewer: Yeela Kaplan <ykaplan(a)redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No