Skip to content

Add boolean required for recent EL 7#3

Open
mscherer wants to merge 1 commit intoOSAS:masterfrom
mscherer:cleanup_selinux
Open

Add boolean required for recent EL 7#3
mscherer wants to merge 1 commit intoOSAS:masterfrom
mscherer:cleanup_selinux

Conversation

@mscherer
Copy link
Contributor

@mscherer mscherer commented Jun 8, 2017

So deploying the role, I found we were missing a boolean on recent RHEL 7. So I added that.

@mscherer mscherer changed the title Cleanup + selinux Add boolean required for recent EL 7 Jun 8, 2017
Otherwise, simple user can't login and it result in the following
AVC error:

   avc:  denied  { mounton } for  pid=695 comm="sshd" path="/" dev="xvda1" ino=2 scontext=system_u:system_r:sshd_t:s0-s0:c0.c1023 tcontext=system_u:object_r:root_t:s0 tclass=dir
name: polyinstantiation_enabled
state: yes
persistent: yes
when: ansible_selinux is not False
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"when: ansible_selinux" would suffice I guess.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess so. At first I tried "when: ansible_selinux is defined", but it seems to be defined to false even on platform without selinux (and to "a structure" when ansible_selinux is set). I think I did see a discussion also on changing that, so I should maybe dig more this part.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants