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

Fix for selinux::module absent case failed notify #59

Merged
merged 1 commit into from
Sep 30, 2015

Conversation

ps-jay
Copy link

@ps-jay ps-jay commented Sep 30, 2015

Before this, selinux::module ensure => 'absent' would fail because:

  • Exec["${name}-buildmod"] wouldn't be created
  • Exec["${name}-checkloaded"] therefore couldn't notify it

Also:

  • Add rspec testing for selinux::module
  • Add custom fact selinux_config_policy to spec_helper

Before this, selinux::module ensure => 'absent' would fail because:
- Exec["${name}-buildmod"] wouldn't be created
- Exec["${name}-checkloaded"] therefore couldn't notify it

Also:
- Add rspec testing for selinux::module
- Add custom fact selinux_config_policy to spec_helper
@jfryman
Copy link
Contributor

jfryman commented Sep 30, 2015

🤘. Thanks for the code.

jfryman added a commit that referenced this pull request Sep 30, 2015
Fix for selinux::module absent case failed notify
@jfryman jfryman merged commit 09e78fb into voxpupuli:master Sep 30, 2015
@ps-jay ps-jay deleted the fix-module-absent-notify branch October 1, 2015 04:52
@ps-jay
Copy link
Author

ps-jay commented Oct 1, 2015

No worries : )

If it isn't too much trouble, could you tag a 0.2.6 release?

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