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.

Sign up for GitHub

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

Jump to bottom

Update removed selinux facts #124

Closed
raphink opened this issue Aug 5, 2019 · 3 comments · Fixed by #125
Closed

Update removed selinux facts #124

raphink opened this issue Aug 5, 2019 · 3 comments · Fixed by #125
Labels
bug

Comments

@raphink
Copy link
Member

raphink commented Aug 5, 2019

Update 1a9c20c to CentOS facts removed selinux_* facts which were listed before. @rodjek do you think you could redo these updates without deactivating selinux?
@saimonn saimonn mentioned this issue Aug 6, 2019
Closed
@rodjek
Copy link
Member

rodjek commented Aug 12, 2019

This is an unfortunate artifact of generating the fact sets on different base images. I'm pulling down the official CentOS 6 vagrant image to see what the default behaviour is (selinux enabled or disabled) and will regenerate the fact sets if it defaults to selinux enabled.

@rodjek
Copy link
Member

rodjek commented Aug 12, 2019

It does indeed ship with selinux enforcing by default. I'll rebuild the fact sets today, sorry about that!

@raphink
Copy link
Member Author

raphink commented Aug 12, 2019

Thanks very much @rodjek !

rodjek added a commit to rodjek/facterdb that referenced this issue Aug 26, 2019
@rodjek
Update RHEL based fact sets with default SELinux values
5d0b917 
Fixes voxpupuli#124
@rodjek rodjek mentioned this issue Aug 26, 2019
Merged
rodjek added a commit that referenced this issue Aug 26, 2019
@rodjek
Merge pull request #125 from rodjek/gh-124
08ba879 
Update RHEL based fact sets with default SELinux values
@rodjek rodjek added the bug label Aug 28, 2019
@bodgit bodgit mentioned this issue Aug 30, 2019
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update RHEL based fact sets with default SELinux values rodjek/facterdb
2 participants
@rodjek @raphink