puppet 6.0.5-x64-mingw32 → 8.8.1-x64-mingw32

data/lib/puppet/application/resource.rb

@@ -1,23 +1,29 @@
-require 'puppet/application'
+# frozen_string_literal: true
+
+require_relative '../../puppet/application'
 
 class Puppet::Application::Resource < Puppet::Application
+  environment_mode :not_required
 
   attr_accessor :host, :extra_params
 
   def preinit
-    @extra_params = []
+    @extra_params = [:provider]
   end
 
-  option("--debug","-d")
-  option("--verbose","-v")
-  option("--edit","-e")
-  option("--to_yaml","-y")
+  option("--debug", "-d")
+  option("--verbose", "-v")
+  option("--edit", "-e")
+  option("--to_yaml", "-y")
+  option('--fail', '-f')
 
-  option("--types", "-t") do |arg|
+  option("--types", "-t") do |_arg|
+    env = Puppet.lookup(:environments).get(Puppet[:environment]) || create_default_environment
     types = []
-    Puppet::Type.loadall
+    Puppet::Type.typeloader.loadall(env)
     Puppet::Type.eachtype do |t|
       next if t.name == :component
+
       types << t.name.to_s
     end
     puts types.sort
@@ -33,107 +39,113 @@ class Puppet::Application::Resource < Puppet::Application
   end
 
   def help
-    <<-HELP
+    <<~HELP
+
+      puppet-resource(8) -- #{summary}
+      ========
 
-puppet-resource(8) -- #{summary}
-========
+      SYNOPSIS
+      --------
+      Uses the Puppet RAL to directly interact with the system.
 
-SYNOPSIS
---------
-Uses the Puppet RAL to directly interact with the system.
 
+      USAGE
+      -----
+      puppet resource [-h|--help] [-d|--debug] [-v|--verbose] [-e|--edit]
+        [-p|--param <parameter>] [-t|--types] [-y|--to_yaml] <type>
+        [<name>] [<attribute>=<value> ...]
 
-USAGE
------
-puppet resource [-h|--help] [-d|--debug] [-v|--verbose] [-e|--edit]
-  [-p|--param <parameter>] [-t|--types] [-y|--to_yaml] <type>
-  [<name>] [<attribute>=<value> ...]
 
+      DESCRIPTION
+      -----------
+      This command provides simple facilities for converting current system
+      state into Puppet code, along with some ability to modify the current
+      state using Puppet's RAL.
 
-DESCRIPTION
------------
-This command provides simple facilities for converting current system
-state into Puppet code, along with some ability to modify the current
-state using Puppet's RAL.
+      By default, you must at least provide a type to list, in which case
+      puppet resource will tell you everything it knows about all resources of
+      that type. You can optionally specify an instance name, and puppet
+      resource will only describe that single instance.
 
-By default, you must at least provide a type to list, in which case
-puppet resource will tell you everything it knows about all resources of
-that type. You can optionally specify an instance name, and puppet
-resource will only describe that single instance.
+      If given a type, a name, and a series of <attribute>=<value> pairs,
+      puppet resource will modify the state of the specified resource.
+      Alternately, if given a type, a name, and the '--edit' flag, puppet
+      resource will write its output to a file, open that file in an editor,
+      and then apply the saved file as a Puppet transaction.
 
-If given a type, a name, and a series of <attribute>=<value> pairs,
-puppet resource will modify the state of the specified resource.
-Alternately, if given a type, a name, and the '--edit' flag, puppet
-resource will write its output to a file, open that file in an editor,
-and then apply the saved file as a Puppet transaction.
 
+      OPTIONS
+      -------
+      Note that any setting that's valid in the configuration
+      file is also a valid long argument. For example, 'ssldir' is a valid
+      setting, so you can specify '--ssldir <directory>' as an
+      argument.
 
-OPTIONS
--------
-Note that any setting that's valid in the configuration
-file is also a valid long argument. For example, 'ssldir' is a valid
-setting, so you can specify '--ssldir <directory>' as an
-argument.
+      See the configuration file documentation at
+      https://puppet.com/docs/puppet/latest/configuration.html for the
+      full list of acceptable parameters. A commented list of all
+      configuration options can also be generated by running puppet with
+      '--genconfig'.
 
