bolt 3.26.2 → 3.27.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7fe078a4da4d1447d8214edee250d80c0132b2a8913bd16f24016315a46ff215
-  data.tar.gz: 89e27a1ed523fd9aca8d80ea94c53071bde9f0b1af947768880b8055dc47f27b
+  metadata.gz: 18244996224b24b6a1a2395a54e97b11389d5b106a18dc8e2ef97563d3a08134
+  data.tar.gz: 3d0e5384b7ce5a5b73a082e4d8529defa179737c25ee0166e7be39de0bf7f7c8
 SHA512:
-  metadata.gz: 8955b0ea1e737618918839b34c9b6049f884c1983e3ebb0f4ee50c4ea85007fade6dabfbf21ae34f201bf0eabb9497bcd5a33fbe3c833ada430ccced628b4477
-  data.tar.gz: 8b1e6f84326c1e5adeb988cdadac80b2bb49b71ed2ce24dc0b0c881970e88aadbede4eaf5f3479b8f2434a9a4a221128e643d145ecffc47b1075d566ef741cd3
+  metadata.gz: 35ae42d1527062768eadc54e4e0864d015a57e61d5ea9319e6ffd835407bd34a7646f6c632c0911862b37a2c010b5d3921e6badb7bda8219bc5334c3b56b6844
+  data.tar.gz: 67109b48c5661ac51ecb9807ca61355006e34165ad287cea54d3b2dcb4615cdd038659842ca1f0c8a98a6e654a747fcfe0aabf030dabb3974173acbadc996a72

data/lib/bolt/applicator.rb

@@ -256,7 +256,7 @@ module Bolt
         futures = targets.map do |target|
           Concurrent::Future.execute(executor: @pool) do
             Thread.current[:name] ||= Thread.current.name
-            @executor.with_node_logging("Compiling manifest block", [target]) do
+            @executor.with_node_logging("Compiling manifest block", [target], :trace) do
               compile(target, scope)
             end
           end

data/lib/bolt/config/options.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require_relative '../../bolt/config/transport/docker'
+require_relative '../../bolt/config/transport/jail'
 require_relative '../../bolt/config/transport/local'
 require_relative '../../bolt/config/transport/lxd'
 require_relative '../../bolt/config/transport/orch'
@@ -16,6 +17,7 @@ module Bolt
       # gets passed along to the inventory.
       TRANSPORT_CONFIG = {
         'docker' => Bolt::Config::Transport::Docker,
+        'jail'   => Bolt::Config::Transport::Jail,
         'local'  => Bolt::Config::Transport::Local,
         'lxd'    => Bolt::Config::Transport::LXD,
         'pcp'    => Bolt::Config::Transport::Orch,
@@ -550,6 +552,12 @@ module Bolt
           _plugin: true,
           _example: { "cleanup" => false, "service-url" => "https://docker.example.com" }
         },
