kitchen-transport-express 1.3.1 → 1.4.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 23840e5574cc8a84acfcb87e24a3a4f6c26ea1174a3c5c2bc8390d3fc23843b6
-  data.tar.gz: eb2165a04339df078065117b00af095b8cba4903d0c5d03233cedfc5b3ac6a8e
+  metadata.gz: 747b9b46fe258347afe6a875be3589040dd32a7a0a683f55daf7db89432f56b8
+  data.tar.gz: 5718e229fd6b29f60aee5347ac04805e69f105f3b243371710d1533f80fe90e1
 SHA512:
-  metadata.gz: 8257d47351212661fd997b9b3397cdcc63f5f998352fd96362cd6d21092aba753943349db7c31688a705acbbec2582e4754f31b2dc57938b5771c3a726bb95f5
-  data.tar.gz: a3d90892108b59c591f965debb21bb38de8a6eab34f397703660ee5866c7c9a5158bb5a1e90974788d0c41ee8829c95a9457409ab8f4dca62c9b650a17cf2d2a
+  metadata.gz: 0ca322124891398ced97700e80c53e5ea8c2b62fda982444919b78e4304127fdad4bd9eae74736cd28e6ce9902cd1c7ee09052d33c4487cdb71d6cb912b9868d
+  data.tar.gz: 37bdcf4336a387fc525ed2e8bba704e23adfcc5e50e8d017adf090521afd97dcfbb3ad9d32313c5e8b11542c863499f0e395df328c71c51f25dc4e1bb32104b6

data/.github/workflows/gem-push.yml

@@ -0,0 +1,47 @@
+name: Ruby Gem
+
+on:
+  release:
+    types: [published]
+
+jobs:
+  build:
+    name: Build + Publish
+    runs-on: ubuntu-latest
+    permissions:
+      contents: read
+      packages: write
+
+    steps:
+    - uses: actions/checkout@v4
+    - name: Set up Ruby
+    # To automatically get bug fixes and new Ruby versions for ruby/setup-ruby,
+    # change this to (see https://github.com/ruby/setup-ruby#versioning):
+      uses: ruby/setup-ruby@v1
+      with:
+        ruby-version: 3.1
+        rubygems: 3.2.3
+        bundler-cache: true # runs 'bundle install' and caches installed gems automatically
+
+    - name: Publish to GPR
+      run: |
+        mkdir -p $HOME/.gem
+        touch $HOME/.gem/credentials
+        chmod 0600 $HOME/.gem/credentials
+        printf -- "---\n:github: ${GEM_HOST_API_KEY}\n" > $HOME/.gem/credentials
+        gem build *.gemspec
+        gem push --KEY github --host https://rubygems.pkg.github.com/${OWNER} *.gem
+      env:
+        GEM_HOST_API_KEY: "Bearer ${{secrets.GH_TOKEN}}"
+        OWNER: ${{ github.repository_owner }}
+
+    - name: Publish to RubyGems
+      run: |
+        mkdir -p $HOME/.gem
+        touch $HOME/.gem/credentials
+        chmod 0600 $HOME/.gem/credentials
+        printf -- "---\n:rubygems_api_key: ${GEM_HOST_API_KEY}\n" > $HOME/.gem/credentials
+        gem build *.gemspec
+        gem push *.gem
+      env:
+        GEM_HOST_API_KEY: "${{secrets.RUBYGEMS_AUTH_TOKEN}}"

data/CHANGELOG.md

@@ -1,5 +1,12 @@
 # kitchen-transport-express CHANGELOG
 
+## 1.4.1
+* fix: 👽️ remove remote command that has become incompatible with `test-kitchen >= 3.9.0`
+
+## 1.4.0
+* feat: ♻️ utilize concurrent futures for enhanced parallelism
+* feat: 🔊 enhanced logging
+
 ## 1.3.1
 * fix: 🐛 ensure directories that only contain dot files get archived
 

data/lib/kitchen/transport/express/archiver.rb