-See the configuration file documentation at
-https://puppet.com/docs/puppet/latest/configuration.html for the
-full list of acceptable parameters. A commented list of all
-configuration options can also be generated by running puppet with
-'--genconfig'.
+      * --debug:
+        Enable full debugging.
 
-* --debug:
-  Enable full debugging.
+      * --edit:
+        Write the results of the query to a file, open the file in an editor,
+        and read the file back in as an executable Puppet manifest.
 
-* --edit:
-  Write the results of the query to a file, open the file in an editor,
-  and read the file back in as an executable Puppet manifest.
+      * --help:
+        Print this help message.
 
-* --help:
-  Print this help message.
+      * --param:
+        Add more parameters to be outputted from queries.
 
-* --param:
-  Add more parameters to be outputted from queries.
+      * --types:
+        List all available types.
 
-* --types:
-  List all available types.
+      * --verbose:
+        Print extra information.
 
-* --verbose:
-  Print extra information.
+      * --to_yaml:
+        Output found resources in yaml format, suitable to use with Hiera and
+        create_resources.
 
-* --to_yaml:
-  Output found resources in yaml format, suitable to use with Hiera and create_resources.
+      * --fail:
+        Fails and returns an exit code of 1 if the resource could not be modified.
 
-EXAMPLE
--------
-This example uses `puppet resource` to return a Puppet configuration for
-the user `luke`:
+      EXAMPLE
+      -------
+      This example uses `puppet resource` to return a Puppet configuration for
+      the user `luke`:
 
-    $ puppet resource user luke
-    user { 'luke':
-     home => '/home/luke',
-     uid => '100',
-     ensure => 'present',
-     comment => 'Luke Kanies,,,',
-     gid => '1000',
-     shell => '/bin/bash',
-     groups => ['sysadmin','audio','video','puppet']
-    }
+          $ puppet resource user luke
+          user { 'luke':
+           home => '/home/luke',
+           uid => '100',
+           ensure => 'present',
+           comment => 'Luke Kanies,,,',
+           gid => '1000',
+           shell => '/bin/bash',
+           groups => ['sysadmin','audio','video','puppet']
+          }
 
 
-AUTHOR
-------
-Luke Kanies
+      AUTHOR
+      ------
+      Luke Kanies
 
 
-COPYRIGHT
----------
-Copyright (c) 2011 Puppet Inc., LLC Licensed under the Apache 2.0 License
+      COPYRIGHT
+      ---------
+      Copyright (c) 2011 Puppet Inc., LLC Licensed under the Apache 2.0 License
 
     HELP
   end
 
   def main
-    env = Puppet.lookup(:environments).get(Puppet[:environment])
+    # If the specified environment does not exist locally, fall back to the default (production) environment
+    env = Puppet.lookup(:environments).get(Puppet[:environment]) || create_default_environment
+
     Puppet.override(:current_environment => env, :loaders => Puppet::Pops::Loaders.new(env)) do
       type, name, params = parse_args(command_line.args)
 
@@ -141,16 +153,16 @@ Copyright (c) 2011 Puppet Inc., LLC Licensed under the Apache 2.0 License
 
       resources = find_or_save_resources(type, name, params)
 
-    if options[:to_yaml]
-      text = resources.map do |resource|
-        resource.prune_parameters(:parameters_to_include => @extra_params).to_hierayaml.force_encoding(Encoding.default_external)
-      end.join("\n")
-      text.prepend("#{type.downcase}:\n")
-    else
-      text = resources.map do |resource|
-        resource.prune_parameters(:parameters_to_include => @extra_params).to_manifest.force_encoding(Encoding.default_external)
-      end.join("\n")
-    end
+      if options[:to_yaml]
+        data = resources.map do |resource|
+          resource.prune_parameters(:parameters_to_include => @extra_params).to_hiera_hash
+        end.inject(:merge!)
+        text = YAML.dump(type.downcase => data)
+      else
+        text = resources.map do |resource|
+          resource.prune_parameters(:parameters_to_include => @extra_params).to_manifest.force_encoding(Encoding.default_external)
+        end.join("\n")
+      end
 
       options[:edit] ?
         handle_editing(text) :
