Dan Kenigsberg has posted comments on this change.

Change subject: Moving getSELinuxEnforeMode to utils
......................................................................


Patch Set 2: Code-Review-1

(1 comment)

http://gerrit.ovirt.org/#/c/34446/2/lib/vdsm/utils.py
File lib/vdsm/utils.py:

Line 1135:     flags |= os.O_NONBLOCK
Line 1136:     fcntl.fcntl(fd, fcntl.F_SETFL, flags)
Line 1137: 
Line 1138: 
Line 1139: def getSELinuxEnforceMode():
new functions in utils should be named get_selinux_enforce_mode()
Line 1140:     """
Line 1141:     Returns the SELinux mode as reported by kernel.
Line 1142: 
Line 1143:     1 = enforcing - SELinux security policy is enforced.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I801adfb0d8835ef9083f24ee5732219b8f719315
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Vinzenz Feenstra <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to