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

Remove Ruby CA code and associated tests

    Details

    • Type: Task
    • Status: Closed
    • Priority: Normal
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: PUP 6.0.0
    • Component/s: None
    • Labels:
      None
    • Template:
    • Acceptance Criteria:
      • Puppet::SSL::CertificateAuthority and associated tests are removed
      • ca_location logic is removed
      • Other utility methods for detecting whether or not we are the CA are removed
    • Team:
      Server
    • Release Notes:
      New Feature
    • Release Notes Summary:
      Hide
      Puppet no longer has a Ruby CA. All CA actions now rely entirely on the Clojure implementation in Puppet Server. It can be interacted with via the CA API and the `puppetserver ca` command, which leverages the API via subcommands like those provided by `puppet cert`.
      Show
      Puppet no longer has a Ruby CA. All CA actions now rely entirely on the Clojure implementation in Puppet Server. It can be interacted with via the CA API and the `puppetserver ca` command, which leverages the API via subcommands like those provided by `puppet cert`.
    • QA Risk Assessment:
      Needs Assessment

      Description

      In addition to removing the Ruby CLI for CA actions (see PUP-3648) and the CA-related settings from puppet.conf, we need to actually delete the underlying CA code and its associated tests. This will also allow us to remove some of the more complicated logic that had to be added to the SSL::Host class around ca_location (see https://github.com/puppetlabs/puppet/blob/master/lib/puppet/ssl/host.rb#L94-L114), since it will necessarily be remote once this removal has taken place. Likewise, we should be able to remove the various checks for whether or not we are the CA, simplifying logic elsewhere.

        Attachments

          Activity

            jsd-sla-details-panel

              People

              • Assignee:
                maggie Maggie Dreyer
                Reporter:
                maggie Maggie Dreyer
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: