beaker-puppet 1.22.1 → 1.24.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 13f2e03144c47918054b8cb3b3f470df71761a7b06c513f5c0455a541107c0ec
-  data.tar.gz: f03a957fb46bce9f5645640e738534f3bef4eddc84d30a722645a2f713378513
+  metadata.gz: 5557cad6a494d24bd383c9583db5ce43546677ede516270f1c8c390a2951951b
+  data.tar.gz: c0d92915ae1b5bcc059e4728a2699d3eac446de1c141fdc5487a553e34e3e89e
 SHA512:
-  metadata.gz: f7853cf2cc0836170a6c0642404e647e6adac07ad315bc9199848aa0c6cc86e6dab010dadee890fb4b8d7050fbddac15e68b00c7a06b05ba9861e547043d729e
-  data.tar.gz: 0be933ece5a86492d4c348564cf634376ec51b71fe46f318c4c187e91132674f46a33d7637ed4e7626162164fba3c31cb3f61999af6b232e3df0bd39e9cddd39
+  metadata.gz: 81f04cba47d505b66eed94d1fdd9fb80bbe1d11d4d18b55edf238815405d8d030b0e6506ad44af2f523664f7d0a468fc635ec9a6cda99fc1b653962c1d30f551
+  data.tar.gz: 5e691ffcaf6d56e408ee6be5acf025d1ae5605d1c5081fb9d095c948aa1968a9d67172d311932407567f6cd3b534b8267f19822606c3195d4bac17daa03af2bc

data/.github/workflows/test.yml

@@ -20,6 +20,8 @@ jobs:
           - ruby: "2.7"
           - ruby: "3.0"
             coverage: "yes"
+          - ruby: "3.1"
+
     env:
       COVERAGE: ${{ matrix.coverage }}
     name: RSpec - Ruby ${{ matrix.ruby }}

data/CHANGELOG.md

@@ -2,6 +2,41 @@
 
 All notable changes to this project will be documented in this file.
 