@@ -180,7 +192,7 @@ Copyright (c) 2011 Puppet Inc., LLC Licensed under the Apache 2.0 License
       tmpfile.puts text
 
       # edit the content
-      system(ENV["EDITOR"] || 'vi', tmpfile.path)
+      system(ENV.fetch("EDITOR", nil) || 'vi', tmpfile.path)
 
       # ...and, now, pass that file to puppet to apply.  Because
       # many editors rename or replace the original file we need to
@@ -199,7 +211,7 @@ Copyright (c) 2011 Puppet Inc., LLC Licensed under the Apache 2.0 License
     params = {}
     args.each do |setting|
       if setting =~ /^(\w+)=(.+)$/
-        params[$1] = $2
+        params[::Regexp.last_match(1)] = ::Regexp.last_match(2)
       else
         raise _("Invalid parameter setting %{setting}") % { setting: setting }
       end
@@ -208,24 +220,39 @@ Copyright (c) 2011 Puppet Inc., LLC Licensed under the Apache 2.0 License
     [type, name, params]
   end
 
+  def create_default_environment
+    Puppet.debug("Specified environment '#{Puppet[:environment]}' does not exist on the filesystem, defaulting to 'production'")
+    Puppet[:environment] = :production
+    basemodulepath = Puppet::Node::Environment.split_path(Puppet[:basemodulepath])
+    modulepath = Puppet[:modulepath]
+    modulepath = (modulepath.nil? || modulepath.empty?) ? basemodulepath : Puppet::Node::Environment.split_path(modulepath)
+    Puppet::Node::Environment.create(Puppet[:environment], modulepath, Puppet::Node::Environment::NO_MANIFEST)
+  end
+
   def find_or_save_resources(type, name, params)
     key = local_key(type, name)
 
-    if name
-      if params.empty?
-        [ Puppet::Resource.indirection.find( key ) ]
+    Puppet.override(stringify_rich: true) do
+      if name
+        if params.empty?
+          [Puppet::Resource.indirection.find(key)]
+        else
+          resource = Puppet::Resource.new(type, name, :parameters => params)
+
+          # save returns [resource that was saved, transaction log from applying the resource]
+          save_result, report = Puppet::Resource.indirection.save(resource, key)
+          status = report.resource_statuses[resource.ref]
+          raise "Failed to manage resource #{resource.ref}" if status&.failed? && options[:fail]
+
+          [save_result]
+        end
       else
-        resource = Puppet::Resource.new( type, name, :parameters => params )
+        if type == "file"
+          raise _("Listing all file instances is not supported.  Please specify a file or directory, e.g. puppet resource file /etc")
+        end
 
-        # save returns [resource that was saved, transaction log from applying the resource]
-        save_result = Puppet::Resource.indirection.save(resource, key)
-        [ save_result.first ]
-      end
-    else
-      if type == "file"
-        raise _("Listing all file instances is not supported.  Please specify a file or directory, e.g. puppet resource file /etc")
+        Puppet::Resource.indirection.search(key, {})
       end
-      Puppet::Resource.indirection.search( key, {} )
     end
   end
 end

data/lib/puppet/application/script.rb

@@ -1,16 +1,17 @@
-require 'puppet/application'
-require 'puppet/configurer'
-require 'puppet/util/profiler/aggregate'
-require 'puppet/parser/script_compiler'
+# frozen_string_literal: true
 
-class Puppet::Application::Script < Puppet::Application
+require_relative '../../puppet/application'
+require_relative '../../puppet/configurer'
+require_relative '../../puppet/util/profiler/aggregate'
+require_relative '../../puppet/parser/script_compiler'
 
-  option("--debug","-d")
-  option("--execute EXECUTE","-e") do |arg|
+class Puppet::Application::Script < Puppet::Application
+  option("--debug", "-d")
+  option("--execute EXECUTE", "-e") do |arg|
     options[:code] = arg
   end
-  option("--test","-t")
-  option("--verbose","-v")
+  option("--test", "-t")
+  option("--verbose", "-v")
 
   option("--logdest LOGDEST", "-l") do |arg|
     handle_logdest_arg(arg)
