Issue #16793 has been updated by Andrew  Parker.

Target version set to 3.x

I don't consider this a small bugfix. So this should not go into a bugfix 
release, but a feature release should be fine.
----------------------------------------
Bug #16793: Dead settings are getting silly. Let's kill them all!
https://projects.puppetlabs.com/issues/16793#change-72776

Author: Nick Fagerlund
Status: Needs Decision
Priority: Normal
Assignee: eric sorenson
Category: 
Target version: 3.x
Affected Puppet version: 3.0.0
Keywords: 
Branch: 


I've found at least six dead settings on `master` today, and have already 
handled two this week in individual issues. Time for a mass cleanup. Today's 
haul so far:

- `certdnsnames` -- dead for about a year now. 
- `ca_md` -- not referenced anywhere in code. 
- `catalog_format` -- deprecated since at least 2009.
- `dynamicfacts` -- not referenced anywhere in code. 
- `localconfig` -- not referenced anywhere in code. 

There was also `authconfig` and `ldapnodes` (already dealt with in other 
issues), and I'm getting a real hinky feeling from `ignorecache`, since I can 
see two places in the codebase that SET it, but nowhere that READS it. Haven't 
tested behavior on that one yet.

I was working alphabetically on the full list of settings and got through "L". 
I suspect there are more to be found. I'm handing this off because the rest of 
the searching should probably be done by someone who's faster at it than me. My 
approach so far has been:

* Ack for `"Puppet(\.settings)?\[:setting_name\]"`
* If that turns up nothing, ack for `setting_name` alone.
* If that looks sketchy, check defaults.rb and make sure there's not a `:hook 
=>` for the setting with meaningful side effects.


-- 
You have received this notification because you have either subscribed to it, 
or are involved in it.
To change your notification preferences, please click here: 
http://projects.puppetlabs.com/my/account

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Bugs" group.
To post to this group, send email to [email protected].
To unsubscribe from this group, send email to 
[email protected].
For more options, visit this group at 
http://groups.google.com/group/puppet-bugs?hl=en.

Reply via email to