+## [1.24.0](https://github.com/voxpupuli/beaker-puppet/tree/1.24.0) (2022-03-04)
+
+[Full Changelog](https://github.com/voxpupuli/beaker-puppet/compare/1.23.0...1.24.0)
+
+**Implemented enhancements:**
+
+- module setup on the target: Igrore files at the root level only [\#177](https://github.com/voxpupuli/beaker-puppet/pull/177) ([smortex](https://github.com/smortex))
+
+**Merged pull requests:**
+
+- gemspec: fix typo in Apache-2.0 license [\#180](https://github.com/voxpupuli/beaker-puppet/pull/180) ([bastelfreak](https://github.com/bastelfreak))
+- Add Ruby 3.1 to CI [\#179](https://github.com/voxpupuli/beaker-puppet/pull/179) ([bastelfreak](https://github.com/bastelfreak))
+- Depend on Ruby 2.4 or newer in gemspec [\#178](https://github.com/voxpupuli/beaker-puppet/pull/178) ([bastelfreak](https://github.com/bastelfreak))
+
+## [1.23.0](https://github.com/voxpupuli/beaker-puppet/tree/1.23.0) (2022-02-23)
+
+[Full Changelog](https://github.com/voxpupuli/beaker-puppet/compare/1.22.2...1.23.0)
+
+**Implemented enhancements:**
+
+- \(maint\) Follow redirects when curling MSI [\#176](https://github.com/voxpupuli/beaker-puppet/pull/176) ([joshcooper](https://github.com/joshcooper))
+- Update agent download from URL logic to handle redirects [\#173](https://github.com/voxpupuli/beaker-puppet/pull/173) ([cthorn42](https://github.com/cthorn42))
+
+**Merged pull requests:**
+
+- Update specs to fix ruby 3 spec failures [\#174](https://github.com/voxpupuli/beaker-puppet/pull/174) ([cthorn42](https://github.com/cthorn42))
+
+## [1.22.2](https://github.com/voxpupuli/beaker-puppet/tree/1.22.2) (2021-12-31)
+
+[Full Changelog](https://github.com/voxpupuli/beaker-puppet/compare/1.22.1...1.22.2)
+
+**Merged pull requests:**
+
+- Fix install\_puppet\_on on el8 [\#170](https://github.com/voxpupuli/beaker-puppet/pull/170) ([gcampbell12](https://github.com/gcampbell12))
+
 ## [1.22.1](https://github.com/voxpupuli/beaker-puppet/tree/1.22.1) (2021-08-19)
 
 [Full Changelog](https://github.com/voxpupuli/beaker-puppet/compare/1.22.0...1.22.1)

data/beaker-puppet.gemspec

@@ -10,7 +10,9 @@ Gem::Specification.new do |s|
   s.homepage    = "https://github.com/voxpupuli/beaker-puppet"
   s.summary     = %q{Beaker's Puppet DSL Extension Helpers!}
   s.description = %q{For use for the Beaker acceptance testing tool}
-  s.license     = 'Apache2'
+  s.license     = 'Apache-2.0'
+
+  s.required_ruby_version = '>= 2.4'
 
   s.files         = `git ls-files`.split("\n")
   s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")

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

@@ -75,7 +75,7 @@ module Beaker
           # redhat-8-arm64 is provided from amazon
           return true if host.host_hash[:template] == 'redhat-8-arm64' && host.hostname =~ /.puppet.net$/
 
-          result = on(host, %(curl -fI "#{url}"), accept_all_exit_codes: true)
+          result = on(host, %(curl --location -fI "#{url}"), accept_all_exit_codes: true)
           return result.exit_code.zero?
         end
 
@@ -303,12 +303,10 @@ module Beaker
             # run_in_parallel option includes install
             run_in_parallel = run_in_parallel? opts, @options, 'install'
             block_on hosts, { :run_in_parallel => run_in_parallel } do |host|
-              if host['platform'] =~ /el-(5|6|7)/
-                relver = $1
-                install_puppet_from_rpm_on(host, opts.merge(:release => relver, :family => 'el'))
-              elsif host['platform'] =~ /fedora-(\d+)/
-                relver = $1
-                install_puppet_from_rpm_on(host, opts.merge(:release => relver, :family => 'fedora'))
+              if host['platform'] =~ /(el|fedora)-(\d+)/
+                family = $1
+                relver = $2
+                install_puppet_from_rpm_on(host, opts.merge(:release => relver, :family => family))
               elsif host['platform'] =~ /(ubuntu|debian|cumulus|huaweios)/
                 install_puppet_from_deb_on(host, opts)
               elsif host['platform'] =~ /windows/
@@ -683,7 +681,7 @@ module Beaker
             if host.is_cygwin?
               # NOTE: it is critical that -o be before -O on Windows
               proxy = opts[:package_proxy] ? "-x #{opts[:package_proxy]} " : ''
-              on host, "curl #{proxy}-o \"#{msi_download_path}\" -O #{link}"
+              on host, "curl #{proxy}--location --output \"#{msi_download_path}\" --remote-name #{link}"
 
               #Because the msi installer doesn't add Puppet to the environment path
               #Add both potential paths for simplicity
@@ -763,9 +761,9 @@ module Beaker
                 raise "You need to specify versions for OSX host\n eg. install_puppet({:version => '3.6.2',:facter_version => '2.1.0',:hiera_version  => '1.3.4',})"
               end
 
-              on host, "curl -O #{opts[:mac_download_url]}/puppet-#{puppet_ver}.dmg"
-              on host, "curl -O #{opts[:mac_download_url]}/facter-#{facter_ver}.dmg"
-              on host, "curl -O #{opts[:mac_download_url]}/hiera-#{hiera_ver}.dmg"
+              on host, "curl --location --remote-name #{opts[:mac_download_url]}/puppet-#{puppet_ver}.dmg"
+              on host, "curl --location --remote-name #{opts[:mac_download_url]}/facter-#{facter_ver}.dmg"
+              on host, "curl --location --remote-name #{opts[:mac_download_url]}/hiera-#{hiera_ver}.dmg"
 
               host.install_package("puppet-#{puppet_ver}")
               host.install_package("facter-#{facter_ver}")
@@ -817,7 +815,7 @@ module Beaker
 
             pkg_name = "puppet-agent-#{agent_version}*"
             dmg_name = "puppet-agent-#{agent_version}-1.osx#{version}.dmg"
-            on host, "curl -O #{download_url}/#{dmg_name}"
+            on host, "curl --location --remote-name #{download_url}/#{dmg_name}"
 
             host.install_package(pkg_name)
 
@@ -828,7 +826,7 @@ module Beaker
         # Returns the latest puppet-agent version number from a given url.
         #
         # @param [String] url         URL containing list of puppet-agent packages.
-        #                             Example: https://downloads.puppetlabs.com/mac/10.11/PC1/x86_64
+        #                             Example: https://downloads.puppetlabs.com/mac/puppet7/10.15/x86_64/
         #
         # @return [String] version    puppet-agent version number (e.g. 1.4.1)
         #                             Empty string if none found.
@@ -836,8 +834,20 @@ module Beaker
         def get_latest_puppet_agent_build_from_url(url)
           require 'oga'
           require 'net/http'
-          body = Net::HTTP.get(URI.parse("#{url}/index_by_lastModified_reverse.html"))
-          document = Oga.parse_html(body)
+
+          full_url = "#{url}/index_by_lastModified_reverse.html"
+          response = Net::HTTP.get_response(URI(full_url))
+          counter = 0
+
+          # Redirect following
+          while response.is_a?(Net::HTTPRedirection) && counter < 15
+            response = Net::HTTP.get_response(URI.parse(Net::HTTP.get_response(URI(full_url))['location']))
+            counter = counter + 1
+          end
+
+          raise "The URL for puppet-agent download, #{response.uri}, returned #{response.message} with #{response.code}" unless response.is_a?(Net::HTTPSuccess)
+
+          document = Oga.parse_html(response.body)
           agents = document.xpath('//a[contains(@href, "puppet-agent")]')
 
           latest_match = agents.shift.attributes[0].value

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

@@ -15,8 +15,8 @@ module Beaker
 
         # The directories in the module directory that will not be scp-ed to the test system when using
         # `copy_module_to`
-        PUPPET_MODULE_INSTALL_IGNORE = ['.bundle', '.git', '.idea', '.vagrant', '.vendor', 'vendor', 'acceptance',
-                                        'bundle', 'spec', 'tests', 'log', '.svn', 'junit', 'pkg', 'example', 'tmp']
+        PUPPET_MODULE_INSTALL_IGNORE = ['/.bundle', '/.git', '/.idea', '/.vagrant', '/.vendor', '/vendor', '/acceptance',
+                                        '/bundle', '/spec', '/tests', '/log', '/.svn', '/junit', '/pkg', '/example', '/tmp']
 
         # Install the desired module on all hosts using either the PMT or a
         #   staging forge

data/lib/beaker-puppet/version.rb

@@ -1,3 +1,3 @@
 module BeakerPuppet
-  VERSION = '1.22.1'
+  VERSION = '1.24.0'
 end

data/spec/beaker-puppet/helpers/puppet_helpers_spec.rb

@@ -174,7 +174,7 @@ describe ClassMixedWithDSLHelpers do
 
       expect( subject ).to receive( :on ).
         with( agent, 'puppet_command',
-              :acceptable_exit_codes => [0] )
+              {:acceptable_exit_codes => [0]} )
 
       subject.apply_manifest_on( agent, 'class { "boo": }')
     end
@@ -189,7 +189,7 @@ describe ClassMixedWithDSLHelpers do
           and_return( 'puppet_command' )
 
         expect( subject ).to receive( :on ).
-          with( host, 'puppet_command', :acceptable_exit_codes => [0] )
+          with( host, 'puppet_command', {:acceptable_exit_codes => [0]} )
       end
 
       result = subject.apply_manifest_on( the_hosts, 'include foobar' )
@@ -209,7 +209,7 @@ describe ClassMixedWithDSLHelpers do
         and_return( 'puppet_command' )
       the_hosts.each do |host|
         allow( subject ).to receive( :on ).
-            with( host, 'puppet_command', :acceptable_exit_codes => [0] )
+          with( host, 'puppet_command', {:acceptable_exit_codes => [0]} )
       end
 
       result = nil
@@ -228,11 +228,11 @@ describe ClassMixedWithDSLHelpers do
         and_return( 'puppet_command' )
       the_hosts.each do |host|
         allow( subject ).to receive( :on ).
-          with( host, 'puppet_command', :acceptable_exit_codes => [0] )
+          with( host, 'puppet_command', {:acceptable_exit_codes => [0]} )
       end
       expect( subject ).to receive( :block_on ).with(
         anything,
-        :run_in_parallel => true
+        {:run_in_parallel => true}
       )
 
       subject.apply_manifest_on( the_hosts, 'include foobar', { :run_in_parallel => true } )
@@ -246,11 +246,11 @@ describe ClassMixedWithDSLHelpers do
 
       expect( subject ).to receive( :on ).
         with( agent, 'puppet_command',
-              :acceptable_exit_codes => [0,2] )
+              {:acceptable_exit_codes => [0,2]} )
 
       subject.apply_manifest_on( agent,
                                 'class { "boo": }',
-                                :catch_failures => true )
+                                {:catch_failures => true} )
     end
     it 'allows acceptable exit codes through :catch_failures' do
       allow( subject ).to receive( :hosts ).and_return( hosts )
@@ -260,12 +260,12 @@ describe ClassMixedWithDSLHelpers do
 
       expect( subject ).to receive( :on ).
         with( agent, 'puppet_command',
-              :acceptable_exit_codes => [4,0,2] )
+             {:acceptable_exit_codes => [4,0,2]} )
 
       subject.apply_manifest_on( agent,
                                 'class { "boo": }',
-                                :acceptable_exit_codes => [4],
-                                :catch_failures => true )
+                                {:acceptable_exit_codes => [4],
+                                 :catch_failures => true} )
     end
     it 'enforces a 0 exit code through :catch_changes' do
       allow( subject ).to receive( :hosts ).and_return( hosts )
@@ -276,13 +276,13 @@ describe ClassMixedWithDSLHelpers do
       expect( subject ).to receive( :on ).with(
         agent,
         'puppet_command',
-        :acceptable_exit_codes => [0]
+        {:acceptable_exit_codes => [0]}
       )
 
       subject.apply_manifest_on(
         agent,
         'class { "boo": }',
-        :catch_changes => true
+        {:catch_changes => true}
       )
     end
     it 'enforces a 2 exit code through :expect_changes' do
@@ -294,13 +294,13 @@ describe ClassMixedWithDSLHelpers do
       expect( subject ).to receive( :on ).with(
         agent,
         'puppet_command',
-        :acceptable_exit_codes => [2]
+        {:acceptable_exit_codes => [2]}
       )
 
       subject.apply_manifest_on(
         agent,
         'class { "boo": }',
-        :expect_changes => true
+        {:expect_changes => true}
       )
     end
     it 'enforces exit codes through :expect_failures' do
@@ -312,13 +312,13 @@ describe ClassMixedWithDSLHelpers do
       expect( subject ).to receive( :on ).with(
         agent,
         'puppet_command',
-        :acceptable_exit_codes => [1,4,6]
+        {:acceptable_exit_codes => [1,4,6]}
       )
 
       subject.apply_manifest_on(
         agent,
         'class { "boo": }',
-        :expect_failures => true
+        {:expect_failures => true}
       )
     end
     it 'enforces exit codes through :expect_failures' do
@@ -341,14 +341,14 @@ describe ClassMixedWithDSLHelpers do
       expect( subject ).to receive( :on ).with(
         agent,
         'puppet_command',
-        :acceptable_exit_codes => [1,2,3,4,5,6]
+        {:acceptable_exit_codes => [1,2,3,4,5,6]}
       )
 
       subject.apply_manifest_on(
         agent,
         'class { "boo": }',
-        :acceptable_exit_codes => (1..5),
-        :expect_failures       => true
+        {:acceptable_exit_codes => (1..5),
+         :expect_failures       => true}
       )
     end
 

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

@@ -341,7 +341,7 @@ describe ClassMixedWithDSLInstallUtils do
 
     it 'installs puppet on cygwin windows' do
       allow(subject).to receive(:link_exists?).and_return( true )
-      expect(subject).to receive(:on).with(winhost, "curl -o \"#{win_temp}\\puppet-3.7.1-x64.msi\" -O http://downloads.puppet.com/windows/puppet-3.7.1-x64.msi")
+      expect(subject).to receive(:on).with(winhost, "curl --location --output \"#{win_temp}\\puppet-3.7.1-x64.msi\" --remote-name http://downloads.puppet.com/windows/puppet-3.7.1-x64.msi")
       expect(subject).to receive(:on).with(winhost, " echo 'export PATH=$PATH:\"/cygdrive/c/Program Files (x86)/Puppet Labs/Puppet/bin\":\"/cygdrive/c/Program Files/Puppet Labs/Puppet/bin\"' > /etc/bash.bashrc ")
       expect(subject).to receive(:install_msi_on).with(winhost, "#{win_temp}\\puppet-3.7.1-x64.msi", {}, {:debug => nil})
 
@@ -350,7 +350,7 @@ describe ClassMixedWithDSLInstallUtils do
 
     it 'installs puppet on cygwin windows via proxy' do
       allow(subject).to receive(:link_exists?).and_return( true )
-      expect(subject).to receive(:on).with(winhost, "curl -x https://proxy.com -o \"#{win_temp}\\puppet-3.7.1-x64.msi\" -O http://downloads.puppet.com/windows/puppet-3.7.1-x64.msi")
+      expect(subject).to receive(:on).with(winhost, "curl -x https://proxy.com --location --output \"#{win_temp}\\puppet-3.7.1-x64.msi\" --remote-name http://downloads.puppet.com/windows/puppet-3.7.1-x64.msi")
       expect(subject).to receive(:on).with(winhost, " echo 'export PATH=$PATH:\"/cygdrive/c/Program Files (x86)/Puppet Labs/Puppet/bin\":\"/cygdrive/c/Program Files/Puppet Labs/Puppet/bin\"' > /etc/bash.bashrc ")
       expect(subject).to receive(:install_msi_on).with(winhost, "#{win_temp}\\puppet-3.7.1-x64.msi", {}, {:debug => nil})
 

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

@@ -99,7 +99,7 @@ describe ClassMixedWithDSLInstallUtils do
       expect_version_log_called
       expect( subject ).to receive( :create_install_msi_batch_on ).with(
           anything, anything,
-          'PUPPET_AGENT_STARTUP_MODE' => 'Manual')
+          {'PUPPET_AGENT_STARTUP_MODE' => 'Manual'})
       subject.install_msi_on(hosts, msi_path, {})
     end
 
@@ -112,7 +112,7 @@ describe ClassMixedWithDSLInstallUtils do
       value = 'Automatic'
       expect( subject ).to receive( :create_install_msi_batch_on ).with(
           anything, anything,
-          'PUPPET_AGENT_STARTUP_MODE' => value)
+          {'PUPPET_AGENT_STARTUP_MODE' => value})
       subject.install_msi_on(hosts, msi_path, {'PUPPET_AGENT_STARTUP_MODE' => value})
     end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: beaker-puppet
 version: !ruby/object:Gem::Version
-  version: 1.22.1
+  version: 1.24.0
 platform: ruby
 authors:
 - Vox Pupuli
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2021-08-19 00:00:00.000000000 Z
+date: 2022-03-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
@@ -251,7 +251,7 @@ files:
 - tasks/ci.rake
 homepage: https://github.com/voxpupuli/beaker-puppet
 licenses:
-- Apache2
+- Apache-2.0
 metadata: {}
 post_install_message: 
 rdoc_options: []
@@ -261,14 +261,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: '2.4'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.22
+rubygems_version: 3.2.32
 signing_key: 
 specification_version: 4
 summary: Beaker's Puppet DSL Extension Helpers!