@@ -21,95 +22,100 @@ class Puppet::Application::Script < Puppet::Application
   end
 
   def help
-    <<-HELP
+    <<~HELP
 
-puppet-script(8) -- #{summary}
-========
+      puppet-script(8) -- #{summary}
+      ========
 
-SYNOPSIS
---------
-Runs a puppet language script without compiling a catalog.
+      SYNOPSIS
+      --------
+      Runs a puppet language script without compiling a catalog.
 
 
-USAGE
------
-puppet script [-h|--help] [-V|--version] [-d|--debug] [-v|--verbose]
-  [-e|--execute]
-  [-l|--logdest syslog|eventlog|<FILE>|console] [--noop]
-  <file>
+      USAGE
+      -----
+      puppet script [-h|--help] [-V|--version] [-d|--debug] [-v|--verbose]
+        [-e|--execute]
+        [-l|--logdest syslog|eventlog|<FILE>|console] [--noop]
+        <file>
 
 
-DESCRIPTION
------------
-This is a standalone puppet script runner tool; use it to run puppet code
-without compiling a catalog.
+      DESCRIPTION
+      -----------
+      This is a standalone puppet script runner tool; use it to run puppet code
+      without compiling a catalog.
 
-When provided with a modulepath, via command line or config file, puppet
-script can load functions, types, tasks and plans from modules.
+      When provided with a modulepath, via command line or config file, puppet
+      script can load functions, types, tasks and plans from modules.
 
-OPTIONS
--------
-Note that any setting that's valid in the configuration
-file is also a valid long argument. For example, 'environment' is a
-valid setting, so you can specify '--environment mytest'
-as an argument.
+      OPTIONS
+      -------
+      Note that any setting that's valid in the configuration
+      file is also a valid long argument. For example, 'environment' is a
+      valid setting, so you can specify '--environment mytest'
+      as an argument.
 
-See the configuration file documentation at
-https://puppet.com/docs/puppet/latest/configuration.html for the
-full list of acceptable parameters. A commented list of all
-configuration options can also be generated by running puppet with
-'--genconfig'.
+      See the configuration file documentation at
+      https://puppet.com/docs/puppet/latest/configuration.html for the
+      full list of acceptable parameters. A commented list of all
+      configuration options can also be generated by running puppet with
+      '--genconfig'.
 
-* --debug:
-  Enable full debugging.
+      * --debug:
+        Enable full debugging.
 
-* --help:
-  Print this help message
+      * --help:
+        Print this help message
 
 
-* --logdest:
-  Where to send log messages. Choose between 'syslog' (the POSIX syslog
-  service), 'eventlog' (the Windows Event Log), 'console', or the path to a log
-  file. Defaults to 'console'.
+      * --logdest:
+        Where to send log messages. Choose between 'syslog' (the POSIX syslog
+        service), 'eventlog' (the Windows Event Log), 'console', or the path to a log
+        file. Defaults to 'console'.
+        Multiple destinations can be set using a comma separated list
+        (eg: `/path/file1,console,/path/file2`)"
 
-  A path ending with '.json' will receive structured output in JSON format. The
-  log file will not have an ending ']' automatically written to it due to the
-  appending nature of logging. It must be appended manually to make the content
-  valid JSON.
+        A path ending with '.json' will receive structured output in JSON format. The
+        log file will not have an ending ']' automatically written to it due to the
+        appending nature of logging. It must be appended manually to make the content
+        valid JSON.
 
-* --noop:
-  Use 'noop' mode where Puppet runs in a no-op or dry-run mode. This
-  is useful for seeing what changes Puppet will make without actually
-  executing the changes. Applies to tasks only.
+        A path ending with '.jsonl' will receive structured output in JSON Lines
+        format.
 
-* --execute:
-  Execute a specific piece of Puppet code
+      * --noop:
+        Use 'noop' mode where Puppet runs in a no-op or dry-run mode. This
+        is useful for seeing what changes Puppet will make without actually
+        executing the changes. Applies to tasks only.
 
-* --verbose:
-  Print extra information.
+      * --execute:
+        Execute a specific piece of Puppet code
 
-EXAMPLE
--------
-    $ puppet script -l /tmp/manifest.log manifest.pp
-    $ puppet script --modulepath=/root/dev/modules -e 'notice("hello world")'
+      * --verbose:
+        Print extra information.
 
