bolt 3.25.0 → 3.26.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fe3e07dae1f3e80d3b4ade7c9d78749f54c592e246b2fd82e6264ab908b8baa1
-  data.tar.gz: 44af49cb408f92496c9795d892bc2e43e04788dd6ead42070a04e258958f00ac
+  metadata.gz: '0591d13a9aa93f3c96585985ffdf1619c29f3f13130cbde98e1b24a1a357b84b'
+  data.tar.gz: 3fe46914478ad07fab90348eb0464a5e45f5c10fa5f921baf3fa089f8d17cad3
 SHA512:
-  metadata.gz: 0d158d8a0d8c72c99a34464ad4de0df9e7b04a83b1509b048c5b650baccbea9a06f99dfd512bfc8bdc91f5065e22148b76814a895766717bfe9f2c34bd81e55f
-  data.tar.gz: eca8af5592551ff9a273ad508831f9a1e07edf0327aa29dcc7078e5fb44743033c92388d79ab22ae2d3c5ea9574984b0c50849544159ab8dd32c2090a0dad939
+  metadata.gz: a12e686a2ce4a3a15f22e2d45f8210eb6211c910e49e59377e37165554ae5f776c4827dfdb93984d8acdd6295cb7c31f49a090a4239ecc2150bff27f2aeb2bbc
+  data.tar.gz: 5f6ccddd83c3026b1a34be7ab439d800f82cb997f179ab9967ead53c87b5141f2419dce5ae0a29ad17fc4257b294856e39aa61916fca4cbb907919520ad8fbf3

data/lib/bolt/inventory/inventory.rb

@@ -125,16 +125,25 @@ module Bolt
       end
 
       # If target is a group name, expand it to the members of that group.
-      # Else match against targets in inventory by name or alias.
+      # Else match against groups and targets in inventory by name or alias.
       # If a wildcard string, error if no matches are found.
       # Else fall back to [target] if no matches are found.
       def resolve_name(target, ext_glob: false)
         if (group = group_lookup[target])
-          group.all_targets
+          group.all_targets.to_a
         else
-          # Try to wildcard match targets in inventory
-          # Ignore case because hostnames are generally case-insensitive
-          targets = groups.all_targets.select { |targ| match_wildcard?(target, targ, ext_glob: ext_glob) }
+          targets = []
+
+          # Find groups that match the glob
+          group_lookup.each do |name, grp|
+            next unless match_wildcard?(target, name, ext_glob: ext_glob)
+            targets += grp.all_targets.to_a
+          end
+
+          # Find target names that match the glob
+          targets += groups.all_targets.select { |targ| match_wildcard?(target, targ, ext_glob: ext_glob) }
+
+          # Find target aliases that match the glob
           targets += groups.target_aliases
                            .select { |tgt_alias, _| match_wildcard?(target, tgt_alias, ext_glob: ext_glob) }
                            .values
@@ -143,7 +152,7 @@ module Bolt
             raise(WildcardError, target) if target.include?('*')
             [target]
           else
-            targets
+            targets.uniq
           end
         end
       end

data/lib/bolt/outputter/human.rb

@@ -194,7 +194,7 @@ module Bolt
               end
             end
           when 'lookup'
-            @stream.puts(indent(2, result['value']))
+            @stream.puts(indent(2, Bolt::Util::Format.stringify(result['value'])))
           else
             if result.generic_value.any?
               @stream.puts(indent(2, ::JSON.pretty_generate(result.generic_value)))
@@ -363,7 +363,7 @@ module Bolt
                  end
         usage << (Bolt::Util.powershell? ? ' [-Noop]' : ' [--noop]') if task.supports_noop
         params.each do |name, data|
-          usage << if data['type']&.start_with?('Optional')
+          usage << if data['type']&.start_with?('Optional') || data.key?('default')
                      " [#{name}=<value>]"
                    else
                      " #{name}=<value>"

data/lib/bolt/plugin/puppetdb.rb

@@ -79,8 +79,9 @@ module Bolt
 
         Bolt::Util.walk_vals(template) do |value|
           # This is done in parts instead of in place so that we only need to
-          # make one puppetDB query
-          if value.is_a?(String)
+          # make one puppetDB query. Don't gather certname since we already
+          # have that and it's not a fact.
+          if value.is_a?(String) && value != 'certname'
             facts << fact_path(value)
           end
           value

data/lib/bolt/puppetdb/client.rb

@@ -110,11 +110,11 @@ module Bolt
         return {} if certnames.empty? || facts.empty?
 
         certnames.uniq!
-        name_query = certnames.map { |c| ["=", "certname", c] }
-        name_query.insert(0, "or")
+        name_query = certnames.each_slice(100).map { |slice| ["in", "certname", ["array", slice]] }
+        name_query.unshift("or")
 
         facts_query = facts.map { |f| ["=", "path", f] }
-        facts_query.insert(0, "or")
+        facts_query.unshift("or")
 
         query = ['and', name_query, facts_query]
 

data/lib/bolt/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: bolt
 version: !ruby/object:Gem::Version
-  version: 3.25.0
+  version: 3.26.1
 platform: ruby
 authors:
 - Puppet
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2022-07-27 00:00:00.000000000 Z
+date: 2022-08-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: addressable