chef 14.0.202 → 14.1.1

data/lib/chef/resource.rb

@@ -515,15 +515,16 @@ class Chef
     #
     # Whether to ignore failures.  If set to `true`, and this resource when an
     # action is run, the resource will be marked as failed but no exception will
-    # be thrown (and no error will be output).  Defaults to `false`.
+    # be thrown (and no error will be output).  Defaults to `false`. If set to
+    # `:quiet` or `'quiet'`, the normal error trace will be suppressed.
     #
     # TODO ignore_failure and retries seem to be mutually exclusive; I doubt
     # that was intended.
     #
-    # @param arg [Boolean] Whether to ignore failures.
+    # @param arg [Boolean, String, Symbol] Whether to ignore failures.
     # @return Whether this resource will ignore failures.
     #
-    property :ignore_failure, [ TrueClass, FalseClass ], default: false, desired_state: false
+    property :ignore_failure, [ true, false, :quiet, "quiet" ], default: false, desired_state: false
 
     #
     # Make this resource into an exact (shallow) copy of the other resource.

data/lib/chef/resource/apt_preference.rb

@@ -25,9 +25,7 @@ class Chef
       resource_name :apt_preference
       provides(:apt_preference) { true }
 
-      description "The apt_preference resource allows for the creation of APT preference"\
-                  " files. Preference files are used to control which package versions and"\
-                  " sources are prioritized during installation."
+      description "The apt_preference resource allows for the creation of APT preference files. Preference files are used to control which package versions and sources are prioritized during installation."
       introduced "13.3"
 
       property :package_name, String,

data/lib/chef/resource/apt_repository.rb

@@ -24,8 +24,7 @@ class Chef
       resource_name :apt_repository
       provides(:apt_repository) { true }
 
-      description "Use the apt_repository resource to specify additional APT repositories."\
-                  " Adding a new repository will update APT package cache immediately."
+      description "Use the apt_repository resource to specify additional APT repositories. Adding a new repository will update the APT package cache immediately."
       introduced "12.9"
 
       # There's a pile of [ String, nil, FalseClass ] types in these properties.
@@ -35,22 +34,50 @@ class Chef
       # to allow that so don't refactor this however tempting it is
       property :repo_name, String,
                regex: [/^[^\/]+$/],
+               description: "The name of the repository to configure, if it differs from the name of the resource block. The value of this setting must not contain spaces.",
                validation_message: "repo_name property cannot contain a forward slash '/'",
                name_property: true
 
-      property :uri, String
-      property :distribution, [ String, nil, FalseClass ], default: lazy { node["lsb"]["codename"] }
-      property :components, Array, default: lazy { [] }
-      property :arch, [String, nil, FalseClass]
-      property :trusted, [TrueClass, FalseClass], default: false
-      # whether or not to add the repository as a source repo, too
-      property :deb_src, [TrueClass, FalseClass], default: false
-      property :keyserver, [String, nil, FalseClass], default: "keyserver.ubuntu.com"
-      property :key, [String, Array, nil, FalseClass], default: lazy { [] }, coerce: proc { |x| x ? Array(x) : x }
-      property :key_proxy, [String, nil, FalseClass]
-
-      property :cookbook, [String, nil, FalseClass], desired_state: false
-      property :cache_rebuild, [TrueClass, FalseClass], default: true, desired_state: false
+      property :uri, String,
+               description: "The base of the Debian distribution."
+
+      property :distribution, [ String, nil, FalseClass ],
+               description: "Usually a distribution’s codename, such as trusty, xenial or bionic. Default value: the codename of the node’s distro.",
+               default: lazy { node["lsb"]["codename"] }
+
+      property :components, Array,
+               description: "Package groupings, such as ‘main’ and ‘stable’.",
+               default: lazy { [] }
+
+      property :arch, [String, nil, FalseClass],
+               description: "Constrain packages to a particular CPU architecture such as 'i386' or 'amd64'."
+
+      property :trusted, [TrueClass, FalseClass],
+               description: "Determines whether you should treat all packages from this repository as authenticated regardless of signature.",
+               default: false
+
+      property :deb_src, [TrueClass, FalseClass],
+               description: "Determines whether or not to add the repository as a source repo as well.",
+               default: false
+
+      property :keyserver, [String, nil, FalseClass],
+               description: "The GPG keyserver where the key for the repo should be retrieved.",
+               default: "keyserver.ubuntu.com"
+
+      property :key, [String, Array, nil, FalseClass],
+               description: "If a keyserver is provided, this is assumed to be the fingerprint; otherwise it can be either the URI of GPG key for the repo, or a cookbook_file.",
+               default: lazy { [] }, coerce: proc { |x| x ? Array(x) : x }
+
+      property :key_proxy, [String, nil, FalseClass],
+               description: "If set, a specified proxy is passed to GPG via http-proxy=."
+
+      property :cookbook, [String, nil, FalseClass],
+               description: "If key should be a cookbook_file, specify a cookbook where the key is located for files/default. Default value is nil, so it will use the cookbook where the resource is used.",
+               desired_state: false
+
+      property :cache_rebuild, [TrueClass, FalseClass],
+               description: "Determines whether to rebuild the APT package cache.",
+               default: true, desired_state: false
 
       default_action :add
       allowed_actions :add, :remove