+      EXAMPLE
+      -------
+          $ puppet script -l /tmp/manifest.log manifest.pp
+          $ puppet script --modulepath=/root/dev/modules -e 'notice("hello world")'
 
-AUTHOR
-------
-Henrik Lindberg
 
+      AUTHOR
+      ------
+      Henrik Lindberg
 
-COPYRIGHT
----------
-Copyright (c) 2017 Puppet Inc., LLC Licensed under the Apache 2.0 License
+
+      COPYRIGHT
+      ---------
+      Copyright (c) 2017 Puppet Inc., LLC Licensed under the Apache 2.0 License
 
     HELP
   end
 
   def app_defaults
     super.merge({
-      :default_file_terminus => :file_server,
-    })
+                  :default_file_terminus => :file_server,
+                })
   end
 
   def run_command
@@ -132,23 +138,22 @@ Copyright (c) 2017 Puppet Inc., LLC Licensed under the Apache 2.0 License
     else
       manifest = command_line.args.shift
       raise _("Could not find file %{manifest}") % { manifest: manifest } unless Puppet::FileSystem.exist?(manifest)
+
       Puppet.warning(_("Only one file can be used per run. Skipping %{files}") % { files: command_line.args.join(', ') }) if command_line.args.size > 0
     end
 
     unless Puppet[:node_name_fact].empty?
       # Collect the facts specified for that node
-      unless facts = Puppet::Node::Facts.indirection.find(Puppet[:node_name_value])
-        raise _("Could not find facts for %{node}") % { node: Puppet[:node_name_value] }
-      end
+      facts = Puppet::Node::Facts.indirection.find(Puppet[:node_name_value])
+      raise _("Could not find facts for %{node}") % { node: Puppet[:node_name_value] } unless facts
 
       Puppet[:node_name_value] = facts.values[Puppet[:node_name_fact]]
       facts.name = Puppet[:node_name_value]
     end
 
     # Find the Node
-    unless node = Puppet::Node.indirection.find(Puppet[:node_name_value])
-      raise _("Could not find node %{node}") % { node: Puppet[:node_name_value] }
-    end
+    node = Puppet::Node.indirection.find(Puppet[:node_name_value])
+    raise _("Could not find node %{node}") % { node: Puppet[:node_name_value] } unless node
 
     configured_environment = node.environment || Puppet.lookup(:current_environment)
 
@@ -164,7 +169,7 @@ Copyright (c) 2017 Puppet Inc., LLC Licensed under the Apache 2.0 License
     node.environment = apply_environment
 
     # TRANSLATION, the string "For puppet script" is not user facing
-    Puppet.override({:current_environment => apply_environment}, "For puppet script") do
+    Puppet.override({ :current_environment => apply_environment }, "For puppet script") do
       # Merge in the facts.
       node.merge(facts.values) if facts
 
@@ -208,9 +213,8 @@ Copyright (c) 2017 Puppet Inc., LLC Licensed under the Apache 2.0 License
           topscope.merge_settings(node.environment.name, false)
 
           compiler.compile()
-
-        rescue Puppet::ParseErrorWithIssue, Puppet::Error
-          # already logged and handled by the compiler for these two cases
+        rescue Puppet::Error
+          # already logged and handled by the compiler, including Puppet::ParseErrorWithIssue
           exit(1)
         end
 
@@ -220,7 +224,6 @@ Copyright (c) 2017 Puppet Inc., LLC Licensed under the Apache 2.0 License
         exit(1)
       end
     end
-
   ensure
     if @profiler
       Puppet::Util::Profiler.remove_profiler(@profiler)
@@ -244,7 +247,7 @@ Copyright (c) 2017 Puppet Inc., LLC Licensed under the Apache 2.0 License
     # installed where a settings catalog have already been applied...
     # This saves 1/5th of the startup time
 
-#    Puppet.settings.use :main, :agent, :ssl
+    #    Puppet.settings.use :main, :agent, :ssl
 
     # When running a script, the catalog is not relevant, and neither is caching of it
     Puppet::Resource::Catalog.indirection.cache_class = nil