+        "jail" => {
+          description: "A map of configuration options for the jail transport.",
+          type: Hash,
+          _plugin: true,
+          _example: { cleanup: false }
+        },
         "local" => {
           description: "A map of configuration options for the local transport. The set of available options is "\
                        "platform dependent.",

data/lib/bolt/config/transport/jail.rb

@@ -0,0 +1,33 @@
+# frozen_string_literal: true
+
+require_relative '../../../bolt/error'
+require_relative '../../../bolt/config/transport/base'
+
+module Bolt
+  class Config
+    module Transport
+      class Jail < Base
+        OPTIONS = %w[
+          cleanup
+          host
+          interpreters
+          shell-command
+          tmpdir
+          user
+        ].concat(RUN_AS_OPTIONS).sort.freeze
+
+        DEFAULTS = {
+          'cleanup' => true
+        }.freeze
+
+        private def validate
+          super
+
+          if @config['interpreters']
+            @config['interpreters'] = normalize_interpreters(@config['interpreters'])
+          end
+        end
+      end
+    end
+  end
+end

data/lib/bolt/executor.rb

@@ -14,6 +14,7 @@ require_relative '../bolt/result'
 require_relative '../bolt/result_set'
 # Load transports
 require_relative '../bolt/transport/docker'
+require_relative '../bolt/transport/jail'
 require_relative '../bolt/transport/local'
 require_relative '../bolt/transport/lxd'
 require_relative '../bolt/transport/orch'
@@ -25,6 +26,7 @@ require_relative '../bolt/transport/winrm'
 module Bolt
   TRANSPORTS = {
     docker: Bolt::Transport::Docker,
+    jail: Bolt::Transport::Jail,
     local: Bolt::Transport::Local,
     lxd: Bolt::Transport::LXD,
     pcp: Bolt::Transport::Orch,

data/lib/bolt/shell/bash.rb

@@ -356,7 +356,7 @@ module Bolt
           if defined? conn.add_env_vars
             conn.add_env_vars(options[:environment])
           else
-            env_decl = options[:environment].map do |env, val|
+            env_decl = '/usr/bin/env ' + options[:environment].map do |env, val|
               "#{env}=#{Shellwords.shellescape(val)}"
             end.join(' ')
           end

data/lib/bolt/transport/jail/connection.rb

@@ -0,0 +1,81 @@
+# frozen_string_literal: true
+
+require 'logging'
+require_relative '../../../bolt/node/errors'
+
+module Bolt
+  module Transport
+    class Jail < Simple
+      class Connection
+        attr_reader :user, :target
+
+        def initialize(target)
+          raise Bolt::ValidationError, "Target #{target.safe_name} does not have a host" unless target.host
+          @target = target
+          @user = @target.user || ENV['USER'] || Etc.getlogin
+          @logger = Bolt::Logger.logger(target.safe_name)
+          @jail_info = {}
+          @logger.trace("Initializing jail connection to #{target.safe_name}")
+        end
+
+        def shell
+          @shell ||= Bolt::Shell::Bash.new(target, self)
+        end
+
+        def reset_cwd?
+          true
+        end
+
+        def jail_id
+          @jail_info['jid'].to_s
+        end
+
+        def jail_path
+          @jail_info['path']
+        end
+
+        def connect
+          output = JSON.parse(`jls --libxo=json`)
+          @jail_info = output['jail-information']['jail'].select { |jail| jail['hostname'] == target.host }.first
+          raise "Could not find a jail with name matching #{target.host}" if @jail_info.nil?
+          @logger.trace { "Opened session" }
+          true
+        rescue StandardError => e
+          raise Bolt::Node::ConnectError.new(
+            "Failed to connect to #{target.safe_name}: #{e.message}",
+            'CONNECT_ERROR'
+          )
+        end
+
+        def execute(command)
+          args = ['-lU', @user]
+
+          jail_command = %w[jexec] + args + [jail_id] + Shellwords.split(command)
+          @logger.trace { "Executing #{jail_command.join(' ')}" }
+
+          Open3.popen3({}, *jail_command)
+        rescue StandardError
+          @logger.trace { "Command aborted" }
+          raise
+        end
+
+        def upload_file(source, destination)
+          @logger.trace { "Uploading #{source} to #{destination}" }
+          jail_destination = File.join(jail_path, destination)
+          FileUtils.cp(source, jail_destination)
+        rescue StandardError => e
+          raise Bolt::Node::FileError.new(e.message, 'WRITE_ERROR')
+        end
+
+        def download_file(source, destination, _download)
+          @logger.trace { "Downloading #{source} to #{destination}" }
+          jail_source = File.join(jail_path, source)
+          FileUtils.mkdir_p(destination)
+          FileUtils.cp(jail_source, destination)
+        rescue StandardError => e
+          raise Bolt::Node::FileError.new(e.message, 'WRITE_ERROR')
+        end
+      end
+    end
+  end
+end

data/lib/bolt/transport/jail.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+require_relative '../../bolt/transport/simple'
+
+module Bolt
+  module Transport
+    class Jail < Simple
+      def provided_features
+        ['shell']
+      end
+
+      def with_connection(target)
+        conn = Connection.new(target)
+        conn.connect
+        yield conn
+      end
+    end
+  end
+end
+
+require_relative 'jail/connection'

data/lib/bolt/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Bolt
-  VERSION = '3.26.2'
+  VERSION = '3.27.2'
 end

data/lib/bolt_server/schemas/action-run_task.json

@@ -9,7 +9,11 @@
       "type": "object",
       "description": "JSON formatted parameters to be provided to task"
     },
-    "target": { "$ref": "partial:target-any" }
+    "target": { "$ref": "partial:target-any" },
+    "timeout": {
+      "type": "integer",
+      "description": "Number of seconds to wait before abandoning the task execution on the tartet."
+    }
   },
   "required": ["target", "task"],
   "additionalProperties": false

data/lib/bolt_server/transport_app.rb

