Change in vdsm[master]: vdsm-tool: add the sebool configuration module

danken at redhat.com danken at redhat.com
Mon Dec 3 20:22:49 UTC 2012


Dan Kenigsberg has posted comments on this change.

Change subject: vdsm-tool: add the sebool configuration module
......................................................................


Patch Set 2: Fails

(1 inline comment)

seems that I'm missing something basic.

....................................................
File vdsm-tool/seboolsetup.py
Line 39:     sebool_obj = seobject.booleanRecords()
Line 40:     sebool_status = sebool_obj.get_all()
Line 41: 
Line 42:     sebool_obj.start()
Line 43: 
to verify this patch, I've manually

 setsebool virt_use_nfs off

and install vdsm. The boolean was not turned on; how come do I have both

 >>> sebool_status["virt_use_nfs"]
 [1, 1, 1]
 $ getsebool virt_use_nfs
 virt_use_nfs --> off
Line 44:     for sebool_variable in VDSM_SEBOOL_LIST:
Line 45:         if status and not all(sebool_status[sebool_variable]):
Line 46:             sebool_obj.modify(sebool_variable, SEBOOL_ENABLED)
Line 47: 


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I01baf1c17f7cad2a0525b261be6b5ac648576252
Gerrit-PatchSet: 2
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: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server


More information about the vdsm-patches mailing list