Bug #15611
'configtimeout' setting is erroneously named
| Status: | Needs Decision | Start date: | 07/19/2012 | |
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assignee: | % 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.