beaker-puppet 2.0.0 → 3.0.1

data/CODEOWNERS

@@ -0,0 +1 @@
+* @puppetlabs/phoenix @puppetlabs/unicorn

data/README.md

@@ -1,12 +1,11 @@
 # beaker-puppet: The Puppet-Specific Beaker Library
 
-[![License](https://img.shields.io/github/license/voxpupuli/beaker-puppet.svg)](https://github.com/voxpupuli/beaker-puppet/blob/master/LICENSE)
-[![Test](https://github.com/voxpupuli/beaker-puppet/actions/workflows/test.yml/badge.svg)](https://github.com/voxpupuli/beaker-puppet/actions/workflows/test.yml)
-[![codecov](https://codecov.io/gh/voxpupuli/beaker-puppet/branch/master/graph/badge.svg?token=Mypkl78hvK)](https://codecov.io/gh/voxpupuli/beaker-puppet)
-[![Release](https://github.com/voxpupuli/beaker-puppet/actions/workflows/release.yml/badge.svg)](https://github.com/voxpupuli/beaker-puppet/actions/workflows/release.yml)
+[![License](https://img.shields.io/github/license/puppetlabs/beaker-puppet.svg)](https://github.com/puppetlabs/beaker-puppet/blob/master/LICENSE)
+[![Test](https://github.com/puppetlabs/beaker-puppet/actions/workflows/test.yml/badge.svg)](https://github.com/puppetlabs/beaker-puppet/actions/workflows/test.yml)
+[![codecov](https://codecov.io/gh/puppetlabs/beaker-puppet/branch/master/graph/badge.svg?token=Mypkl78hvK)](https://codecov.io/gh/puppetlabs/beaker-puppet)
+[![Release](https://github.com/puppetlabs/beaker-puppet/actions/workflows/release.yml/badge.svg)](https://github.com/puppetlabs/beaker-puppet/actions/workflows/release.yml)
 [![RubyGem Version](https://img.shields.io/gem/v/beaker-puppet.svg)](https://rubygems.org/gems/beaker-puppet)
 [![RubyGem Downloads](https://img.shields.io/gem/dt/beaker-puppet.svg)](https://rubygems.org/gems/beaker-puppet)
-[![Donated by Puppet Inc](https://img.shields.io/badge/donated%20by-Puppet%20Inc-fb7047.svg)](#transfer-notice)
 
 The purpose of this library is to hold all puppet-specific info & DSL methods.
 This includes all helper & installer methods.
@@ -79,14 +78,6 @@ file, or you can provide a beaker-hostgenerator value to the `TEST_TARGET`
 environment variable. You can also specify the tests that get executed with the
 `TESTS` environment variable.
 
-## Transfer Notice
-
-This plugin was originally authored by [Puppet Inc](http://puppet.com).
-The maintainer preferred that Puppet Community take ownership of the module for future improvement and maintenance.
-Existing pull requests and issues were transferred over, please fork and continue to contribute here.
-
-Previously: https://github.com/puppetlabs/beaker
-
 ## License
 
 This gem is licensed under the Apache-2 license.

data/Rakefile

@@ -189,7 +189,7 @@ else
   GitHubChangelogGenerator::RakeTask.new :changelog do |config|
     config.header = "# Changelog\n\nAll notable changes to this project will be documented in this file."
     config.exclude_labels = %w[duplicate question invalid wontfix wont-fix skip-changelog]
-    config.user = 'voxpupuli'
+    config.user = 'puppetlabs'
     config.project = 'beaker-puppet'
     config.future_release = "#{Gem::Specification.load("#{config.project}.gemspec").version}"
   end

data/acceptance/tests/stub_host.rb

@@ -44,7 +44,7 @@ step 'verify with_stub_host_on' do
     hosts.each do |host|
       hosts_file = get_hosts_file(host)
       result = on host, "cat #{hosts_file}"
-      assert_no_match(/sleepy/, result.stdout)
+      refute_match(/sleepy/, result.stdout)
     end
   end
 end

data/beaker-puppet.gemspec

@@ -11,7 +11,7 @@ Gem::Specification.new do |s|
   s.description = 'For use for the Beaker acceptance testing tool'
   s.license     = 'Apache-2.0'
 
-  s.required_ruby_version = '>= 2.7'
+  s.required_ruby_version = '>= 2.7', '< 3.2'
 
   s.files         = `git ls-files`.split("\n")
   s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
@@ -23,12 +23,12 @@ Gem::Specification.new do |s|
   s.add_development_dependency 'rake', '~> 13.0'
   s.add_development_dependency 'rspec', '~> 3.0'
   s.add_development_dependency 'rspec-its', '~> 1.3'
-  s.add_development_dependency 'voxpupuli-rubocop', '~> 1.2'
+  s.add_development_dependency 'voxpupuli-rubocop', '~> 2.1.0'
 
   # Acceptance Testing Dependencies
   s.add_development_dependency 'beaker-vmpooler', '~> 1.4'
 
   # Run time dependencies
-  s.add_runtime_dependency 'beaker', '~> 4.1'
+  s.add_runtime_dependency 'beaker', '~> 5.0'
   s.add_runtime_dependency 'oga', '~> 3.4'
 end

data/lib/beaker-puppet/helpers/puppet_helpers.rb

@@ -81,21 +81,13 @@ module Beaker
         # Test Puppet running in a certain run mode with specific options.
         # This ensures the following steps are performed:
         # 1. The pre-test Puppet configuration is backed up
-        # 2. A new Puppet configuraton file is layed down
+        # 2. Lay down a new Puppet configuraton file
         # 3. Puppet is started or restarted in the specified run mode
         # 4. Ensure Puppet has started correctly
         # 5. Further tests are yielded to
         # 6. Revert Puppet to the pre-test state
         # 7. Testing artifacts are saved in a folder named for the test
         #
-        # @note Whether Puppet is started or restarted depends on what kind of
-        #   server you're running.  Passenger and puppetserver are restarted before.
-        #   Webrick is started before and stopped after yielding, unless you're using
-        #   service scripts, then it'll behave like passenger & puppetserver.
-        #   Passenger and puppetserver (or webrick using service scripts)
-        #   restart after yielding by default.  You can stop this from happening
-        #   by setting the :restart_when_done flag of the conf_opts argument.
-        #
         # @param [Host] host        One object that act like Host
         #
         # @param [Hash{Symbol=>String}] conf_opts  Represents puppet settings.
@@ -112,22 +104,11 @@ module Beaker
         #
         #                            These will only be applied when starting a FOSS
         #                            master, as a pe master is just bounced.
-        # @option conf_opts [Hash]   :__service_args__  A special setting of options
-        #                            for controlling how the puppet master service is
-        #                            handled. The only setting currently is
-        #                            :bypass_service_script, which if set true will
-        #                            force stopping and starting a webrick master
-        #                            using the start_puppet_from_source_* methods,
-        #                            even if it seems the host has passenger.
-        #                            This is needed in FOSS tests to initialize
-        #                            SSL.
         # @option conf_opts [Boolean] :restart_when_done  determines whether a restart
         #                            should be run after the test has been yielded to.
         #                            Will stop puppet if false. Default behavior
         #                            is to restart, but you can override this on the
         #                            host or with this option.
-        #                            (Note: only works for passenger & puppetserver
-        #                            masters (or webrick using the service scripts))
         # @param [File] testdir      The temporary directory which will hold backup
         #                            configuration, and other test artifacts.
         #
@@ -156,12 +137,11 @@ module Beaker
           end
 
           cmdline_args = conf_opts[:__commandline_args__]
-          service_args = conf_opts[:__service_args__] || {}
           restart_when_done = true
           restart_when_done = host[:restart_when_done] if host.has_key?(:restart_when_done)
           restart_when_done = conf_opts.fetch(:restart_when_done, restart_when_done)
           conf_opts = conf_opts.reject do |k, v|
-            %i[__commandline_args__ __service_args__ restart_when_done].include?(k)
+            %i[__commandline_args__ restart_when_done].include?(k)
           end
 
           curl_retries = host['master-start-curl-retries'] || options['master-start-curl-retries']
@@ -202,13 +182,8 @@ module Beaker
                                           puppet_config(host, 'confdir', section: 'master'),
                                           testdir,
                                           'puppet.conf')
-            lay_down_new_puppet_conf host, conf_opts, testdir
-
-            if host.use_service_scripts? && !service_args[:bypass_service_script]
-              bounce_service(host, host['puppetservice'], curl_retries)
-            else
-              puppet_master_started = start_puppet_from_source_on!(host, cmdline_args)
-            end
+            lay_down_new_puppet_conf(host, conf_opts, testdir)
+            bounce_service(host, host['puppetservice'], curl_retries)
 
             yield self if block_given?
 
@@ -229,20 +204,11 @@ module Beaker
             raise(original_exception)
           ensure
             begin
-              if host.use_service_scripts? && !service_args[:bypass_service_script]
-                restore_puppet_conf_from_backup(host, backup_file)
-                if restart_when_done
-                  bounce_service(host, host['puppetservice'], curl_retries)
-                else
-                  host.exec puppet_resource('service', host['puppetservice'], 'ensure=stopped')
-                end
+              restore_puppet_conf_from_backup(host, backup_file)
+              if restart_when_done
+                bounce_service(host, host['puppetservice'], curl_retries)
               else
-                if puppet_master_started
-                  stop_puppet_from_source_on(host)
-                else
-                  dump_puppet_log(host)
-                end
-                restore_puppet_conf_from_backup(host, backup_file)
+                host.exec puppet_resource('service', host['puppetservice'], 'ensure=stopped')
               end
             rescue Exception => teardown_exception
               begin
@@ -281,19 +247,6 @@ module Beaker
           end
         end
 
-        # @!visibility private
-        def start_puppet_from_source_on!(host, args = '')
-          host.exec(puppet('master', args))
-
-          logger.debug 'Waiting for the puppet master to start'
-          raise Beaker::DSL::FailTest, 'Puppet master did not start in a timely fashion' unless port_open_within?(
-            host, 8140, 10
-          )
-
-          logger.debug 'The puppet master has started'
-          true
-        end
-
         # @!visibility private
         def stop_puppet_from_source_on(host)
           pid = host.exec(Command.new('cat `puppet config print --section master pidfile`')).stdout.chomp
@@ -355,19 +308,12 @@ module Beaker
         def bounce_service(host, service, curl_retries = nil, port = nil)
           curl_retries = 120 if curl_retries.nil?
           port = options[:puppetserver_port] if port.nil?
-          if host.graceful_restarts?
-            service = host.check_for_command('apache2ctl') ? 'apache2ctl' : 'apachectl'
-            apachectl_path = host.is_pe? ? "#{host['puppetsbindir']}/#{service}" : service
-            host.exec(Command.new("#{apachectl_path} graceful"))
-          else
-            result = host.exec(Command.new("service #{service} reload"),
-                               acceptable_exit_codes: [0, 1, 3])
-            return result if result.exit_code == 0
+          result = host.exec(Command.new("service #{service} reload"), acceptable_exit_codes: [0, 1, 3])
+          return result if result.exit_code == 0
 
-            host.exec puppet_resource('service', service, 'ensure=stopped')
-            host.exec puppet_resource('service', service, 'ensure=running')
+          host.exec puppet_resource('service', service, 'ensure=stopped')
+          host.exec puppet_resource('service', service, 'ensure=running')
 
-          end
           curl_with_retries(" #{service} ", host, "https://localhost:#{port}", [35, 60], curl_retries)
         end
 
@@ -819,28 +765,17 @@ module Beaker
         def sign_certificate_for(host = [])
           hostnames = []
           hosts = host.is_a?(Array) ? host : [host]
-          puppet_version = on(master, puppet('--version')).stdout.chomp
           hosts.each do |current_host|
             if [master, dashboard, database].include? current_host
-              on current_host, puppet('agent -t'), acceptable_exit_codes: [0, 1, 2]
-
-              if version_is_less(puppet_version, '5.99')
-                on master, puppet("cert --allow-dns-alt-names sign #{current_host}"), acceptable_exit_codes: [0, 24]
-              else
-                on master, "puppetserver ca sign --certname #{current_host}"
-              end
+              on(current_host, puppet('agent -t'), acceptable_exit_codes: [0, 1, 2])
+              on(master, "puppetserver ca sign --certname #{current_host}")
             else
               hostnames << Regexp.escape(current_host.node_name)
             end
           end
 
           if hostnames.size < 1
-            if version_is_less(puppet_version, '5.99')
-              on master, puppet('cert --sign --all --allow-dns-alt-names'),
-                 acceptable_exit_codes: [0, 24]
-            else
-              on master, 'puppetserver ca sign --all', acceptable_exit_codes: [0, 24]
-            end
+            on(master, 'puppetserver ca sign --all', acceptable_exit_codes: [0, 24])
             return
           end
 
@@ -852,21 +787,11 @@ module Beaker
                 fail_test("Failed to sign cert for #{hostnames}")
                 hostnames.clear
               end
-
-              if version_is_less(puppet_version, '5.99')
-                on master, puppet('cert --sign --all --allow-dns-alt-names'), acceptable_exit_codes: [0, 24]
-                out = on(master, puppet('cert --list --all')).stdout
-                if hostnames.all? { |hostname| out =~ /\+ "?#{hostname}"?/ }
-                  hostnames.clear
-                  break
-                end
-              else
-                on master, 'puppetserver ca sign --all', acceptable_exit_codes: [0, 24]
-                out = on(master, 'puppetserver ca list --all').stdout
-                if out !~ /.*Requested.*/ && hostnames.all? { |hostname| out =~ /\b#{hostname}\b/ }
-                  hostnames.clear
-                  break
-                end
+              on(master, 'puppetserver ca sign --all', acceptable_exit_codes: [0, 24])
+              out = on(master, 'puppetserver ca list --all').stdout
+              if out !~ /.*Requested.*/ && hostnames.all? { |hostname| out =~ /\b#{hostname}\b/ }
+                hostnames.clear
+                break
               end
 
               sleep next_sleep

data/lib/beaker-puppet/install_utils/foss_utils.rb

@@ -98,8 +98,8 @@ module Beaker
             git_protocol = 'https://'
           end
 
-          repo = git_server == 'github.com' ? "#{git_fork}/#{project_name}.git" : "#{git_fork}-#{project_name}.git"
-          git_protocol == 'git@' ? "#{git_protocol}#{git_server}:#{repo}" : "#{git_protocol}#{git_server}/#{repo}"
+          repo = (git_server == 'github.com') ? "#{git_fork}/#{project_name}.git" : "#{git_fork}-#{project_name}.git"
+          (git_protocol == 'git@') ? "#{git_protocol}#{git_server}:#{repo}" : "#{git_protocol}#{git_server}/#{repo}"
         end
         alias build_giturl build_git_url
 
@@ -406,7 +406,7 @@ module Beaker
               end
 
               case host['platform']
-              when /el-|redhat|fedora|sles|centos|cisco_/
+              when /amazon|el-|redhat|fedora|sles|centos|cisco_/
                 package_name = 'puppet-agent'
                 package_name << "-#{opts[:puppet_agent_version]}" if opts[:puppet_agent_version]
               when /debian|ubuntu|huaweios/
@@ -993,7 +993,7 @@ module Beaker
             opts = sanitize_opts(opts)
 
             case variant
-            when /^(fedora|el|redhat|centos|sles|cisco_nexus|cisco_ios_xr)$/
+            when /^(amazon|fedora|el|redhat|centos|sles|cisco_nexus|cisco_ios_xr)$/
               variant_url_value = (%w[redhat centos].include?(::Regexp.last_match(1)) ? 'el' : ::Regexp.last_match(1))
               if variant == 'cisco_nexus'
                 variant_url_value = 'cisco-wrlinux'
@@ -1148,7 +1148,7 @@ module Beaker
                                         repo_configs_dir = nil,
                                         opts = options)
           variant, version, arch, codename = host['platform'].to_array
-          if variant !~ /^(fedora|el|redhat|centos|debian|ubuntu|huaweios|cisco_nexus|cisco_ios_xr|sles)$/
+          if variant !~ /^(amazon|fedora|el|redhat|centos|debian|ubuntu|huaweios|cisco_nexus|cisco_ios_xr|sles)$/
             raise "No repository installation step for #{variant} yet..."
           end
 
@@ -1162,7 +1162,7 @@ module Beaker
           # url type
           _, protocol, hostname = opts[:dev_builds_url].partition %r{.*://}
           dev_builds_url = protocol + hostname
-          dev_builds_url = opts[:dev_builds_url] if variant =~ /^(fedora|el|redhat|centos)$/
+          dev_builds_url = opts[:dev_builds_url] if variant =~ /^(amazon|fedora|el|redhat|centos)$/
 
           install_repo_configs(host, dev_builds_url, package_name,
                                build_version, platform_configs_dir)
@@ -1195,193 +1195,6 @@ module Beaker
           configure_type_defaults_on(host)
         end
 
-        # Install development repo of the puppet-agent on the given host(s).  Downloaded from
-        # location of the form DEV_BUILDS_URL/puppet-agent/AGENT_VERSION/repos
-        #
-        # @param [Host, Array<Host>, String, Symbol] hosts    One or more hosts to act upon,
-        #                            or a role (String or Symbol) that identifies one or more hosts.
-        # @param [Hash{Symbol=>String}] opts An options hash
-        # @option opts [String] :puppet_agent_version The version of puppet-agent to install. This
-        #                       parameter is used by puppet with the +SUITE_VERSION+ environment
-        #                       variable to provide a `git describe` value to beaker to create a
-        #                       build server URL. Note that +puppet_agent_sha+ will still be used
-        #                       instead of this if a value is provided for that option
-        # @option opts [String] :puppet_agent_sha The sha of puppet-agent to install, defaults to provided
-        #                       puppet_agent_version
-        # @option opts [String] :copy_base_local Directory where puppet-agent artifact
-        #                       will be stored locally
-        #                       (default: 'tmp/repo_configs')
-        # @option opts [String] :copy_dir_external Directory where puppet-agent
-        #                       artifact will be pushed to on the external machine
-        #                       (default: '/root')
-        # @option opts [String] :puppet_collection Defaults to 'PC1'
-        # @option opts [String] :dev_builds_url Base URL to pull artifacts from
-        # @option opts [String] :copy_base_local Directory where puppet-agent artifact
-        #                       will be stored locally
-        #                       (default: 'tmp/repo_configs')
-        # @option opts [String] :copy_dir_external Directory where puppet-agent
-        #                       artifact will be pushed to on the external machine
-        #                       (default: '/root')
-        #
-        # @note on windows, the +:ruby_arch+ host parameter can determine in addition
-        # to other settings whether the 32 or 64bit install is used
-        #
-        # @example
-        #   install_puppet_agent_dev_repo_on(host, { :puppet_agent_sha => 'd3377feaeac173aada3a2c2cedd141eb610960a7', :puppet_agent_version => '1.1.1.225.gd3377fe'  })
-        #
-        # @return nil
-        def install_puppet_agent_dev_repo_on(hosts, global_opts)
-          global_opts[:puppet_agent_version] ||= global_opts[:version] # backward compatability
-          unless global_opts[:puppet_agent_version]
-            raise 'must provide :puppet_agent_version (puppet-agent version) for install_puppet_agent_dev_repo_on'
-          end
-
-          block_on hosts do |host|
-            opts = global_opts.dup
-
-            # TODO: consolidate these values as they serve no purpose from beaker's side
-            # you could provide any values you could to one to the other
-            puppet_agent_version = opts[:puppet_agent_sha] || opts[:puppet_agent_version]
-
-            opts = sanitize_opts(opts)
-            opts[:download_url] = "#{opts[:dev_builds_url]}/puppet-agent/#{puppet_agent_version}/repos/"
-            opts[:copy_base_local]    ||= File.join('tmp', 'repo_configs')
-            opts[:puppet_collection]  ||= 'PC1'
-
-            release_path = opts[:download_url]
-
-            variant, version, arch, codename = host['platform'].to_array
-            add_role(host, 'aio') # we are installing agent, so we want aio role
-            copy_dir_local = File.join(opts[:copy_base_local], variant)
-            onhost_copy_base = opts[:copy_dir_external] || host.external_copy_base
-
-            case variant
-            when /^(fedora|el|redhat|centos|debian|ubuntu|huaweios|cisco_nexus|cisco_ios_xr)$/
-              if arch == 's390x' || host['hypervisor'] == 'ec2'
-                logger.trace("#install_puppet_agent_dev_repo_on: unsupported host #{host} for repo detected. using dev package")
-              else
-                install_puppetlabs_dev_repo(host, 'puppet-agent', puppet_agent_version, nil, opts)
-                host.install_package('puppet-agent')
-                logger.trace('#install_puppet_agent_dev_repo_on: install_puppetlabs_dev_repo finished')
-                next
-              end
-            when /^(eos|osx|windows|solaris|sles|aix)$/
-              # Download installer package file & run install manually.
-              # Done below, so that el hosts with s390x arch or on ec2 can use this
-              # workflow as well
-            else
-              raise "No repository installation step for #{variant} yet..."
-            end
-
-            release_path_end, release_file = host.puppet_agent_dev_package_info(
-              opts[:puppet_collection], opts[:puppet_agent_version], opts
-            )
-            release_path << release_path_end
-            logger.trace('#install_puppet_agent_dev_repo_on: dev_package_info, continuing...')
-
-            if variant =~ /eos/
-              host.get_remote_file("#{release_path}/#{release_file}")
-            else
-              onhost_copied_file = File.join(onhost_copy_base, release_file)
-              fetch_http_file(release_path, release_file, copy_dir_local)
-              scp_to host, File.join(copy_dir_local, release_file), onhost_copy_base
-            end
-
-            case variant
-            when /^eos/
-              host.install_from_file(release_file)
-            when /^(sles|aix|fedora|el|redhat|centos)$/
-              # NOTE: AIX does not support repo management. This block assumes
-              # that the desired rpm has been mirrored to the 'repos' location.
-              # NOTE: the AIX 7.1 package will only install on 7.2 with
-              # --ignoreos. This is a bug in package building on AIX 7.1's RPM
-              aix_72_ignoreos_hack = '--ignoreos' if variant == 'aix' && version == '7.2'
-              on host, "rpm -ivh #{aix_72_ignoreos_hack} #{onhost_copied_file}"
-            when /^windows$/
-              result = on host, "echo #{onhost_copied_file}"
-              onhost_copied_file = result.raw_output.chomp
-              msi_opts = { debug: host[:pe_debug] || opts[:pe_debug] }
-              install_msi_on(host, onhost_copied_file, {}, msi_opts)
-            when /^osx$/
-              host.install_package("puppet-agent-#{opts[:puppet_agent_version]}*")
-            when /^solaris$/
-              host.solaris_install_local_package(release_file, onhost_copy_base)
-            end
-            configure_type_defaults_on(host)
-          end
-        end
-        alias install_puppetagent_dev_repo install_puppet_agent_dev_repo_on
-
-        # Install shared repo of the puppet-agent on the given host(s).  Downloaded from
-        # location of the form PE_PROMOTED_BUILDS_URL/PE_VER/puppet-agent/AGENT_VERSION/repo
-        #
-        # @param [Host, Array<Host>, String, Symbol] hosts    One or more hosts to act upon,
-        #                            or a role (String or Symbol) that identifies one or more hosts.
-        # @param [Hash{Symbol=>String}] opts An options hash
-        # @option opts [String] :puppet_agent_version The version of puppet-agent to install, defaults to 'latest'
-        # @option opts [String] :pe_ver The version of PE (will also use host['pe_ver']), defaults to '4.0'
-        # @option opts [String] :copy_base_local Directory where puppet-agent artifact
-        #                       will be stored locally
-        #                       (default: 'tmp/repo_configs')
-        # @option opts [String] :copy_dir_external Directory where puppet-agent
-        #                       artifact will be pushed to on the external machine
-        #                       (default: '/root')
-        # @option opts [String] :puppet_collection Defaults to 'PC1'
-        # @option opts [String] :pe_promoted_builds_url Base URL to pull artifacts from
-        #
-        # @note on windows, the +:ruby_arch+ host parameter can determine in addition
-        #   to other settings whether the 32 or 64bit install is used
-        #
-        # @example
-        #   install_puppet_agent_pe_promoted_repo_on(host, { :puppet_agent_version => '1.1.0.227', :pe_ver => '4.0.0-rc1'})
-        #
-        # @return nil
-        def install_puppet_agent_pe_promoted_repo_on(hosts, opts)
-          opts[:puppet_agent_version] ||= 'latest'
-
-          block_on hosts do |host|
-            pe_ver = host[:pe_ver] || opts[:pe_ver] || '4.0.0-rc1'
-            opts = sanitize_opts(opts)
-            opts[:download_url] =
-              "#{opts[:pe_promoted_builds_url]}/puppet-agent/#{pe_ver}/#{opts[:puppet_agent_version]}/repos"
-            opts[:copy_base_local]    ||= File.join('tmp', 'repo_configs')
-            opts[:copy_dir_external]  ||= host.external_copy_base
-            opts[:puppet_collection] ||= puppet_collection_for(:puppet_agent, opts[:puppet_agent_version])
-            add_role(host, 'aio') # we are installing agent, so we want aio role
-            release_path = opts[:download_url]
-            variant, version, arch, codename = host['platform'].to_array
-            copy_dir_local = File.join(opts[:copy_base_local], variant)
-            onhost_copy_base = opts[:copy_dir_external]
-
-            release_path_end, release_file, download_file =
-              host.pe_puppet_agent_promoted_package_info(
-                opts[:puppet_collection], opts
-              )
-            release_path << release_path_end
-
-            onhost_copied_download = File.join(onhost_copy_base, download_file)
-            onhost_copied_file = File.join(onhost_copy_base, release_file)
-            fetch_http_file(release_path, download_file, copy_dir_local)
-            scp_to host, File.join(copy_dir_local, download_file), onhost_copy_base
-
-            if variant == 'windows'
-              result = on host, "echo #{onhost_copied_file}"
-              onhost_copied_file = result.raw_output.chomp
-              opts = { debug: host[:pe_debug] || opts[:pe_debug] }
-              # couldn't pull this out, because it's relying on
-              # {Beaker::DSL::InstallUtils::WindowsUtils} methods,
-              # which I didn't want to attack right now. TODO
-              install_msi_on(host, onhost_copied_file, {}, opts)
-            else
-              host.pe_puppet_agent_promoted_package_install(
-                onhost_copy_base, onhost_copied_download,
-                onhost_copied_file, download_file, opts
-              )
-            end
-            configure_type_defaults_on(host)
-          end
-        end
-
         # This method will install a pem file certificate on a windows host
         #
         # @param [Host] host                 A host object
@@ -1518,7 +1331,10 @@ module Beaker
             end
 
             # delete any residual files
-            on(host, 'find / -name "*puppet*" -print | xargs rm -rf')
+            result = on(host, 'find / -name "*puppet*" -print | xargs rm -rf', accept_all_exit_codes: true)
+            unless result.exit_code == 0
+              logger.notify("Attempt to clean residual puppet files errored, but can maybe be ignored.\n #{result.stderr}")
+            end
           end
         end
 

data/lib/beaker-puppet/install_utils/puppet_utils.rb

@@ -65,23 +65,6 @@ module Beaker
           end
         end
 
-        # Given an agent_version, return the puppet collection associated with that agent version
-        #
-        # @param [String] agent_version version string or 'latest'
-        # @deprecated This method returns 'PC1' as the latest puppet collection;
-        #     this is incorrect. Use {#puppet_collection_for} instead.
-        def get_puppet_collection(agent_version = 'latest')
-          collection = 'PC1'
-          if agent_version != 'latest'
-            if !version_is_less(agent_version, '5.5.4') and version_is_less(agent_version, '5.99')
-              collection = 'puppet5'
-            elsif !version_is_less(agent_version, '5.99')
-              collection = 'puppet6'
-            end
-          end
-          collection
-        end
-
         # Determine the puppet collection that matches a given package version. The package
         # must be one of
         #   * :puppet_agent (you can get this version from the `aio_agent_version_fact`)

data/lib/beaker-puppet/version.rb

@@ -1,3 +1,3 @@
 module BeakerPuppet
-  VERSION = '2.0.0'
+  VERSION = '3.0.1'
 end

data/release-prep.sh

@@ -0,0 +1,11 @@
+#!/usr/bin/env bash
+
+# Update Gemfile.lock
+docker run -t --rm \
+  -v $(pwd):/app \
+  ruby:3.1.4 \
+  /bin/bash -c 'apt-get update -qq && apt-get install -y --no-install-recommends git make netbase && cd /app && gem install bundler && bundle install --jobs 3; echo "LOCK_FILE_UPDATE_EXIT_CODE=$?"'
+
+docker run -t --rm -e CHANGELOG_GITHUB_TOKEN -v $(pwd):/usr/local/src/your-app \
+  githubchangeloggenerator/github-changelog-generator:1.16.2 \
+  github_changelog_generator --future-release $(grep VERSION lib/beaker-puppet/version.rb |rev |cut -d "'" -f 2 |rev)

data/setup/aio/010_Install_Puppet_Agent.rb

@@ -1,16 +1,30 @@
 test_name 'Install Puppet Agent Packages' do
-  opts = {
-    nightly_builds_url: ENV.fetch('NIGHTLY_BUILDS_URL', nil),
-    dev_builds_url: ENV.fetch('DEV_BUILDS_URL', nil),
-    puppet_agent_version: ENV.fetch('SHA', nil),
-    puppet_collection: ENV.fetch('RELEASE_STREAM', nil),
-  }
+  agents.each do |agent|
+    path = ENV.fetch('DEV_BUILD_PATH', nil)
+    if path
+      raise ArgumentError, "The path #{path} does not exist" unless File.exist?(path)
+
+      basename = File.basename(path)
+      scp_to(agent, path, basename)
 
-  install_puppet_agent_on(hosts, opts)
+      # configure locations for ruby, puppet, config files, etc
+      add_aio_defaults_on(agent)
+      agent.install_package(basename)
+    else
+      opts = {
+        nightly_builds_url: ENV.fetch('NIGHTLY_BUILDS_URL', nil),
+        dev_builds_url: ENV.fetch('DEV_BUILDS_URL', nil),
+        puppet_agent_version: ENV.fetch('SHA', nil),
+        puppet_collection: ENV.fetch('RELEASE_STREAM', nil),
+      }
+
+      install_puppet_agent_on(hosts, opts)
+    end
+  end
 
   # make sure install is sane, beaker has already added puppet and ruby
   # to PATH in ~/.ssh/environment
-  agents.each do |agent|
+  agents.each do |agent| # rubocop:disable Style/CombinableLoops
     on agent, puppet('--version')
     ruby = ruby_command(agent)
     on agent, "#{ruby} --version"

data/setup/common/040_ValidateSignCert.rb

@@ -7,15 +7,9 @@ test_name 'Validate Sign Cert' do
   skip_test 'not testing with puppetserver' unless @options['is_puppetserver']
   hostname = on(master, 'facter hostname').stdout.strip
   fqdn = on(master, 'facter fqdn').stdout.strip
-  puppet_version = on(master, puppet('--version')).stdout.chomp
 
-  if master.use_service_scripts?
-    step 'Ensure puppet is stopped'
-    # Passenger, in particular, must be shutdown for the cert setup steps to work,
-    # but any running puppet master will interfere with webrick starting up and
-    # potentially ignore the puppet.conf changes.
-    on(master, puppet('resource', 'service', master['puppetservice'], 'ensure=stopped'))
-  end
+  step 'Ensure puppet is stopped'
+  on(master, puppet('resource', 'service', master['puppetservice'], 'ensure=stopped'))
 
   step 'Clear SSL on all hosts'
   hosts.each do |host|
@@ -38,11 +32,10 @@ test_name 'Validate Sign Cert' do
       },
     }
 
-    # In Puppet 6, we want to be using an intermediate CA
-    on master, 'puppetserver ca setup' if !version_is_less(puppet_version, '5.99') && !master['use_existing_container']
+    on(master, 'puppetserver ca setup')
     with_puppet_running_on(master, master_opts) do
       step 'Agents: Run agent --test with autosigning enabled to get cert'
-      on agents, puppet('agent --test'), acceptable_exit_codes: [0, 2]
+      on(agents, puppet('agent --test'), acceptable_exit_codes: [0, 2])
     end
   end
 end