Uploaded image for project: 'Puppet'
  1. Puppet
  2. PUP-3666

Replace 'configtimeout' with separate HTTP connect and read timeout settings

    Details

    • Template:
    • Story Points:
      1
    • Sprint:
      Platform Client 2014-11-26

      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.


      Final decision:

      • New setting: http_connect_timeout — controls how long Puppet should attempt to make a connection. Normal to have a small value for this.
      • New setting: http_read_timeout — controls how long Puppet should allow transfers to continue. Normal to let this last a long time or be infinite, since some things just take a while to compile or download.
      • configtimeout is deprecated, and logs a warning if it's set.

        Attachments

          Issue Links

            Activity

              jsd-sla-details-panel

                People

                • Assignee:
                  nick.fagerlund Nicholas Fagerlund
                  Reporter:
                  redmine.exporter redmine.exporter
                  QA Contact:
                  Kurt Wall
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  8 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: