bolt 1.37.0 → 1.38.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4e9167735253231222010ffb141c42048039e6b02c13061dbc6394db8876a6d7
-  data.tar.gz: b10298a67bb280b58c265ce2238f91ac611b672eb3c303e9f507e15db6e05e7e
+  metadata.gz: 56eab83c9278b21bfe1241b6fcbbfac9a06adc976d77a7d5900a4940393ea5cd
+  data.tar.gz: c1389de9cad81f4f8ba9b2af9b3aa74bffdbc92e0f9c88ae6f018e7fa43ded1a
 SHA512:
-  metadata.gz: 857e83019439072c518e50ad9c9d1c0dc93cc0e8b536f53348824091d6367fdc6df4b08c2d6c9dbf0cb45fb7fda29051c2743d470e4bba82ec7b708d7ed997cc
-  data.tar.gz: f91623e0811677b7c9c9f643f3a512120e7af52edbaff7d74381b0d7a40556d77dc0f7c98cf7117fdd69db504b310681157e3c44367a27fe57089e6a3dc3dc41
+  metadata.gz: 104b2eda64ad81d4fc6bbf054309498866b6e8f3e256f455c06b19bb5a2c6e50192dc2c480ffa7c885e2afb2bebc0648e5feb295f22f7a9cc6b5f250b548ea22
+  data.tar.gz: 6115db7ac02b54224e346d12c2bd2fa98d6eee3031b4ae5093f89d0c2883e000cccb63428ace63b6e9a6ed173afafe327274b0b829109c9d3bd1c6fd75b9f699

data/Puppetfile

@@ -7,7 +7,7 @@ moduledir File.join(File.dirname(__FILE__), 'modules')
 # Core modules used by 'apply'
 mod 'puppetlabs-service', '1.1.0'
 mod 'puppetlabs-facts', '0.6.0'
-mod 'puppetlabs-puppet_agent', '2.2.1'
+mod 'puppetlabs-puppet_agent', '2.2.2'
 
 # Core types and providers for Puppet 6
 mod 'puppetlabs-augeas_core', '1.0.5'
@@ -33,6 +33,7 @@ mod 'puppetlabs-azure_inventory', '0.2.0'
 mod 'puppetlabs-terraform', '0.2.0'
 mod 'puppetlabs-vault', '0.2.2'
 mod 'puppetlabs-aws_inventory', '0.2.0'
+mod 'puppetlabs-yaml', '0.1.0'
 
 # If we don't list these modules explicitly, r10k will purge them
 mod 'canary', local: true

data/bolt-modules/boltlib/lib/puppet/functions/apply_prep.rb

@@ -50,8 +50,8 @@ Puppet::Functions.create_function(:apply_prep) do
   end
   # rubocop:enable Naming/AccessorMethodName
 
-  def run_task(targets, task, args = {})
-    executor.run_task(targets, task, args)
+  def run_task(targets, task, args = {}, options = {})
+    executor.run_task(targets, task, args, options)
   end
 
   # Returns true if the target has the puppet-agent feature defined, either from inventory or transport.

data/bolt-modules/file/lib/puppet/functions/file/exists.rb