data/lib/chef/resource/apt_update.rb

@@ -29,7 +29,9 @@ class Chef
 
       # allow bare apt_update with no name
       property :name, String, default: ""
-      property :frequency, Integer, default: 86_400
+      property :frequency, Integer,
+                description: "Determines how frequently (in seconds) APT repository updates are made. Use this property when the :periodic action is specified.",
+                default: 86_400
 
       default_action :periodic
       allowed_actions :update, :periodic

data/lib/chef/resource/bash.rb

@@ -22,12 +22,7 @@ require "chef/provider/script"
 class Chef
   class Resource
     class Bash < Chef::Resource::Script
-      description "Use the bash resource to execute scripts using the Bash interpreter."\
-                  " This resource may also use any of the actions and properties that are"\
-                  " available to the execute resource. Commands that are executed with this"\
-                  " resource are (by their nature) not idempotent, as they are typically"\
-                  " unique to the environment in which they are run. Use not_if and only_if"\
-                  " to guard this resource for idempotence."
+      description "Use the bash resource to execute scripts using the Bash interpreter. This resource may also use any of the actions and properties that are available to the execute resource. Commands that are executed with this resource are (by their nature) not idempotent, as they are typically unique to the environment in which they are run. Use not_if and only_if to guard this resource for idempotence."
 
       def initialize(name, run_context = nil)
         super

data/lib/chef/resource/batch.rb

@@ -23,13 +23,7 @@ class Chef
     class Batch < Chef::Resource::WindowsScript
       provides :batch
 
-      description "Use the batch resource to execute a batch script using the cmd.exe"\
-                  " interpreter on Windows. The batch resource creates and executes a"\
-                  " temporary file (similar to how the script resource behaves), rather"\
-                  " than running the command inline. Commands that are executed with this"\
-                  " resource are (by their nature) not idempotent, as they are typically"\
-                  " unique to the environment in which they are run. Use not_if and only_if"\
-                  " to guard this resource for idempotence."
+      description "Use the batch resource to execute a batch script using the cmd.exe interpreter on Windows. The batch resource creates and executes a temporary file (similar to how the script resource behaves), rather than running the command inline. Commands that are executed with this resource are (by their nature) not idempotent, as they are typically unique to the environment in which they are run. Use not_if and only_if to guard this resource for idempotence."
 
       def initialize(name, run_context = nil)
         super(name, run_context, nil, "cmd.exe")

data/lib/chef/resource/bff_package.rb

@@ -24,10 +24,7 @@ class Chef
       resource_name :bff_package
       provides :bff_package
 
-      description "Use the bff_package resource to manage packages for the AIX platform"\
-                  " using the installp utility. When a package is installed from a local"\
-                  " file, it must be added to the node using the remote_file or cookbook_file"\
-                  " resources."
+      description "Use the bff_package resource to manage packages for the AIX platform using the installp utility. When a package is installed from a local file, it must be added to the node using the remote_file or cookbook_file resources."
       introduced "12.0"
     end
   end

data/lib/chef/resource/breakpoint.rb

@@ -24,13 +24,7 @@ class Chef
       provides :breakpoint
       resource_name :breakpoint
 