@@ -9,7 +9,6 @@ require 'bolt/project'
 require 'bolt/target'
 require 'bolt_server/file_cache'
 require 'bolt_server/plugin'
-require 'bolt_server/plugin/puppet_connect_data'
 require 'bolt_server/request_error'
 require 'bolt/task/puppet_server'
 require 'json'
@@ -42,7 +41,6 @@ module BoltServer
       action-upload_file
       transport-ssh
       transport-winrm
-      connect-data
       action-apply_prep
       action-apply
     ].freeze
@@ -126,7 +124,17 @@ module BoltServer
       end
     end
 
-    def task_helper(target, task, parameters)
+    def unwrap_sensitive_results(result_set)
+      # Take a ResultSet and unwrap sensitive values
+      result_set.each do |result|
+        value = result.value
+        next unless value.is_a?(Hash)
+        next unless value.key?('_sensitive')
+        value['_sensitive'] = value['_sensitive'].unwrap
+      end
+    end
+
+    def task_helper(target, task, parameters, timeout = nil)
       # Wrap parameters marked with '"sensitive": true' in the task metadata with a
       # Sensitive wrapper type. This way it's not shown in logs.
       if (param_spec = task.parameters)
@@ -137,11 +145,20 @@ module BoltServer
         end
       end
 
-      @executor.run_task(target, task, parameters).each do |result|
-        value = result.value
-        next unless value.is_a?(Hash)
-        next unless value.key?('_sensitive')
-        value['_sensitive'] = value['_sensitive'].unwrap
+      if timeout && timeout > 0
+        task_thread = Thread.new do
+          unwrap_sensitive_results(@executor.run_task(target, task, parameters))
+        end
+        # Wait for the timeout for the task to execute in the thread. If `join` times out, result will be nil.
+        if task_thread.join(timeout).nil?
+          task_thread.kill
+          raise Bolt::Error.new("Task execution on #{target.first.safe_name} timed out after #{timeout} seconds",
+                                'boltserver/task-timeout')
+        else
+          task_thread.value
+        end
+      else
+        unwrap_sensitive_results(@executor.run_task(target, task, parameters))
       end
     end
 
@@ -150,7 +167,7 @@ module BoltServer
 
       task_data = body['task']
       task = Bolt::Task::PuppetServer.new(task_data['name'], task_data['metadata'], task_data['files'], @file_cache)
-      task_helper(target, task, body['parameters'] || {})
+      task_helper(target, task, body['parameters'] || {}, body['timeout'])
     end
 
     def extract_install_task(target)
@@ -825,53 +842,6 @@ module BoltServer
       [500, e.message]
     end
 
-    # Returns a list of targets parsed from a Project inventory
-    #
-    # @param versioned_project [String] the versioned_project to compute the inventory from
-    post '/project_inventory_targets' do
-      content_type :json
-      body = JSON.parse(request.body.read)
-      validate_schema(@schemas["connect-data"], body)
-      in_bolt_project(body['versioned_project']) do |context|
-        if context[:config].inventoryfile &&
-           context[:config].project.inventory_file.to_s !=
-           context[:config].inventoryfile
-          raise Bolt::ValidationError, "Project inventory must be defined in the " \
-            "inventory.yaml file at the root of the project directory"
-        end
-
-        Bolt::Util.validate_file('inventory file', context[:config].project.inventory_file)
-
-        begin
-          # Set the default puppet_library plugin hook if it has not already been
-          # set
-          context[:config].data['plugin-hooks']['puppet_library'] ||= {
-            'plugin'     => 'task',
-            'task'       => 'puppet_agent::install',
-            'parameters' => {
-              'stop_service' => true
-            }
-          }
-
-          connect_plugin = BoltServer::Plugin::PuppetConnectData.new(body['puppet_connect_data'])
-          plugins = Bolt::Plugin.new(context[:config], context[:pal], load_plugins: false)
-          plugins.add_plugin(connect_plugin)
-          %w[aws_inventory azure_inventory gcloud_inventory].each do |plugin_name|
-            plugins.add_module_plugin(plugin_name) if plugins.known_plugin?(plugin_name)
-          end
-          inventory = Bolt::Inventory.from_config(context[:config], plugins)
-          target_list = inventory.get_targets('all').map do |targ|
-            targ.to_h.merge({ 'transport' => targ.transport, 'plugin_hooks' => targ.plugin_hooks })
-          end
-        rescue Bolt::Plugin::PluginError::LoadingDisabled => e
-          msg = "Cannot load plugin #{e.details['plugin_name']}: plugin not supported"
-          raise BoltServer::Plugin::PluginNotSupported.new(msg, e.details['plugin_name'])
-        end
-
-        [200, target_list.to_json]
-      end
-    end
-
     # Returns the base64 encoded tar archive of plugin code that is needed to calculate
     # custom facts
     #

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bolt
 version: !ruby/object:Gem::Version
