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

              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:

                  Zendesk Support