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

(FACT-596) Update selinux_config_policy fact #844

Closed
wants to merge 1 commit into from
Closed

(FACT-596) Update selinux_config_policy fact #844

wants to merge 1 commit into from

Conversation

frconil
Copy link

@frconil frconil commented Feb 4, 2015

The newer version of selinux use the term "Loaded policy name"
in lieu of "Policy from config file".

This updated version matches on both for backard compatibility
reasons.

This should fix FACT-756 as well.

The newer version of selinux use the term "Loaded policy name"
in lieu of "Policy from config file".

This updated version matches on both for backard compatibility
reasons.

This should fix FACT-756 as well.
@puppetcla
Copy link

CLA signed by all contributors.

@kylog
Copy link

kylog commented Feb 4, 2015

@peterhuene do you know if cfacter needs touchup to match this?

@joshcooper
Copy link
Contributor

@frconil Thank you for your contribution!

PR Triage Notes:

  • @frconil can you add a unit test for this change?
  • @frconil can you retarget at the stable branch?
  • We will also need to verify if this is an issue for cfacter, and file a ticket if needed /cc @peterhuene @MikaelSmith

@ScottGarman ScottGarman added the triaged Jira issue has been created for this label Feb 4, 2015
@frconil
Copy link
Author

frconil commented Feb 4, 2015

@joshcooper I've added the changes and test on the stable branch, should I open a new PR and refer to this one?

@kylog
Copy link

kylog commented Feb 11, 2015

@frconil yes, can you close in favor of a pull on stable (also including the spec changes)?

Thanks!

@kylog
Copy link

kylog commented Feb 11, 2015

Oops! I see you did as #848. Thanks!

Closing this one in favor of that.

@kylog kylog closed this Feb 11, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
triaged Jira issue has been created for this
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants