beaker-puppet 3.0.1 → 4.1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a3fe57eaf8134ebaf8e7d3059b5cbf927a023b9264fba6bed84ab25c8fc475b8
-  data.tar.gz: b336af52d9d10b87473a51b76471f17f23d9913a59071c76395e413e5096331f
+  metadata.gz: a8afbe94a259d88e99964c542bc6e9c321c073969c05d9ab34b55540ca077118
+  data.tar.gz: 7d257b9d096051e9dec2566172352791eecb410b476003d1d21147185a4bd698
 SHA512:
-  metadata.gz: 93fabb76ba35f1d2af79beb763da85bc9e65bd1fd412214f2f0d898251fd1018630a00ac319b15098b57258ef925ed835a50d83fee98f1d6d25c3e50a83320c8
-  data.tar.gz: caddb56e11d1c2f44996d70bc9b732321ce3aa48d8e492da82b67fca3f2d0b871e4e4015348429db1147b23f912739447a8d7eabf009bbf63c44261fbe470a56
+  metadata.gz: a57b72b62904a6b4bd9042e8d3befd6fe8f2e6edbcf55ca1aacf26969fe8abc3bfedf836a38e02fdd0176d36e64afd85b3a80efac562244a9fdd14efbf33ab37
+  data.tar.gz: a32f96fccd55f6ecb77605a3d20ed51ee7913d807a1bdab9c2274abefafac279fec24486210091cb846907f792575dc1941f323cb9db22ba243af0ba105aba68

data/.github/workflows/release.yml

@@ -29,4 +29,4 @@ jobs:
           echo ":github: Bearer ${{ secrets.GITHUB_TOKEN }}" >> ~/.gem/credentials
           chmod 0600 ~/.gem/credentials
       - name: Publish gem to GitHub packages
-        run: gem push --key github --host https://rubygems.pkg.github.com/voxpupuli *.gem
+        run: gem push --key github --host https://rubygems.pkg.github.com/puppetlabs *.gem

data/CHANGELOG.md

@@ -2,6 +2,23 @@
 
 All notable changes to this project will be documented in this file.
 
