HomePhorge

(#7624) Auditing should not be enabled by default for purged resources.
c5448b79e690Unpublished

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

(#7624) Auditing should not be enabled by default for purged resources.

  • Auditing creates logging messages as of 2.6.5 so it should not be enabled by default.
  • This patch removes the :audit => :all settting from resources created via self.instances (which is used for purging).

Please note that we believe this change to be safe, and *should* not result in
user-visible behavioural differences when you use the instances method on a
type, but we can't give you a perfect assurance of that.

If you do have code that depends on the current behaviour, and it misbehaves
after this patch, please let us know so we can weep ^W find another solution
that works for everyone.

Reviewed-By: Daniel Pittman <daniel@puppetlabs.com>
Reviewed-By: Nigel Kersten <nigel@puppetlabs.com>

Details

Provenance
Dan Bode <bodepd@gmail.com>Authored on
Daniel Pittman <daniel@puppetlabs.com>Committed on Jun 3 2011, 7:40 PM
vanmeeuwenPushed on Jun 2 2015, 2:22 PM
Parents
rPUc295ae699ff7: Merge branch 'bug/2.7rc/7746-fix-broken-master-start' into 2.7rc
Branches
Unknown
Tags
Unknown

Event Timeline

Daniel Pittman <daniel@puppetlabs.com> committed rPUc5448b79e690: (#7624) Auditing should not be enabled by default for purged resources. (authored by Dan Bode <bodepd@gmail.com>).Jun 3 2011, 7:40 PM