Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

comment disableselinux #21

Merged
merged 1 commit into from
Dec 6, 2019
Merged

comment disableselinux #21

merged 1 commit into from
Dec 6, 2019

Conversation

ch-stark
Copy link
Contributor

@ch-stark ch-stark commented Dec 2, 2019

according to testing it should not be necessary to set selinux to permissive

@simon3z
Copy link
Member

simon3z commented Dec 3, 2019

@ch-stark maybe the PR title is misleading? Aren't you commenting disableselinux (vs uncommenting it). cc @sivak

@ch-stark ch-stark changed the title uncomment disableselinux comment disableselinux Dec 3, 2019
@simon3z
Copy link
Member

simon3z commented Dec 3, 2019

cc @MarSik

@rlopez133 rlopez133 merged commit ad7d5f7 into openshift-kni:master Dec 6, 2019
@schmaustech schmaustech mentioned this pull request Dec 9, 2019
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.

None yet

4 participants