apt_stage_artifacts 0.9.0 → 0.11.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e22e83a8811e9598f4dc9fd27b639c743451fdc06e0502092daceb5cea04a9f8
-  data.tar.gz: 9d19f393ac7f403a8dbdd7e56e6888e6392d556a8da28c9b5190eefc0c0a6098
+  metadata.gz: aab821806d4c6d510c44ab92fb338d00480aeca0bd9739cc4fd399d232d7da3b
+  data.tar.gz: f80728f81a04388c1656d40ff2efc1d636b1211c950885ae760d86e7bc2a77a5
 SHA512:
-  metadata.gz: ae8a11f11b2a4698b684c169d74943770945b9e1f04489c4eaf8a1e3bf28cdfb64d5fdd3629c5febb6eba5f84e4bc5a8e9dd9e3ad1cbab6f659b62eeeabf5a8b
-  data.tar.gz: dcc70849ba88e635c3fbf8d3022f74d0e136fab374f0f06d166208601479858b834765b6833265e7583638f6c0ae0245e8061a96d02ef48a76104896115df1c7
+  metadata.gz: d76bdf2c262e71b58b8b7dd46fef2fbc349e48d7172007ec2989d795f49df582a85dec43e30c2e5de6f8b76d48c7fcb4984e6d9e6b80f9c7f3b3c2022ce6c5c1
+  data.tar.gz: ed1f6177067eed7aaa6391a6b666e30753d9e60eb580b1e2e22302875f9a57cbbd325b18179d91a152fe0b3dcf29ab2dbd9d9218da5860dc8c9bca9b3636dd82

data/.rubocop.yml

@@ -3,150 +3,49 @@
 require: rubocop-rake
 
 AllCops:
+  TargetRubyVersion: 2.3
+
   Exclude:
   - Rakefile
   - Gemfile
   - bin/*
   - spec/**/*.rb
