Change in vdsm[ovirt-3.5]: tool: bugfix function called as attribute.

ybronhei at redhat.com ybronhei at redhat.com
Thu Nov 13 07:42:49 UTC 2014


Yaniv Bronhaim has submitted this change and it was merged.

Change subject: tool: bugfix function called as attribute.
......................................................................


tool: bugfix function called as attribute.

to fix bug, introduced in commit 8f87f21, instead of using
selinux.is_selinux_enabled I suggest using the recently added
utils.get_selinux_enforce_mode().

In both cases the code means to check if selinux != disabled
(permissive | enforcing).

Change-Id: Id7c52e6f409b808c79653e40255cf62743b1fd0c
Signed-off-by: Mooli Tayer <mtayer at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/34487
Reviewed-by: Yaniv Bronhaim <ybronhei at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
(cherry picked from commit 975bcacbea064f8a54e3bea0af701176cc3da1b9)
Reviewed-on: http://gerrit.ovirt.org/35049
Tested-by: Yaniv Bronhaim <ybronhei at redhat.com>
---
M lib/vdsm/tool/configurators/configfile.py
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Yaniv Bronhaim: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Id7c52e6f409b808c79653e40255cf62743b1fd0c
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.5
Gerrit-Owner: mooli tayer <mtayer at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>


More information about the vdsm-patches mailing list