Bug #12361

"masterlog" option does not appear to be used

Added by Chris Price 4 months ago. Updated about 1 month ago.

Status:Needs Decision Start date:02/01/2012
Priority:High Due date:
Assignee:Nick Fagerlund % Done:

0%
Category:-
Target version:-
Affected Puppet version: Branch:
Keywords:
Votes: 0

Description

“masterlog” appears in our “defaults” option list for the master, as well as in our documentation (e.g. http://docs.puppetlabs.com/references/2.7.9/configuration.html ).

However it does not appear to be used in the code anywhere (it seems that —logdest is the preferred method for dealing with this these days).

There were some tickets related to this in the past, some of which were marked as dupes (e.g. #4550); however, the issue of removing this from our documentation does not actually seem like a dupe of the other tickets to me.

Related issues

related to Puppet - Bug #4550: puppetmasterd ignores masterlog setting Duplicate 08/17/2010

History

#1
Updated by Chris Price 3 months ago

  • Status changed from Unreviewed to Investigating
  • Assignee set to Chris Price

Daniel is OK with removing this option if it is no longer used; I would like to get one more set of eyes on the code before I pull it out, will ask Patrick to do so.

#2
Updated by Chris Price 3 months ago

  • Priority changed from Normal to High

#3
Updated by James Turnbull 3 months ago

Chris – this ticket might interest – https://projects.puppetlabs.com/issues/5952 and the related tickets.

#4
Updated by James Turnbull 3 months ago

This is the old branch that no one loved: https://github.com/jamtur01/puppet_old/tree/tickets/2.6.x/5952

#5
Updated by Chris Price 3 months ago

  • Assignee changed from Chris Price to Patrick Carlisle

#6
Updated by Chris Price about 1 month ago

  • Status changed from Investigating to Needs Decision
  • Assignee changed from Patrick Carlisle to Nick Fagerlund

Hey Nick—would you mind providing me with a sanity check on this? It appears to me that the docs refer to an option “—masterlog” that isn’t actually respected, and that the “—logdest” option (which does appear to be respected) does not show up in the docs. Thoughts?

Also available in: Atom PDF