+## [4.1.0](https://github.com/puppetlabs/beaker-puppet/tree/4.1.0) (2024-05-31)
+
+[Full Changelog](https://github.com/puppetlabs/beaker-puppet/compare/4.0.0...4.1.0)
+
+**Implemented enhancements:**
+
+- Add support for Ubuntu 24.04 [\#257](https://github.com/puppetlabs/beaker-puppet/pull/257) ([joshcooper](https://github.com/joshcooper))
+
+## [4.0.0](https://github.com/puppetlabs/beaker-puppet/tree/4.0.0) (2024-05-28)
+
+[Full Changelog](https://github.com/puppetlabs/beaker-puppet/compare/3.0.1...4.0.0)
+
+**Breaking changes:**
+
+- Add support for Beaker 6 [\#255](https://github.com/puppetlabs/beaker-puppet/pull/255) ([mhashizume](https://github.com/mhashizume))
+- Remove unsupported network platforms [\#254](https://github.com/puppetlabs/beaker-puppet/pull/254) ([mhashizume](https://github.com/mhashizume))
+
 ## [3.0.1](https://github.com/puppetlabs/beaker-puppet/tree/3.0.1) (2024-03-12)
 
 [Full Changelog](https://github.com/puppetlabs/beaker-puppet/compare/3.0.0...3.0.1)
@@ -50,6 +67,14 @@ All notable changes to this project will be documented in this file.
 - Bump actions/checkout from 3 to 4 [\#227](https://github.com/puppetlabs/beaker-puppet/pull/227) ([dependabot[bot]](https://github.com/apps/dependabot))
 - Update voxpupuli-rubocop requirement from ~\> 1.2 to ~\> 2.0 [\#226](https://github.com/puppetlabs/beaker-puppet/pull/226) ([dependabot[bot]](https://github.com/apps/dependabot))
 
+## [2.1.0](https://github.com/puppetlabs/beaker-puppet/tree/2.1.0) (2024-03-19)
+
+[Full Changelog](https://github.com/puppetlabs/beaker-puppet/compare/2.0.0...2.1.0)
+
+**Implemented enhancements:**
+
+- \[PA-6181\]: Fix acceptance test failure caused due to addition of Amazon Linux 2023 in pe-client-tools for orchestrator client pipeline [\#250](https://github.com/puppetlabs/beaker-puppet/pull/250) ([span786](https://github.com/span786))
+
 ## [2.0.0](https://github.com/puppetlabs/beaker-puppet/tree/2.0.0) (2023-05-05)
 
 [Full Changelog](https://github.com/puppetlabs/beaker-puppet/compare/1.29.0...2.0.0)

data/beaker-puppet.gemspec

@@ -29,6 +29,6 @@ Gem::Specification.new do |s|
   s.add_development_dependency 'beaker-vmpooler', '~> 1.4'
 
   # Run time dependencies
-  s.add_runtime_dependency 'beaker', '~> 5.0'
+  s.add_runtime_dependency 'beaker', '>= 5.0', '< 7'
   s.add_runtime_dependency 'oga', '~> 3.4'
 end

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

@@ -303,7 +303,7 @@ module Beaker
                 family = ::Regexp.last_match(1)
                 relver = ::Regexp.last_match(2)
                 install_puppet_from_rpm_on(host, opts.merge(release: relver, family: family))
-              elsif host['platform'] =~ /(ubuntu|debian|huaweios)/
+              elsif host['platform'] =~ /(ubuntu|debian)/
                 install_puppet_from_deb_on(host, opts)
               elsif host['platform'] =~ /windows/
                 relver = opts[:version]
@@ -406,10 +406,10 @@ module Beaker
               end
 
               case host['platform']
-              when /amazon|el-|redhat|fedora|sles|centos|cisco_/
+              when /amazon|el-|redhat|fedora|sles|centos/
                 package_name = 'puppet-agent'
                 package_name << "-#{opts[:puppet_agent_version]}" if opts[:puppet_agent_version]
-              when /debian|ubuntu|huaweios/
+              when /debian|ubuntu/
                 package_name = 'puppet-agent'
                 if opts[:puppet_agent_version]
                   package_name << "=#{opts[:puppet_agent_version]}-1#{host['platform'].codename}"
@@ -921,9 +921,9 @@ module Beaker
 
             unless host.check_for_command('gem')
               gempkg = case host['platform']
-                       when /solaris-11/                            then 'ruby-18'
-                       when /ubuntu-14/                             then 'ruby'
-                       when /solaris-10|ubuntu|debian|el-|huaweios/ then 'rubygems'
+                       when /solaris-11/                   then 'ruby-18'
+                       when /ubuntu-14/                    then 'ruby'
+                       when /solaris-10|ubuntu|debian|el-/ then 'rubygems'
                        when /openbsd/ then 'ruby'
                        else
                          raise 'install_puppet() called with default_action ' +
@@ -937,7 +937,7 @@ module Beaker
             # Link 'gem' to /usr/bin instead of adding /opt/csw/bin to PATH.
             on host, 'ln -s /opt/csw/bin/gem /usr/bin/gem' if is_solaris10
 
-            if host['platform'] =~ /debian|ubuntu|solaris|huaweios/
+            if host['platform'] =~ /debian|ubuntu|solaris/
               gem_env = YAML.load(on(host, 'gem environment').stdout)
               gem_paths_array = gem_env['RubyGems Environment'].find { |h| h['GEM PATHS'] != nil }['GEM PATHS']
               path_with_gem = 'export PATH=' + gem_paths_array.join(':') + ':${PATH}'
@@ -993,16 +993,8 @@ module Beaker
             opts = sanitize_opts(opts)
 
             case variant
-            when /^(amazon|fedora|el|redhat|centos|sles|cisco_nexus|cisco_ios_xr)$/
+            when /^(amazon|fedora|el|redhat|centos|sles)$/
               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'
-                version = '5'
-              end
-              if variant == 'cisco_ios_xr'
-                variant_url_value = 'cisco-wrlinux'
-                version = '7'
-              end
               if repo_name.match(/puppet\d*/)
                 url = if repo_name.match(/-nightly$/)
                         opts[:nightly_yum_repo_url]
@@ -1028,19 +1020,11 @@ module Beaker
                 end
               end
 
-              if variant == 'cisco_nexus'
-                # cisco nexus requires using yum to install the repo
-                host.install_package(remote)
-              elsif variant == 'cisco_ios_xr'
-                # cisco ios xr requires using yum to localinstall the repo
-                on host, "yum -y localinstall #{remote}"
-              else
-                opts[:package_proxy] ||= false
-                host.install_package_with_rpm(remote, '--replacepkgs',
-                                              { package_proxy: opts[:package_proxy] })
-              end
+              opts[:package_proxy] ||= false
+              host.install_package_with_rpm(remote, '--replacepkgs',
+                                            { package_proxy: opts[:package_proxy] })
 
-            when /^(debian|ubuntu|huaweios)$/
+            when /^(debian|ubuntu)$/
               if repo_name.match(/puppet\d*/)
                 url = if repo_name.match(/-nightly$/)
                         opts[:nightly_apt_repo_url]
@@ -1111,14 +1095,10 @@ module Beaker
             File.write(repo, contents)
           end
 
-          to_path = if host[:platform] =~ /cisco_nexus/
-                      "#{host.package_config_dir}/#{File.basename(repo)}"
-                    else
-                      host.package_config_dir
-                    end
+          to_path = host.package_config_dir
           scp_to(host, repo, to_path)
 
-          on(host, 'apt-get update') if host['platform'] =~ /ubuntu-|debian-|huaweios-/
+          on(host, 'apt-get update') if host['platform'] =~ /ubuntu-|debian-/
           nil
         end
 
@@ -1148,7 +1128,7 @@ module Beaker
                                         repo_configs_dir = nil,
                                         opts = options)
           variant, version, arch, codename = host['platform'].to_array
-          if variant !~ /^(amazon|fedora|el|redhat|centos|debian|ubuntu|huaweios|cisco_nexus|cisco_ios_xr|sles)$/
+          if variant !~ /^(amazon|fedora|el|redhat|centos|debian|ubuntu|sles)$/
             raise "No repository installation step for #{variant} yet..."
           end
 
@@ -1181,7 +1161,7 @@ module Beaker
         # @note This method is paired to be run directly after {#install_puppetlabs_dev_repo}
         #
         def install_packages_from_local_dev_repo(host, package_name)
-          if host['platform'] =~ /debian|ubuntu|huaweios/
+          if host['platform'] =~ /debian|ubuntu/
             find_filename = '*.deb'
             find_command  = 'dpkg -i'
           elsif host['platform'] =~ /fedora|el|redhat|centos/
@@ -1269,7 +1249,7 @@ module Beaker
 
           # We have to do some silly version munging if we're on a deb-based platform
           case host['platform']
-          when /debian|ubuntu|huaweios/
+          when /debian|ubuntu/
             opts[:version] = "#{opts[:version]}-1#{host['platform'].codename}" if opts[:version]
           end
 
@@ -1291,7 +1271,7 @@ module Beaker
             cmdline_args = ''
             # query packages
             case host[:platform]
-            when /huaweios|ubuntu/
+            when /ubuntu/
               pkgs = on(host, "dpkg-query -l  | awk '{print $2}' | grep -E '(^pe-|puppet)'",
                         acceptable_exit_codes: [0, 1]).stdout.chomp.split(/\n+/)
             when /aix|sles|el|redhat|centos|oracle|scientific/

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

@@ -217,7 +217,12 @@ module Beaker
 
           install_targets.each do |host|
             artifact_url, repoconfig_url = host_urls(host, build_details, base_url)
-            if repoconfig_url.nil?
+            if host.platform.variant == 'ubuntu' && host.platform.version.to_f >= 24.04
+              # install the specific artifact we built, not based on how its repos are configured
+              tmp_file = host.tmpfile('puppet-agent')
+              on(host, "curl -L --output #{tmp_file} #{artifact_url}")
+              host.install_local_package(tmp_file)
+            elsif repoconfig_url.nil?
               install_artifact_on(host, artifact_url, project_name)
             else
               install_repo_configs_on(host, repoconfig_url)

data/lib/beaker-puppet/version.rb

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

data/spec/beaker-puppet/install_utils/foss_utils_spec.rb

@@ -818,48 +818,6 @@ describe ClassMixedWithDSLInstallUtils do
         subject.install_puppetlabs_release_repo_on(host)
       end
     end
-
-    context 'on debian 8' do
-      let(:platform) { Beaker::Platform.new('debian-8-i386') }
-      it 'returns the correct url when repo is set to puppet4' do
-        expect(subject).to receive(:on).with(host, /puppet4-release-jessie\.deb$/).once
-        expect(subject).to receive(:on).with(host, 'dpkg -i --force-all /tmp/puppet.deb').once
-        expect(subject).to receive(:on).with(host, 'apt-get update').once
-        subject.install_puppetlabs_release_repo_on(host, 'puppet4')
-      end
-      it 'returns the correct url when opts[:puppet_collection] is set to puppet7' do
-        expect(subject).to receive(:on).with(host, /puppet7-release-jessie\.deb$/).once
-        expect(subject).to receive(:on).with(host, 'dpkg -i --force-all /tmp/puppet.deb').once
-        expect(subject).to receive(:on).with(host, 'apt-get update').once
-        subject.install_puppetlabs_release_repo_on(host, nil, { puppet_collection: 'puppet7' })
-      end
-      it 'returns the correct url when both repo and opts[:puppet_collection] are nil' do
-        expect(subject).to receive(:on).with(host, /puppet-release-jessie\.deb$/).once
-        expect(subject).to receive(:on).with(host, 'dpkg -i --force-all /tmp/puppet.deb').once
-        expect(subject).to receive(:on).with(host, 'apt-get update').once
-        subject.install_puppetlabs_release_repo_on(host)
-      end
-    end
-
-    context 'on cisco platforms' do
-      context 'version 5' do
-        let(:platform) { Beaker::Platform.new('cisco_nexus-7-x86_64') }
-
-        it 'calls host.install_package' do
-          expect(host).to receive(:install_package).with(/\.rpm$/)
-          subject.install_puppetlabs_release_repo_on(host)
-        end
-      end
-
-      context 'version 7' do
-        let(:platform) { Beaker::Platform.new('cisco_ios_xr-6-x86_64') }
-
-        it 'uses yum localinstall to install the package' do
-          expect(subject).to receive(:on).with(host, /^yum.*localinstall.*\.rpm$/)
-          subject.install_puppetlabs_release_repo_on(host)
-        end
-      end
-    end
   end
 
   describe '#install_puppetlabs_dev_repo' do
@@ -884,6 +842,8 @@ describe ClassMixedWithDSLInstallUtils do
         allow(rez).to receive(:exit_code) { 0 }
         allow(subject).to receive(:link_exists?).and_return(true)
         expect(subject).to receive(:scp_to).with(host, repo_config, /.*/).ordered
+        allow(File).to receive(:read).and_call_original
+        allow(File).to receive(:read).with('repoconfig').and_return('')
         subject.install_puppetlabs_dev_repo host, package_name, package_version
       end
     end
@@ -960,7 +920,7 @@ describe ClassMixedWithDSLInstallUtils do
     end
 
     it 'fails correctly for systems not accounted for' do
-      @platform = 'eos-1-3'
+      @platform = 'freebsd-1-3'
       expect { subject.install_packages_from_local_dev_repo(host, package_name) }.to raise_error RuntimeError
     end
   end
@@ -1108,10 +1068,10 @@ describe ClassMixedWithDSLInstallUtils do
       version = '6.6.6'
 
       context 'on deb-based platform' do
-        let(:host) { make_host('master', platform: Beaker::Platform.new('ubuntu-16.04-amd64')) }
-        it 'munges the version on ubuntu 16.04' do
+        let(:host) { make_host('master', platform: Beaker::Platform.new('ubuntu-24.04-amd64')) }
+        it 'munges the version on ubuntu 24.04' do
           expect(subject).to receive(:install_puppetlabs_release_repo_on).with(host, 'puppet', anything)
-          expect(subject).to receive(:install_package).with(host, 'puppetserver', "#{version}-1xenial")
+          expect(subject).to receive(:install_package).with(host, 'puppetserver', "#{version}-1noble")
           allow(subject).to receive(:dev_builds_accessible_on?).with(host, anything).and_return false
           subject.install_puppetserver_on(host, version: version)
         end

data/spec/beaker-puppet/install_utils/puppet5_spec.rb

@@ -12,7 +12,7 @@ end
 describe ClassMixedWithDSLInstallUtils do
   let(:hosts) do
     make_hosts({ pe_ver: '3.0',
-                 platform: 'linux',
+                 platform: Beaker::Platform.new('redhat-9-x86_64'),
                  roles: ['agent'],
                  type: 'foss', }, 4)
   end
@@ -363,6 +363,23 @@ describe ClassMixedWithDSLInstallUtils do
       end.to raise_error(Beaker::DSL::Outcomes::FailTest, /project_name.*#{sha_yaml_url}/)
     end
 
+    it 'installs the artifact on newer Ubuntu hosts' do
+      artifact_url = 'https://builds.example.com/puppet-agent.deb'
+      project_name = 'fake_project_66'
+      allow(subject).to receive(:fetch_build_details)
+      allow(subject).to receive(:configure_type_defaults_on)
+      allow(subject).to receive(:host_urls) { [artifact_url, ''] }
+
+      hosts = [make_host('host.example.com', platform: Beaker::Platform.new('ubuntu-24.04-x86_64'))]
+      allow(subject).to receive(:hosts).and_return(hosts)
+      hosts.each do |host|
+        allow(host).to receive(:tmpfile).and_return('/tmp/puppet-agent.GcHvLR')
+        allow(subject).to receive(:on).with(host, /^curl/)
+        expect(host).to receive(:install_local_package).with('/tmp/puppet-agent.GcHvLR')
+      end
+      subject.install_from_build_data_url(project_name, 'sha_yaml_url')
+    end
+
     it 'runs host.install_package instead of #install_artifact_on if theres a repo_config' do
       repoconfig_url = 'pants/man/shoot/to/the/stars'
       project_name = 'fake_project_66'

data/spec/helpers.rb

@@ -120,9 +120,7 @@ module HostHelpers
 end
 
 module PlatformHelpers
-  DEBIANPLATFORMS = %w[debian
-                       ubuntu
-                       huaweios]
+  DEBIANPLATFORMS = %w[debian ubuntu]
 
   FEDORASYSTEMD    = (14..29).to_a.collect! { |i| "fedora-#{i}" }
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: beaker-puppet
 version: !ruby/object:Gem::Version
-  version: 3.0.1
+  version: 4.1.0
 platform: ruby
 authors:
 - Vox Pupuli
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-03-12 00:00:00.000000000 Z
+date: 2024-06-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: fakefs
@@ -104,16 +104,22 @@ dependencies:
   name: beaker
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '5.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '7'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '5.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '7'
 - !ruby/object:Gem::Dependency
   name: oga
   requirement: !ruby/object:Gem::Requirement