Bug #15611

'configtimeout' setting is erroneously named

Added by eric sorenson 11 months ago. Updated 10 months ago.

Status:Needs DecisionStart date:07/19/2012
Priority:NormalDue date:
Assignee:eric sorenson% Done:

0%

Category:agent
Target version:3.x
Affected Puppet version: Branch:
Keywords:http configtimeout settings

Description

There’s an agent setting ‘configtimeout’ which purports to:

:configtimeout => {
  :default  => 120,
  :desc     => "How long the client should wait for the configuration to be retrieved
  before considering it a failure.  This can help reduce flapping if too
  many clients contact the server at one time.",
},

This is not correct though, as it’s actually used for all http connections. The option should be renamed to reflect reality (perhaps to httptimeout?) and docs updated accordingly.

History

#1 Updated by Patrick Carlisle 10 months ago

  • Status changed from Unreviewed to Needs Decision
  • Assignee set to eric sorenson

Are we doing this for 3.0? I don’t think we can rename config options outside of a major version increment.

Also available in: Atom PDF