+  - '*.gemspec'
   - templates/*
 
-Gemspec/DateAssignment:
-  Enabled: true
-
-Layout/FirstArrayElementIndentation:
+Style/FrozenStringLiteralComment:
   Enabled: false
 
-Layout/MultilineMethodCallIndentation:
+Style/MutableConstant:
   Enabled: false
 
-Layout/SpaceBeforeBrackets:
-  Enabled: true
-
-Lint/AmbiguousAssignment:
-  Enabled: true
-
-Lint/DeprecatedConstants:
-  Enabled: true
-
-Lint/DuplicateBranch:
-  Enabled: true
-
-Lint/DuplicateRegexpCharacterClassElement:
-  Enabled: true
-
-Lint/EmptyBlock:
-  Enabled: true
-
-Lint/EmptyClass:
-  Enabled: true
-
-Lint/LambdaWithoutLiteralBlock:
-  Enabled: true
-
-Lint/NoReturnInBeginEndBlocks:
-  Enabled: true
-
-Lint/NumberedParameterAssignment:
-  Enabled: true
-
-Lint/OrAssignmentToConstant:
-  Enabled: true
-
-Lint/RedundantDirGlobSort:
-  Enabled: true
+Layout/LineLength:
+  Max: 100
 
-Lint/SymbolConversion:
-  Enabled: true
-
-Lint/ToEnumArguments:
-  Enabled: true
-
-Lint/TripleQuotes:
-  Enabled: true
-
-Lint/UnexpectedBlockArity:
-  Enabled: true
-
-Lint/UnmodifiedReduceAccumulator:
-  Enabled: true
-
-Metrics/AbcSize:
-  Enabled: false
-
-Metrics/ClassLength:
+Style/Documentation:
   Enabled: false
 
-Metrics/CyclomaticComplexity:
+Metrics/AbcSize:
   Enabled: false
 
 Metrics/MethodLength:
   Enabled: false
 
-Metrics/ModuleLength:
-  Enabled: false
-
-Metrics/PerceivedComplexity:
-  Enabled: false
-
-Style/ArgumentsForwarding:
-  Enabled: true
-
-Style/CollectionCompact:
-  Enabled: true
-
-Style/DocumentDynamicEvalDefinition:
-  Enabled: true
-
-Style/Documentation:
-  Enabled: false
-
-Style/EachWithObject:
-  Enabled: false
-
-Style/EndlessMethod:
-  Enabled: true
-
-Style/FormatString:
-  Enabled: false
+Style/CommandLiteral:
+  EnforcedStyle: percent_x
 
-Style/FormatStringToken:
+Metrics/ClassLength:
   Enabled: false
 
-Style/FrozenStringLiteralComment:
+Style/RedundantReturn:
   Enabled: false
 
-Style/HashConversion:
-  Enabled: true
-
-Style/HashExcept:
-  Enabled: true
+Style/PercentLiteralDelimiters:
+  PreferredDelimiters:
+    default: '()'
 
 Style/IfUnlessModifier:
   Enabled: false
 
-Style/IfWithBooleanLiteralBranches:
-  Enabled: true
-
-Style/MutableConstant:
-  Enabled: false
-
-Style/NegatedIfElseCondition:
-  Enabled: true
-
-Style/NilLambda:
-  Enabled: true
-
-Style/RedundantArgument:
-  Enabled: true
-
-Style/RedundantReturn:
-  Enabled: false
-
-Style/StringChars:
-  Enabled: true
-
-Style/SwapValues:
-  Enabled: true
-
-Style/CommandLiteral:
+Metrics/CyclomaticComplexity:
   Enabled: false

data/apt_stage_artifacts.gemspec

@@ -8,7 +8,7 @@ Gem::Specification.new do |spec|
 
   spec.summary       = 'Stages .deb artifacts to a remote freight repository'
   spec.homepage      = 'https://github.com/puppetlabs/apt_stage_artifacts'
-  spec.required_ruby_version = Gem::Requirement.new('>= 2.5.0')
+  spec.required_ruby_version = Gem::Requirement.new('>= 2.3.0')
 
   spec.metadata['allowed_push_host'] = 'https://rubygems.org'
 
@@ -29,6 +29,10 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'pry-byebug', '~> 3.0'
   spec.add_development_dependency 'rake', '~> 13.0'
   spec.add_development_dependency 'rspec', '~> 3.0'
+
+  # Rubocop 0.81 as the last to support Ruby 2.3. Drop this whenever we can stop
+  # supporting older rubies
+  spec.add_development_dependency 'rubocop', '0.81'
   spec.add_development_dependency 'rubocop-rake'
   spec.add_development_dependency 'rubocop-rspec'
   spec.add_development_dependency 'yard', '~> 0.9'

data/lib/apt_add_to_freight_library.rb

@@ -94,7 +94,7 @@ class AptAddToFreightLibrary
       AptStageArtifacts::FREIGHT_CONFIG_DIRECTORY,
       "puppet#{@puppet_version}.conf"
     )
-    fatal "Cannot read '#{freight_config_file} on #{`hostname`.chomp}." unless
+    fatal "Cannot read '#{freight_config_file} on #{%x(hostname).chomp}." unless
       File.readable?(freight_config_file)
 
     logger.info "Using '#{freight_config_file}'"
@@ -115,9 +115,9 @@ class AptAddToFreightLibrary
         end
 
         freight_add_command = %W[
-         sudo --user=jenkins --set-home
-         #{AptStageArtifacts::FREIGHT_COMMAND} add #{@verbose} --conf="#{freight_config_file}"
-         #{deb_artifact_path} "apt/#{@codename}/#{@apt_component}"
+          sudo --user=jenkins --set-home
+          #{AptStageArtifacts::FREIGHT_COMMAND} add #{@verbose} --conf="#{freight_config_file}"
+          #{deb_artifact_path} "apt/#{@codename}/#{@apt_component}"
         ].join(' ')
 
         %x(#{freight_add_command})

data/lib/apt_stage_artifacts.rb

@@ -77,13 +77,16 @@ class AptStageArtifacts
       if File.directory?(File.join(@top_directory, 'deb'))
         @top_directory = File.join(@top_directory, 'deb')
       end
+      unless File.directory?(@top_directory)
+        fatal "Artifact directory \"#{@top_directory}\" does not exist"
+      end
     elsif File.directory?('./output/deb')
       @top_directory = './output/deb'
     elsif File.directory?('./pkg/deb')
       @top_directory = './pkg/deb'
     else
-      logger.error 'Cannot find .deb artifacts. Neither pkg/deb nor output/deb exist.'
-      exit 1
+      logger.warn 'Cannot find .deb artifacts. Neither pkg/deb nor output/deb exist. Skipping.'
+      exit 0
     end
 
     logger.info "Searching for puppetN/*.deb packages in #{@top_directory}"
@@ -92,9 +95,9 @@ class AptStageArtifacts
     end
     return unless @debian_artifacts.empty?
 
-    logger.warn "This project does not contain artifacts suitable for shipping to the " \
-                "APT server. Expected repo layout is 'pkg/<debian codename>/puppet<N>/*.deb'. " \
-                "Skipping."
+    logger.warn 'This project does not contain artifacts suitable for shipping to the ' \
+                'APT server. Expected repo layout is "pkg/<debian codename>/puppet<N>/*.deb". ' \
+                'Skipping.'
     exit 0
   end
 
@@ -197,7 +200,8 @@ class AptStageArtifacts
   # Invoke the remote staging of the sent artifacts
   def invoke_remote_staging
     # Do this in a login shell so we get rvm PATH goodness
-    %x(ssh #{@staging_server} '/bin/bash -l -c "#{@remote_staging_command} #{@remote_tarball_path}"')
+    staging_command = "#{@remote_staging_command} #{@remote_tarball_path}"
+    %x(ssh #{@staging_server} '/bin/bash -l -c "#{staging_command}"')
     return if $CHILD_STATUS.success?
 
     fatal "'ssh #{@staging_server} #{@remote_staging_command} #{@remote_tarball_path}' failed."

data/lib/apt_stage_from_tarball.rb

@@ -31,7 +31,7 @@ class AptStageFromTarball
     extract_tarball
     add_artifacts_to_freight_library
     update_freight_cache
-    # cleanup_tarball_directory
+    cleanup_tarball_directory
     logger.info 'Freight artifact staging successful.'
   end
 
@@ -46,9 +46,10 @@ class AptStageFromTarball
   end
 
   def extract_tarball
-    fatal "'#{@tarball_path}' does not exist." unless File.exist?(@tarball_path)
-    fatal "'#{@tarball_path}' is not a regular file." unless File.file?(@tarball_path)
-    fatal "'#{@tarball_path}' is not readable." unless File.readable?(@tarball_path)
+    fatal "Missing tarball_path command argument." if !@tarball_path
+    fatal "'#{@tarball_path}' does not exist" unless File.exist?(@tarball_path)
+    fatal "'#{@tarball_path}' is not a regular file" unless File.file?(@tarball_path)
+    fatal "'#{@tarball_path}' is not readable" unless File.readable?(@tarball_path)
 
     @tarball_directory = File.dirname(@tarball_path)
 
@@ -69,6 +70,11 @@ class AptStageFromTarball
     logger.info("'#{@tarball_path}' extracted.")
   end
 
+  def cleanup_tarball_directory
+    FileUtils.rm_r @tarball_directory
+    logger.info("'#{@tarball_directory}' deleted.")
+  end
+
   # Read through the staging manifest, adding each line into the correct apt freight
   # library.
   def add_artifacts_to_freight_library
@@ -76,12 +82,12 @@ class AptStageFromTarball
       manifest_file = File.read(@manifest_path)
       JSON.parse(manifest_file, symbolize_names: true).each do |artifact|
         add_to_freight_library_command = %W[
-           #{@apt_add_to_freight_library_command}
-           --verbose
-           --puppet-version=#{artifact[:puppet_version]}
-           --component=#{artifact[:apt_component]}
-           --codename=#{artifact[:codename]}
-           #{File.join(Dir.pwd, artifact[:deb_file_path])}
+          #{@apt_add_to_freight_library_command}
+          --verbose
+          --puppet-version=#{artifact[:puppet_version]}
+          --component=#{artifact[:apt_component]}
+          --codename=#{artifact[:codename]}
+          #{File.join(Dir.pwd, artifact[:deb_file_path])}
         ].join(' ')
 
         %x(#{add_to_freight_library_command})
@@ -102,7 +108,7 @@ class AptStageFromTarball
         #{@apt_update_freight_cache_command}
         --verbose
         --puppet-version=#{puppet_version}
-        ].join(' ')
+      ].join(' ')
 
       %x(#{update_freight_cache_command})
       fatal "#{update_freight_cache_command} failed." unless $CHILD_STATUS.success?

data/lib/apt_update_freight_cache.rb

@@ -72,7 +72,7 @@ class AptUpdateFreightCache
     end
 
     freight_config_file = "/etc/freight.conf.d/puppet#{@puppet_version}.conf"
-    fatal "Cannot read '#{freight_config_file} on #{`hostname`.chomp}." unless
+    fatal "Cannot read '#{freight_config_file} on #{%x(hostname).chomp}." unless
       File.readable?(freight_config_file)
 
     logger.info "Using '#{freight_config_file}'"

data/lib/mixins/constants.rb

@@ -12,8 +12,8 @@ class AptStageArtifacts
 
   # Debian/Ubuntu codenames that we care about.
   # Some codenames are skipped because we don't/haven't shipped for them.
-  DEBIAN_CODENAMES = %w[jessie stretch buster bullseye bookworm trixie]
-  UBUNTU_CODENAMES = %w[xenial bionic focal groovy]
+  DEBIAN_CODENAMES = %w[wheezy jessie stretch buster bullseye bookworm trixie]
+  UBUNTU_CODENAMES = %w[lucid precise trusty xenial bionic focal groovy jammy]
   VALID_CODENAMES = DEBIAN_CODENAMES | UBUNTU_CODENAMES
 
   # Validation constraints

data/lib/version.rb

@@ -1,3 +1,3 @@
 class AptStageArtifacts
-  VERSION = '0.9.0'
+  VERSION = '0.11.0'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: apt_stage_artifacts
 version: !ruby/object:Gem::Version
-  version: 0.9.0
+  version: 0.11.0
 platform: ruby
 authors:
 - Puppet Release Engineering
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2021-09-29 00:00:00.000000000 Z
+date: 2022-05-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: pry
@@ -66,6 +66,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '3.0'
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: '0.81'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: '0.81'
 - !ruby/object:Gem::Dependency
   name: rubocop-rake
   requirement: !ruby/object:Gem::Requirement
@@ -172,14 +186,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.5.0
+      version: 2.3.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.28
+rubygems_version: 3.1.6
 signing_key: 
 specification_version: 4
 summary: Stages .deb artifacts to a remote freight repository