-      description "Use the breakpoint resource to add breakpoints to recipes. Run the"\
-                  " chef-shell in chef-client mode, and then use those breakpoints to debug"\
-                  " recipes. Breakpoints are ignored by the chef-client during an actual"\
-                  " chef-client run. That said, breakpoints are typically used to debug"\
-                  " recipes only when running them in a non-production environment, after"\
-                  " which they are removed from those recipes before the parent cookbook is"\
-                  " uploaded to the Chef server."
+      description "Use the breakpoint resource to add breakpoints to recipes. Run the chef-shell in chef-client mode, and then use those breakpoints to debug recipes. Breakpoints are ignored by the chef-client during an actual chef-client run. That said, breakpoints are typically used to debug recipes only when running them in a non-production environment, after which they are removed from those recipes before the parent cookbook is uploaded to the Chef server."
       introduced "12.0"
 
       default_action :break

data/lib/chef/resource/build_essential.rb

@@ -22,9 +22,12 @@ class Chef
       resource_name :build_essential
       provides(:build_essential) { true }
 
-      description "Use the build_essential resource to install packages required for compiling C software from source"
+      description "Use the build_essential resource to install packages required for compiling C software from source."
       introduced "14.0"
 
+      # this allows us to use build_essential without setting a name
+      property :name, String, default: ""
+
       property :compile_time, [TrueClass, FalseClass],
                description: "Install build essential packages at compile time.",
                default: false

data/lib/chef/resource/cab_package.rb

@@ -27,13 +27,13 @@ class Chef
       resource_name :cab_package
       provides :cab_package
 
-      description "Use the cab_package resource to install or remove Microsoft Windows"\
-                  " cabinet (.cab) packages."
+      description "Use the cab_package resource to install or remove Microsoft Windows cabinet (.cab) packages."
       introduced "12.15"
 
       allowed_actions :install, :remove
 
       property  :source, String,