@@ -16,6 +16,6 @@ Puppet::Functions.create_function(:'file::exists', Puppet::Functions::InternalFu
   def exists(scope, filename)
     Puppet.lookup(:bolt_executor) {}&.report_function_call(self.class.name)
     found = Puppet::Parser::Files.find_file(filename, scope.compiler.environment)
-    found && Puppet::FileSystem.exist?(found)
+    found ? Puppet::FileSystem.exist?(found) : false
   end
 end

data/bolt-modules/file/lib/puppet/functions/file/readable.rb

@@ -16,6 +16,6 @@ Puppet::Functions.create_function(:'file::readable', Puppet::Functions::Internal
   def readable(scope, filename)
     Puppet.lookup(:bolt_executor) {}&.report_function_call(self.class.name)
     found = Puppet::Parser::Files.find_file(filename, scope.compiler.environment)
-    found && File.readable?(found)
+    found ? File.readable?(found) : false
   end
 end

data/bolt-modules/system/lib/puppet/functions/system/env.rb

@@ -7,7 +7,7 @@ Puppet::Functions.create_function(:'system::env') do
   #   system::env('USER')
   dispatch :env do
     required_param 'String', :name
-    return_type 'String'
+    return_type 'Optional[String]'
   end
 
   def env(name)

data/lib/bolt/bolt_option_parser.rb

@@ -7,8 +7,8 @@ require 'optparse'
 module Bolt
   class BoltOptionParser < OptionParser
     OPTIONS = { inventory: %w[nodes targets query rerun description],
-                authentication: %w[user password private-key host-key-check ssl ssl-verify],
-                escalation: %w[run-as sudo-password],
+                authentication: %w[user password password-prompt private-key host-key-check ssl ssl-verify],
+                escalation: %w[run-as sudo-password sudo-password-prompt],
                 run_context: %w[concurrency inventoryfile save-rerun],
                 global_config_setters: %w[modulepath boltdir configfile],
                 transports: %w[transport connect-timeout tty],
@@ -54,6 +54,9 @@ module Bolt
         when 'init'
           { flags: OPTIONS[:global],
             banner: PROJECT_INIT_HELP }
+        when 'migrate'
+          { flags: OPTIONS[:global] + %w[inventoryfile boltdir configfile],
+            banner: PROJECT_MIGRATE_HELP }
         else
           { flags: OPTIONS[:global],
             banner: PROJECT_HELP }
@@ -131,6 +134,7 @@ module Bolt
         bolt inventory show              Show the list of targets an action would run on
         bolt group show                  Show the list of groups in the inventory
         bolt project init                Create a new Bolt project
+        bolt project migrate             Migrate a Bolt project to the latest version
 
       Run `bolt <subcommand> --help` to view specific examples.
 
@@ -325,6 +329,7 @@ module Bolt
 
       Available actions are:
         init                     Create a new Bolt project
+        migrate                  Migrate a Bolt project to the latest version
 
       Available options are:
     PROJECT_HELP
@@ -338,10 +343,22 @@ module Bolt
       Available options are:
     PROJECT_INIT_HELP
 
+    PROJECT_MIGRATE_HELP = <<~PROJECT_MIGRATE_HELP
+      Usage: bolt project migrate
+
+      Migrate a Bolt project to the latest version.
+      Loads a Bolt project's inventory file and migrates it to the latest version. The
+      inventory file is modified in place and will not preserve comments or formatting.
+
+      Available options are:
+    PROJECT_MIGRATE_HELP
+
+    attr_reader :warnings
     def initialize(options)
       super()
 
       @options = options
+      @warnings = []
 
       define('-n', '--nodes NODES',
              'Alias for --targets',
@@ -397,6 +414,9 @@ module Bolt
       define('-p', '--password [PASSWORD]',
              'Password to authenticate with') do |password|
         if password.nil?
+          msg = "Optional parameter for --password is deprecated and will no longer prompt for password. " \
+                "Use the prompt plugin or --password-prompt instead to prompt for passwords."
+          @warnings << { option: 'password', msg: msg }
           STDOUT.print "Please enter your password: "
           @options[:password] = STDIN.noecho(&:gets).chomp
           STDOUT.puts
@@ -404,6 +424,11 @@ module Bolt
           @options[:password] = password
         end
       end
+      define('--password-prompt', 'Prompt for user to input password') do |_password|
+        STDERR.print "Please enter your password: "
+        @options[:password] = STDIN.noecho(&:gets).chomp
+        STDERR.puts
+      end
       define('--private-key KEY', 'Private ssh key to authenticate with') do |key|
         @options[:'private-key'] = key
       end
@@ -424,6 +449,9 @@ module Bolt
       define('--sudo-password [PASSWORD]',
              'Password for privilege escalation') do |password|
         if password.nil?
+          msg = "Optional parameter for --sudo-password is deprecated and will no longer prompt for password. " \
+                "Use the prompt plugin or --sudo-password-prompt instead to prompt for passwords."
+          @warnings << { option: 'sudo-password', msg: msg }
           STDOUT.print "Please enter your privilege escalation password: "
           @options[:'sudo-password'] = STDIN.noecho(&:gets).chomp
           STDOUT.puts
@@ -431,6 +459,11 @@ module Bolt
           @options[:'sudo-password'] = password
         end
       end
+      define('--sudo-password-prompt', 'Prompt for user to input escalation password') do |_password|
+        STDERR.print "Please enter your privilege escalation password: "
+        @options[:'sudo-password'] = STDIN.noecho(&:gets).chomp
+        STDERR.puts
+      end
 
       separator "\nRun context:"
       define('-c', '--concurrency CONCURRENCY', Integer,

data/lib/bolt/cli.rb

@@ -37,7 +37,7 @@ module Bolt
                  'secret' => %w[encrypt decrypt createkeys],
                  'inventory' => %w[show],
                  'group' => %w[show],
-                 'project' => %w[init],
+                 'project' => %w[init migrate],
                  'apply' => %w[] }.freeze
 
     attr_reader :config, :options
@@ -129,6 +129,7 @@ module Bolt
       # Logger must be configured before checking path case, otherwise warnings will not display
       @config.check_path_case('modulepath', @config.modulepath)
 
+      parser.warnings.each { |warning| @logger.warn(warning[:msg]) }
       # After validation, initialize inventory and targets. Errors here are better to catch early.
       # After this step
       # options[:target_args] will contain a string/array version of the targetting options this is passed to plans
@@ -367,7 +368,11 @@ module Bolt
 
       case options[:subcommand]
       when 'project'
-        code = initialize_project
+        if options[:action] == 'init'
+          code = initialize_project
+        elsif options[:action] == 'migrate'
+          code = migrate_project
+        end
       when 'plan'
         code = run_plan(options[:object], options[:task_options], options[:target_args], options)
       when 'puppetfile'
@@ -472,13 +477,27 @@ module Bolt
 
     def run_plan(plan_name, plan_arguments, nodes, options)
       unless nodes.empty?
-        if plan_arguments['nodes']
+        if plan_arguments['nodes'] || plan_arguments['targets']
+          key = plan_arguments.include?('nodes') ? 'nodes' : 'targets'
           raise Bolt::CLIError,
-                "A plan's 'nodes' parameter may be specified using the --nodes option, but in that " \
-                "case it must not be specified as a separate nodes=<value> parameter nor included " \
+                "A plan's '#{key}' parameter may be specified using the --#{key} option, but in that " \
+                "case it must not be specified as a separate #{key}=<value> parameter nor included " \
                 "in the JSON data passed in the --params option"
         end
-        plan_arguments['nodes'] = nodes.join(',')
+
+        plan_params = pal.get_plan_info(plan_name)['parameters']
+        target_param = plan_params.dig('targets', 'type') =~ /TargetSpec/
+        node_param = plan_params.include?('nodes')
+
+        if node_param && target_param
+          msg = "Plan parameters include both 'nodes' and 'targets' with type 'TargetSpec', " \
+                "neither will populated with the value for --nodes or --targets."
+          @logger.warn(msg)
+        elsif node_param
+          plan_arguments['nodes'] = nodes.join(',')
+        elsif target_param
+          plan_arguments['targets'] = nodes.join(',')
+        end
       end
 
       plan_context = { plan_name: plan_name,
@@ -507,6 +526,7 @@ module Bolt
     end
 
     def apply_manifest(code, targets, filename = nil, noop = false)
+      Puppet[:tasks] = false
       ast = pal.parse_manifest(code, filename)
 
       executor = Bolt::Executor.new(config.concurrency, analytics, noop)
@@ -559,6 +579,52 @@ module Bolt
       ok ? 0 : 1
     end
 
+    def migrate_project
+      if inventory.version == 2
+        ok = true
+      else
+        inventory_file = config.inventoryfile || config.boltdir.inventory_file
+
+        begin
+          Bolt::Util.file_stat(inventory_file)
+        rescue Errno::ENOENT
+          raise Bolt::FileError.new("The inventory file '#{inventory_file}' does not exist", inventory_file)
+        end
+
+        inv = YAML.safe_load(File.open(inventory_file))
+        migrate_group(inv)
+
+        ok = File.write(inventory_file, { 'version' => 2 }.merge(inv).to_yaml)
+      end
+
+      result = if ok
+                 "Successfully migrated Bolt project to latest version"
+               else
+                 "Could not migrate Bolt project to latest version"
+               end
+      outputter.print_message result
+
+      ok ? 0 : 1
+    end
+
+    # Walks an inventory hash and replaces all 'nodes' keys with 'targets' keys
+    # and all 'name' keys nested in a 'targets' hash with 'uri' keys. Data is
+    # modified in place.
+    def migrate_group(group)
+      if group.key?('nodes')
+        targets = group['nodes'].map do |target|
+          target['uri'] = target.delete('name') if target.is_a?(Hash)
+          target
+        end
+        group.delete('nodes')
+        group['targets'] = targets
+      end
+      (group['groups'] || []).each do |subgroup|
+        migrate_group(subgroup)
+      end
+      nil
+    end
+
     def install_puppetfile(config, puppetfile, modulepath)
       require 'r10k/cli'
       require 'bolt/r10k_log_proxy'

data/lib/bolt/config.rb

@@ -71,7 +71,7 @@ module Bolt
       @save_rerun = true
       @puppetfile_config = {}
       @plugins = {}
-      @plugin_hooks = { 'puppet_library' => { 'plugin' => 'puppet_agent', 'stop_service' => true } }
+      @plugin_hooks = {}
 
       # add an entry for the default console logger
       @log = { 'console' => {} }
@@ -166,30 +166,14 @@ module Bolt
 
       @save_rerun = data['save-rerun'] if data.key?('save-rerun')
 
-      # Plugins are only settable from config not inventory so we can overwrite
       @plugins = data['plugins'] if data.key?('plugins')
-      @plugin_hooks.merge!(data['plugin_hooks']) if data.key?('plugin_hooks')
+      @plugin_hooks = data['plugin_hooks'] if data.key?('plugin_hooks')
 
-      %w[concurrency format puppetdb color transport].each do |key|
+      %w[concurrency format puppetdb color].each do |key|
         send("#{key}=", data[key]) if data.key?(key)
       end
 
-      TRANSPORTS.each do |key, impl|
-        if data[key.to_s]
-          selected = impl.filter_options(data[key.to_s])
-          if @future
-            to_expand = %w[private-key cacert token-file] & selected.keys
-            to_expand.each do |opt|
-              selected[opt] = File.expand_path(selected[opt], @boltdir.path) if opt.is_a?(String)
-            end
-          end
-
-          @transports[key] = Bolt::Util.deep_merge(@transports[key], selected)
-        end
-        if @transports[key]['interpreters']
-          @transports[key]['interpreters'] = normalize_interpreters(@transports[key]['interpreters'])
-        end
-      end
+      update_transports(data)
     end
     private :update_from_file
 
@@ -229,11 +213,28 @@ module Bolt
     end
 
     def update_from_inventory(data)
-      update_from_file(data)
+      update_transports(data)
+    end
 
-      if data['transport']
-        @transport = data['transport']
+    def update_transports(data)
+      TRANSPORTS.each do |key, impl|
+        if data[key.to_s]
+          selected = impl.filter_options(data[key.to_s])
+          if @future
+            to_expand = %w[private-key cacert token-file] & selected.keys
+            to_expand.each do |opt|
+              selected[opt] = File.expand_path(selected[opt], @boltdir.path) if opt.is_a?(String)
+            end
+          end
+
+          @transports[key] = Bolt::Util.deep_merge(@transports[key], selected)
+        end
+        if @transports[key]['interpreters']
+          @transports[key]['interpreters'] = normalize_interpreters(@transports[key]['interpreters'])
+        end
       end
+
+      @transport = data['transport'] if data.key?('transport')
     end
 
     def transport_conf

data/lib/bolt/inventory.rb

@@ -234,7 +234,7 @@ module Bolt
       set_facts(target.name, data['facts']) unless @target_facts[target.name]
       data['features']&.each { |feature| set_feature(target, feature) } unless @target_features[target.name]
       unless @target_plugin_hooks[target.name]
-        set_plugin_hooks(target.name, @config.plugin_hooks.merge(data['plugin_hooks'] || {}))
+        set_plugin_hooks(target.name, (@plugins&.plugin_hooks || {}).merge(data['plugin_hooks'] || {}))
       end
 
       # Use Config object to ensure config section is treated consistently with config file

data/lib/bolt/inventory/target.rb

@@ -78,7 +78,7 @@ module Bolt
       def plugin_hooks
         # Merge plugin_hooks from the config file with any defined by the group
         # or assigned dynamically to the target
-        @inventory.config.plugin_hooks.merge(group_cache['plugin_hooks']).merge(@plugin_hooks)
+        @inventory.plugins.plugin_hooks.merge(group_cache['plugin_hooks']).merge(@plugin_hooks)
       end
 
       def set_config(key_or_key_path, value)

data/lib/bolt/plugin.rb

@@ -39,9 +39,10 @@ module Bolt
     end
 
     class PluginContext
-      def initialize(config, pal)
+      def initialize(config, pal, plugins)
         @pal = pal
         @config = config
+        @plugins = plugins
       end
 
       def serial_executor
@@ -50,7 +51,7 @@ module Bolt
       private :serial_executor
 
       def empty_inventory
-        @empty_inventory ||= Bolt::Inventory.new({}, @config)
+        @empty_inventory ||= Bolt::Inventory::Inventory2.new({}, @config, plugins: @plugins)
       end
       private :empty_inventory
 
@@ -120,28 +121,46 @@ module Bolt
 
     def self.setup(config, pal, pdb_client, analytics)
       plugins = new(config, pal, analytics)
-      # PDB is special do we want to expose the default client to the context?
+
+      # PDB is special because it needs the PDB client. Since it has no config,
+      # we can just add it first.
       plugins.add_plugin(Bolt::Plugin::Puppetdb.new(pdb_client))
 
-      plugins.add_ruby_plugin('Bolt::Plugin::InstallAgent')
-      plugins.add_ruby_plugin('Bolt::Plugin::Task')
-      plugins.add_ruby_plugin('Bolt::Plugin::Pkcs7')
-      plugins.add_ruby_plugin('Bolt::Plugin::Prompt')
+      # Initialize any plugins referenced in config. This will also indirectly
+      # initialize any plugins they depend on.
+      if plugins.reference?(config.plugins)
+        msg = "The 'plugins' setting cannot be set by a plugin reference"
+        raise PluginError.new(msg, 'bolt/plugin-error')
+      end
+
+      config.plugins.keys.each do |plugin|
+        plugins.by_name(plugin)
+      end
+
+      plugins.plugin_hooks.merge!(plugins.resolve_references(config.plugin_hooks))
 
       plugins
     end
 
-    BUILTIN_PLUGINS = %w[task terraform pkcs7 prompt vault aws_inventory puppetdb azure_inventory].freeze
+    RUBY_PLUGINS = %w[install_agent task pkcs7 prompt].freeze
+    BUILTIN_PLUGINS = %w[task terraform pkcs7 prompt vault aws_inventory puppetdb azure_inventory yaml].freeze
+    DEFAULT_PLUGIN_HOOKS = { 'puppet_library' => { 'plugin' => 'puppet_agent', 'stop_service' => true } }.freeze
 
     attr_reader :pal, :plugin_context
+    attr_accessor :plugin_hooks
+
+    private_class_method :new
 
     def initialize(config, pal, analytics)
       @config = config
       @analytics = analytics
-      @plugin_context = PluginContext.new(config, pal)
+      @plugin_context = PluginContext.new(config, pal, self)
       @plugins = {}
       @pal = pal
       @unknown = Set.new
+      @resolution_stack = []
+      @unresolved_plugin_configs = config.plugins.dup
+      @plugin_hooks = DEFAULT_PLUGIN_HOOKS.dup
     end
 
     def modules
@@ -153,11 +172,11 @@ module Bolt
       @plugins[plugin.name] = plugin
     end
 
-    def add_ruby_plugin(cls_name)
-      snake_name = Bolt::Util.class_name_to_file_name(cls_name)
-      require snake_name
-      cls = Kernel.const_get(cls_name)
-      plugin_name = snake_name.split('/').last
+    def add_ruby_plugin(plugin_name)
+      cls_name = Bolt::Util.snake_name_to_class_name(plugin_name)
+      filename = "bolt/plugin/#{plugin_name}"
+      require filename
+      cls = Kernel.const_get("Bolt::Plugin::#{cls_name}")
       opts = {
         context: @plugin_context,
         config: config_for_plugin(plugin_name)
@@ -177,14 +196,18 @@ module Bolt
       add_plugin(plugin)
     end
 
-    def add_from_config
-      @config.plugins.keys.each do |plugin_name|
-        by_name(plugin_name)
-      end
-    end
-
     def config_for_plugin(plugin_name)
-      @config.plugins[plugin_name] || {}
+      return {} unless @unresolved_plugin_configs.include?(plugin_name)
+      if @resolution_stack.include?(plugin_name)
+        msg = "Configuration for plugin '#{plugin_name}' depends on the plugin itself"
+        raise PluginError.new(msg, 'bolt/plugin-error')
+      else
+        @resolution_stack.push(plugin_name)
+        config = resolve_references(@unresolved_plugin_configs[plugin_name])
+        @unresolved_plugin_configs.delete(plugin_name)
+        @resolution_stack.pop
+        config
+      end
     end
 
     def get_hook(plugin_name, hook)
@@ -200,7 +223,9 @@ module Bolt
     def by_name(plugin_name)
       return @plugins[plugin_name] if @plugins.include?(plugin_name)
       begin
-        unless @unknown.include?(plugin_name)
+        if RUBY_PLUGINS.include?(plugin_name)
+          add_ruby_plugin(plugin_name)
+        elsif !@unknown.include?(plugin_name)
           add_module_plugin(plugin_name)
         end
       rescue PluginError::Unknown
@@ -218,6 +243,9 @@ module Bolt
       Bolt::Util.postwalk_vals(data) do |value|
         reference?(value) ? resolve_references(resolve_single_reference(value)) : value
       end
+    rescue SystemStackError
+      raise Bolt::Error.new("Stack depth exceeded while recursively resolving references.",
+                            "bolt/recursive-reference-loop")
     end
 
     # Iteratively resolves "top-level" references until the result no longer

data/lib/bolt/plugin/module.rb

@@ -158,20 +158,21 @@ module Bolt
         # opts are passed directly from inventory but all of the _ options are
         # handled previously. That may not always be the case so filter them
         # out now.
-        _meta, params = opts.partition { |key, _val| key.start_with?('_') }.map(&:to_h)
-        metaparams = {}
+        meta, params = opts.partition { |key, _val| key.start_with?('_') }.map(&:to_h)
+
         # Send config with `_config` when config is defined in bolt_plugin.json
         # Otherwise, merge config with params
         # TODO: remove @send_config when deprecated
         if @send_config
-          metaparams['_config'] = config if config?
+          validate_params(task, params)
+          params['_config'] = config if config?
         else
           params = @config ? config.merge(params) : params
+          validate_params(task, params)
         end
-        metaparams['_boltdir'] = @context.boltdir
+        params['_boltdir'] = @context.boltdir.to_s
 
-        validate_params(task, params)
-        [params, metaparams]
+        [params, meta]
       end
 
       def extract_task_parameter_schema
@@ -199,6 +200,7 @@ module Bolt
       end
 
       def run_task(task, opts)
+        opts = opts.reject { |key, _val| key.start_with?('_') }
         params, metaparams = process_params(task, opts)
         params = params.merge(metaparams)
 
@@ -271,11 +273,11 @@ module Bolt
 
         params, meta_params = process_params(task, opts)
 
-        # our metaparams are meant for the task not the executor
-        params = params.merge(meta_params)
+        options = {}
+        options[:run_as] = meta_params['_run_as'] if meta_params['_run_as']
 
         proc do
-          apply_prep.run_task([target], task, params).first
+          apply_prep.run_task([target], task, params, options).first
         end
       end
     end

data/lib/bolt/plugin/prompt.rb

@@ -20,7 +20,7 @@ module Bolt
 
       def resolve_reference(opts)
         # rubocop:disable Style/GlobalVars
-        $future ? STDERR.print("#{opts['message']}:") : STDOUT.print("#{opts['message']}:")
+        $future ? STDERR.print("#{opts['message']}: ") : STDOUT.print("#{opts['message']}: ")
         value = STDIN.noecho(&:gets).chomp
         $future ? STDERR.puts : STDOUT.puts
         # rubocop:enable Style/GlobalVars

data/lib/bolt/plugin/task.rb

@@ -48,13 +48,15 @@ module Bolt
 
       def puppet_library(opts, target, apply_prep)
         params = opts['parameters'] || {}
+        run_opts = {}
+        run_opts[:run_as] = opts['_run_as'] if opts['_run_as']
         begin
           task = apply_prep.get_task(opts['task'], params)
         rescue Bolt::Error => e
           raise Bolt::Plugin::PluginError::ExecutionError.new(e.message, name, 'puppet_library')
         end
         proc do
-          apply_prep.run_task([target], task, params).first
+          apply_prep.run_task([target], task, params, run_opts).first
         end
       end
     end

data/lib/bolt/util.rb

@@ -204,6 +204,10 @@ module Bolt
         File.stat(File.expand_path(path))
       end
 
+      def snake_name_to_class_name(snake_name)
+        snake_name.split('_').map(&:capitalize).join
+      end
+
       def class_name_to_file_name(cls_name)
         # Note this turns Bolt::CLI -> 'bolt/cli' not 'bolt/c_l_i'
         # this won't handle Bolt::Inventory2Foo

data/lib/bolt/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bolt
 version: !ruby/object:Gem::Version
-  version: 1.37.0
+  version: 1.38.0
 platform: ruby
 authors:
 - Puppet
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2019-11-08 00:00:00.000000000 Z
+date: 2019-11-15 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: addressable