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

Fixes #242 & Reverts #253 #256

Merged
merged 2 commits into from
Dec 15, 2015
Merged

Fixes #242 & Reverts #253 #256

merged 2 commits into from
Dec 15, 2015

Conversation

Ginja
Copy link
Contributor

@Ginja Ginja commented Dec 13, 2015

@Ginja
Fixes #242
1bab4af 
Removed the management of selinux
@Ginja
Reverted #253
47e1729 
PR #253 broke the world after #252 was merged
@Ginja Ginja mentioned this pull request Dec 13, 2015
Closed
@codecov-io
Copy link

Current coverage is 48.63%

Merging #256 into master will decrease coverage by -21.27% as of 6a2f4d4

@@            master    #256   diff @@
======================================
  Files            6       6       
  Stmts          329     329       
  Branches         0       0       
  Methods          0       0       
======================================
- Hit            230     160    -70
  Partial          0       0       
- Missed          99     169    +70

Review entire Coverage Diff as of 6a2f4d4

Powered by Codecov. Updated on successful CI builds.

@Ginja Ginja changed the title Fixes #242 & Reverts #253 Removes selinux management & Reverts #253 Dec 14, 2015
@Ginja Ginja changed the title Removes selinux management & Reverts #253 Fixes #242 & Reverts #253 Dec 14, 2015
@Ginja Ginja mentioned this pull request Dec 14, 2015
Merged
johnbellone added a commit that referenced this pull request Dec 15, 2015
@johnbellone
Merge pull request #256 from visioncritical/remove-selinux
eedc49d 
Fixes #242 & Reverts #253
@johnbellone johnbellone merged commit eedc49d into sous-chefs:master Dec 15, 2015
@Sauraus
Copy link

Sauraus commented Jan 15, 2016

When is this going to make it into a supermarket release? :)

@knightorc knightorc deleted the remove-selinux branch June 6, 2018 23:05
@lock
Copy link

lock bot commented Apr 25, 2020

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Apr 25, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Basis for selinux set to disabled
4 participants
@Ginja @codecov-io @Sauraus @johnbellone