The Puppet Labs Issue Tracker has Moved: https://tickets.puppetlabs.com

Bug #3984

SELinux gives warnings about making changes that it does not actually make

Added by Darrell Fuhriman over 4 years ago. Updated almost 2 years ago.

Status:ClosedStart date:06/09/2010
Priority:NormalDue date:
Assignee:-% Done:

0%

Category:SELinux
Target version:2.7.20
Affected Puppet version:0.25.5 Branch:https://github.com/puppetlabs/puppet/pull/821
Keywords:

We've Moved!

Ticket tracking is now hosted in JIRA: https://tickets.puppetlabs.com

This issue is currently not available for export. If you are experiencing the issue described below, please file a new ticket in JIRA. Once a new ticket has been created, please add a link to it that points back to this Redmine ticket.


Description

See my previous (ignored) comments in bug #2792 note 6.

The attached patch resolves the issue.

selinux-warnings.patch Magnifier (583 Bytes) Darrell Fuhriman, 06/09/2010 10:47 pm


Related issues

Related to Puppet - Bug #4870: Puppet reports that SELinux attributes have been set when... Re-opened 09/29/2010

History

#1 Updated by James Turnbull over 4 years ago

  • Status changed from Unreviewed to Investigating
  • Assignee set to Jesse Wolfe
  • Target version set to 49

#2 Updated by Markus Roberts about 4 years ago

  • Status changed from Investigating to In Topic Branch Pending Review
  • Assignee deleted (Jesse Wolfe)
  • Target version changed from 49 to 52
  • Branch set to http://github.com/MarkusQ/puppet/tree/ticket/next/3984

Patch ported to next, pushed to github, etc.

#3 Updated by James Turnbull about 4 years ago

  • Target version changed from 52 to 2.7.x

#4 Updated by James Turnbull about 3 years ago

  • Assignee set to Jacob Helwig

Jacob – this one was approved on the list (https://groups.google.com/forum/#!topic/puppet-dev/TFw8TkFpNhs) but doesn’t look like it made it to a merge.

#5 Updated by Jacob Helwig almost 3 years ago

  • Assignee changed from Jacob Helwig to Anonymous

Daniel,

This is sitting in a branch in Markus’s fork on GitHub from back before we started using Patchwork, or pull requests.

#6 Updated by Darrell Fuhriman over 2 years ago

Hey guys, any chance we can get this one merged? It’s been sitting around for five months since the last bit of action (and really two years, which is kind of crazy for an already approved patch)…

#7 Updated by Anonymous over 2 years ago

  • Assignee deleted (Anonymous)
  • Branch changed from http://github.com/MarkusQ/puppet/tree/ticket/next/3984 to https://github.com/puppetlabs/puppet/pull/821

Sorry, this slipped through the cracks during the transition.

I have transformed this into a pull request at https://github.com/puppetlabs/puppet/pull/821

That gets it back into the regular process for code review and merging. It clearly isn’t ready yet: there are absolutely no tests, so those need to get written before we can merge the code into the tree, but at least it will not get lost again.

#8 Updated by Anonymous about 2 years ago

  • Status changed from In Topic Branch Pending Review to Merged - Pending Release
  • Target version changed from 2.7.x to 2.7.20

#10 Updated by Matthaus Owens about 2 years ago

Released in Puppet 3.0.0-rc6. Release pending for 2.7.x.

#11 Updated by Matthaus Owens almost 2 years ago

  • Status changed from Merged - Pending Release to Closed

Released in Puppet 2.7.20

Also available in: Atom PDF