@@ -31,11 +31,24 @@ module Kitchen
           archive_basename = ::File.basename(path) + ".tgz"
           archive_full_name = ::File.join(::File.dirname(path), archive_basename)
           files = all_files(path)
-          logger.debug("[#{Express::LOG_PREFIX}] #{path} contains #{files.size} files.")
+          start_time = Time.now
           create_archive(path, files, archive_full_name)
+          Express.log(logger, "create archive #{File.basename(archive_full_name)} (#{files.size} files)", start_time)
           archive_full_name
         end
 
+        # Transfers the archive to the remote host.
+        #
+        # @param session [Net::SSH::Connection::Session] the SSH session used to connect to the remote host and execute the extract and cleanup commands.
+        # @param local [String] the directory in the local sandbox that is being processed.
+        # @param remote [String] the remote directory (kitchen_root).
+        # @param opts [Hash] the ssh options that came in from the Kitchen instance.
+        def scp(session, local, remote, opts = {})
+          start_time = Time.now
+          session.scp.upload!(local, remote, opts)
+          Express.log(logger, "upload #{File.basename(local)} (Thread ID: #{Thread.current.object_id})", start_time)
+        end
+
         # Extracts the archive on the remote host.
         #
         # @param session [Net::SSH::Connection::Session] the SSH session used to connect to the remote host and execute the extract and cleanup commands.
@@ -44,13 +57,14 @@ module Kitchen
         def extract(session, local, remote)
           return unless local.match(/.*\.tgz/)
 
+          start_time = Time.now
           archive_basename = File.basename(local)
-          logger.debug("[#{Express::LOG_PREFIX}] Extracting #{::File.join(remote, archive_basename)}")
           session.open_channel do |channel|
             channel.request_pty
             channel.exec("tar -xzf #{::File.join(remote, archive_basename)} -C #{remote} && rm -f #{File.join(remote, archive_basename)}")
           end
           session.loop
+          Express.log(logger, "extract #{File.basename(local)} (Thread ID: #{Thread.current.object_id})", start_time)
         end
 
         private

data/lib/kitchen/transport/express/version.rb

@@ -20,7 +20,7 @@ module Kitchen
       # The version string for Kitchen Transport Express.
       #
       # @author Justin Steele <justin.steele@oracle.com>
-      VERSION = "1.3.1"
+      VERSION = "1.4.1"
     end
   end
 end

data/lib/kitchen/transport/express_ssh.rb

@@ -27,6 +27,16 @@ module Kitchen
     class Express
       # A constant that gets prepended to debugger messages.
       LOG_PREFIX = "EXPRESS"
+
+      # Logger class method to unify logging.
+      #
+      # @param logger [Kitchen::Logger] the logger that was created by the kitchen instance.
+      # @param message [String] the message to output.
+      # @param start_time [Time] the start time of the process if duration is desired to be part of the message.
+      def self.log(logger, message = nil, start_time = nil)
+        message = "#{message} (#{Time.now - start_time}s)" if start_time
+        logger.debug "[#{Express::LOG_PREFIX}] [#{Time.now.getutc.strftime("%Y-%m-%dT%H:%M:%S%:z")}] #{message}"
+      end
     end
 
     # Express SSH Transport Error class.
@@ -51,7 +61,7 @@ module Kitchen
       # @return [Ssh::Connection] an instance of Kitchen::Transport::ExpressSsh::Connection.
       def create_new_connection(options, &block)
         if @connection
-          logger.debug("[#{Express::LOG_PREFIX}] Shutting previous connection #{@connection}")
+          Express.log(logger, "shutting previous connection #{@connection}")
           @connection.close
         end
 
@@ -95,33 +105,35 @@ module Kitchen
         # @param locals [Array] the top-level list of directories and files to be transfered.
         # @param remote [String] the remote directory (kitchen_root).
         # @raise [ExpressFailed] if any of the threads raised an exception.
-        def upload(locals, remote) # rubocop: disable Metrics/MethodLength
+        def upload(locals, remote)
           return super unless valid_remote_requirements?(remote)
 
