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

The new Rest::Client does not respect the server_list setting

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Normal
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: PUP 6.0.0
    • Component/s: None
    • Labels:
      None
    • Template:
      PUP Bug Template
    • Acceptance Criteria:
      Hide

      Puppet::Rest::Client respects the server_list setting and command line flag

      Show
      Puppet::Rest::Client respects the server_list setting and command line flag
    • Team:
      Froyo
    • Method Found:
      Automated Test
    • Release Notes:
      Not Needed
    • QA Risk Assessment:
      Needs Assessment

      Description

      The way the new rest client is currently being used (see https://github.com/puppetlabs/puppet/blob/master/lib/puppet/rest/routes.rb) does not respect the server list setting, because we are always setting a default server, which skips server resolution (see https://github.com/puppetlabs/puppet/blob/master/lib/puppet/rest/route.rb#L66-L90). Assuming that HA expects the CA to fail over along with the rest of the master functions, we need to update this logic so that if server_list is specified, the values there are preferred over the ca_server setting that we would normally check (which itself defaults to the server setting).

        Attachments

          Activity

            People

            Assignee:
            amy.sahli Amy Sahli
            Reporter:
            maggie Maggie Dreyer
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Zendesk Support