+                description: "The local file path or URL for the CAB package.",
                 coerce: (proc do |s|
                   unless s.nil?
                     uri_scheme?(s) ? s : Chef::Util::PathHelper.canonical_path(s, false)

data/lib/chef/resource/chocolatey_package.rb

@@ -24,18 +24,26 @@ class Chef
       resource_name :chocolatey_package
       provides :chocolatey_package
 
-      description "Use the chocolatey_package resource to manage packages using Chocolatey"\
-                  " on the Microsoft Windows platform."
+      description "Use the chocolatey_package resource to manage packages using Chocolatey on the Microsoft Windows platform."
       introduced "12.7"
 
       allowed_actions :install, :upgrade, :remove, :purge, :reconfig
 
       # windows can't take Array options yet
-      property :options, String
+      property :options, String,
+                description: "One (or more) additional options that are passed to the command."
 
-      property :package_name, [String, Array], coerce: proc { |x| [x].flatten }
-      property :version, [String, Array], coerce: proc { |x| [x].flatten }
-      property :returns, [Integer, Array], default: [ 0 ], desired_state: false
+      property :package_name, [String, Array],
+                description: "The name of the package. Default value: the name of the resource block See “Syntax” section above for more information.",
+                coerce: proc { |x| [x].flatten }
+
+      property :version, [String, Array],
+                description: "The version of a package to be installed or upgraded.",
+                coerce: proc { |x| [x].flatten }
+
+      property :returns, [Integer, Array],
+                description: "The exit code(s) returned a chocolatey package that indicate success.",
+                default: [ 0 ], desired_state: false
     end
   end
 end

data/lib/chef/resource/cookbook_file.rb

@@ -29,20 +29,14 @@ class Chef
 
       resource_name :cookbook_file
 
-      description "Use the cookbook_file resource to transfer files from a sub-directory"\
-                  " of COOKBOOK_NAME/files/ to a specified path located on a host that is"\
-                  " running the chef-client. The file is selected according to file specificity,"\
-                  " which allows different source files to be used based on the hostname, host"\
-                  " platform (operating system, distro, or as appropriate), or platform version."\
-                  " Files that are located in the COOKBOOK_NAME/files/default sub-directory may be"\
-                  " used on any platform.\n\n"\
-                  "During a chef-client run, the checksum for each local file is calculated and then"\
-                  " compared against the checksum for the same file as it currently exists in the"\
-                  " cookbook on the Chef server. A file is not transferred when the checksums match."\
-                  " Only files that require an update are transferred from the Chef server to a node."
+      description "Use the cookbook_file resource to transfer files from a sub-directory of COOKBOOK_NAME/files/ to a specified path located on a host that is running the chef-client. The file is selected according to file specificity, which allows different source files to be used based on the hostname, host platform (operating system, distro, or as appropriate), or platform version. Files that are located in the COOKBOOK_NAME/files/default sub-directory may be used on any platform.\n\nDuring a chef-client run, the checksum for each local file is calculated and then compared against the checksum for the same file as it currently exists in the cookbook on the Chef server. A file is not transferred when the checksums match. Only files that require an update are transferred from the Chef server to a node."
 
-      property :source, [ String, Array ], default: lazy { ::File.basename(name) }
-      property :cookbook, String
+      property :source, [ String, Array ],
+                description: "The name of the file in COOKBOOK_NAME/files/default or the path to a file located in COOKBOOK_NAME/files. The path must include the file name and its extension. This can be used to distribute specific files depending upon the platform used.",
+                default: lazy { ::File.basename(name) }
+
+      property :cookbook, String,
+                description: "The cookbook in which a file is located (if it is not located in the current cookbook)."
 
       default_action :create
     end

data/lib/chef/resource/cron.rb

@@ -26,9 +26,7 @@ class Chef
       resource_name :cron
       provides :cron
 
-      description "Use the cron resource to manage cron entries for time-based job scheduling."\
-                  " Properties for a schedule will default to * if not provided. The cron resource"\
-                  " requires access to a crontab program, typically cron."
+      description "Use the cron resource to manage cron entries for time-based job scheduling. Properties for a schedule will default to * if not provided. The cron resource requires access to a crontab program, typically cron."
 
       state_attrs :minute, :hour, :day, :month, :weekday, :user
 

data/lib/chef/resource/launchd.rb

@@ -31,8 +31,8 @@ class Chef
       allowed_actions :create, :create_if_missing, :delete, :enable, :disable, :restart
 
       property :label, String, identity: true, name_property: true
-      property :backup, [Integer, FalseClass]
-      property :cookbook, String
+      property :backup, [Integer, FalseClass], desired_state: false
+      property :cookbook, String, desired_state: false
       property :group, [String, Integer]
       property :plist_hash, Hash
       property :mode, [String, Integer]

data/lib/chef/resource/link.rb

@@ -47,11 +47,25 @@ class Chef
         super
       end
 
-      property :target_file, String, name_property: true, identity: true
-      property :to, String
-      property :link_type, [String, Symbol], coerce: proc { |arg| arg.kind_of?(String) ? arg.to_sym : arg }, equal_to: [ :symbolic, :hard ], default: :symbolic
-      property :group, [String, Integer], regex: [Chef::Config[:group_valid_regex]]
-      property :user, [String, Integer], regex: [Chef::Config[:user_valid_regex]]
+      property :target_file, String,
+               description: "The name of the link. Default value: the name of the resource block See “Syntax” section above for more information.",
+               name_property: true, identity: true
+
+      property :to, String,
+               description: "The actual file to which the link is to be created."
+
+      property :link_type, [String, Symbol],
+               description: "The type of link: :symbolic or :hard.",
+               coerce: proc { |arg| arg.kind_of?(String) ? arg.to_sym : arg },
+               equal_to: [ :symbolic, :hard ], default: :symbolic
+
+      property :group, [String, Integer],
+               description: "A string or ID that identifies the group associated with a symbolic link.",
+               regex: [Chef::Config[:group_valid_regex]]
+
+      property :owner, [String, Integer],
+               description: "The owner associated with a symbolic link.",
+               regex: [Chef::Config[:user_valid_regex]]
 
       # make link quack like a file (XXX: not for public consumption)
       def path

data/lib/chef/resource/ohai_hint.rb

@@ -33,7 +33,7 @@ class Chef
 
       property :compile_time, [TrueClass, FalseClass],
                description: "Should the resource execute during the compile time phase",
-               default: true
+               default: true, desired_state: false
 
       action :create do
         description "Create an Ohai hint file"

data/lib/chef/resource/openssl_rsa_private_key.rb

@@ -65,7 +65,7 @@ class Chef
 
       property :force, [TrueClass, FalseClass],
                description: "Force creating the key even if the existing key exists.",
-               default: false
+               default: false, desired_state: false
 
       action :create do
         return if new_resource.force || priv_key_file_valid?(new_resource.path, new_resource.key_pass)