-          processed_locals = process_locals(locals)
-          pool, exceptions = thread_pool(processed_locals)
-          processed_locals.each do |local|
-            pool.post do
-              transfer(local, remote, session.options)
-            rescue => e
-              exceptions << e.cause
-            end
-          end
-          pool.shutdown
-          pool.wait_for_termination
-
-          raise ExpressFailed, exceptions.pop unless exceptions.empty?
-        end # rubocop: enable Metrics/MethodLength
+          start_time = Time.now
+          processed_local = process_locals(locals)
+          futures = create_futures(processed_local, remote)
+          all_done = Concurrent::Promise.zip(*futures).execute
+          all_done.value!
+        rescue => e
+          raise ExpressFailed, e.cause.to_s
+        ensure
+          Express.log(logger, "transport express complete", start_time)
+        end
 
         private
 
-        # Creates the thread pool and exceptions queue.
+        # Creates the concurrent futures.
         #
-        # @param processed_locals [Array] list of files and archives to be uploaded.
-        # @return [Array(Concurrent::FixedThreadPool, Queue)]
+        # @param locals [Array] list of files and archives to be uploaded.
+        # @return [Array(Concurrent::Promise)]
         # @api private
-        def thread_pool(processed_locals)
-          [Concurrent::FixedThreadPool.new([processed_locals.length, 10].min), Queue.new]
+        def create_futures(locals, remote)
+          # Start upload futures
+          executor = Concurrent::FixedThreadPool.new([locals.length, 10].min)
+          locals.map do |local|
+            Concurrent::Promise.execute(executor: executor) do
+              transfer(local, remote, session.options)
+            end
+          end
         end
 
         # Ensures the remote host has the minimum-required executables to extract the archives.
@@ -131,11 +143,9 @@ module Kitchen
         # @api private
         def valid_remote_requirements?(remote)
           execute("(which tar && which gzip) > /dev/null")
-          execute("mkdir -p #{remote}")
           true
         rescue => e
-          logger.debug("[#{Express::LOG_PREFIX}] Requirements not met on remote host for Express transport.")
-          logger.debug("[#{Express::LOG_PREFIX}] #{e}")
+          Express.log(logger, "Requirements not met on remote host for Express transport.\n#{e}")
           false
         end
 
@@ -166,13 +176,11 @@ module Kitchen
         # @raise [StandardError] if the files could not be uploaded successfully.
         # @api private
         def transfer(local, remote, opts = {})
-          logger.debug("[#{Express::LOG_PREFIX}] Transferring #{local} to #{remote}")
-
           Net::SSH.start(session.host, opts[:user], **opts) do |ssh|
-            ssh.scp.upload!(local, remote, opts)
+            scp(ssh, local, remote, opts)
             extract(ssh, local, remote)
           rescue Net::SCP::Error => ex
-            logger.debug("[#{Express::LOG_PREFIX}] upload failed with #{ex.message.strip}")
+            Express.log(logger, "upload failed with #{ex.message.strip}")
             raise "(#{ex.message.strip})"
           end
         end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: kitchen-transport-express
 version: !ruby/object:Gem::Version
-  version: 1.3.1
+  version: 1.4.1
 platform: ruby
 authors:
 - Justin Steele
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-04-11 00:00:00.000000000 Z
+date: 2025-11-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: test-kitchen
@@ -144,6 +144,7 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/workflows/gem-push.yml"
 - ".github/workflows/ruby.yml"
 - ".gitignore"
 - ".rubocop.yml"
@@ -165,7 +166,7 @@ metadata:
   documentation_uri: https://github.com/justintsteele/kitchen-transport-express/blob/main/README.md
   homepage_uri: https://github.com/justintsteele/kitchen-transport-express
   source_code_uri: https://github.com/justintsteele/kitchen-transport-express
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -181,7 +182,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubygems_version: 3.3.27
-signing_key: 
+signing_key:
 specification_version: 4
 summary: Skip the long lines in Kitchen Transport!
 test_files: []