-  version: 3.26.2
+  version: 3.27.2
 platform: ruby
 authors:
 - Puppet
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2022-11-16 00:00:00.000000000 Z
+date: 2023-08-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: addressable
@@ -42,14 +42,14 @@ dependencies:
   name: CFPropertyList
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '2.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '2.2'
 - !ruby/object:Gem::Dependency
@@ -59,6 +59,9 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: 1.2.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -66,6 +69,9 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.0'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: 1.2.0
 - !ruby/object:Gem::Dependency
   name: ffi
   requirement: !ruby/object:Gem::Requirement
@@ -146,16 +152,22 @@ dependencies:
   name: net-scp
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '1.2'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '5.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '1.2'
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '5.0'
 - !ruby/object:Gem::Dependency
   name: net-ssh
   requirement: !ruby/object:Gem::Requirement
@@ -165,7 +177,7 @@ dependencies:
         version: '4.0'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '7.0'
+        version: '8.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -175,7 +187,7 @@ dependencies:
         version: '4.0'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '7.0'
+        version: '8.0'
 - !ruby/object:Gem::Dependency
   name: net-ssh-krb
   requirement: !ruby/object:Gem::Requirement
@@ -261,7 +273,7 @@ dependencies:
         version: 2.3.0
     - - "<"
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '5.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -271,7 +283,7 @@ dependencies:
         version: 2.3.0
     - - "<"
       - !ruby/object:Gem::Version
-        version: '4.0'
+        version: '5.0'
 - !ruby/object:Gem::Dependency
   name: r10k
   requirement: !ruby/object:Gem::Requirement
@@ -374,16 +386,16 @@ dependencies:
   name: puppetlabs_spec_helper
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "<="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.15.0
+        version: '5.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "<="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.15.0
+        version: '5.0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -509,6 +521,7 @@ files:
 - lib/bolt/config/options.rb
 - lib/bolt/config/transport/base.rb
 - lib/bolt/config/transport/docker.rb
+- lib/bolt/config/transport/jail.rb
 - lib/bolt/config/transport/local.rb
 - lib/bolt/config/transport/lxd.rb
 - lib/bolt/config/transport/options.rb
@@ -606,6 +619,8 @@ files:
 - lib/bolt/transport/base.rb
 - lib/bolt/transport/docker.rb
 - lib/bolt/transport/docker/connection.rb
+- lib/bolt/transport/jail.rb
+- lib/bolt/transport/jail/connection.rb
 - lib/bolt/transport/local.rb
 - lib/bolt/transport/local/connection.rb
 - lib/bolt/transport/lxd.rb
@@ -640,7 +655,6 @@ files:
 - lib/bolt_server/schemas/action-run_script.json
 - lib/bolt_server/schemas/action-run_task.json
 - lib/bolt_server/schemas/action-upload_file.json
-- lib/bolt_server/schemas/connect-data.json
 - lib/bolt_server/schemas/partials/target-any.json
 - lib/bolt_server/schemas/partials/target-ssh.json
 - lib/bolt_server/schemas/partials/target-winrm.json
@@ -696,7 +710,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.6
+rubygems_version: 3.4.12
 signing_key: 
 specification_version: 4
 summary: Execute commands remotely over SSH and WinRM

data/lib/bolt_server/schemas/connect-data.json

@@ -1,25 +0,0 @@
-{
-  "$schema": "http://json-schema.org/draft-04/schema#",
-  "title": "project_inventory_targets connect plugin data",
-  "description": "POST project_inventory_targets connect plugin data",
-  "type": "object",
-  "properties": {
-    "puppet_connect_data": {
-      "type": "object",
-      "patternProperties": {
-        ".*": {
-          "type": "object",
-          "properties": {
-            "value": {}
-          },
-          "required": ["value"]
-        }
-      },
-      "additionalProperties": false
-    },
-    "versioned_project": {
-      "type": "string"
-    }
-  },
-  "required": ["puppet_connect_data", "versioned_project"]
-}