chef 18.5.0 → 18.7.3

data/lib/chef/resource/homebrew_cask.rb

@@ -18,7 +18,7 @@
 #
 
 require_relative "../resource"
-require_relative "../mixin/homebrew_user"
+require_relative "../mixin/homebrew"
 
 class Chef
   class Resource
@@ -29,7 +29,7 @@ class Chef
       description "Use the **homebrew_cask** resource to install binaries distributed via the Homebrew package manager."
       introduced "14.0"
 
-      include Chef::Mixin::HomebrewUser
+      include Chef::Mixin::Homebrew
 
       property :cask_name, String,
         description: "An optional property to set the cask name if it differs from the resource block's name.",
@@ -40,10 +40,6 @@ class Chef
       property :options, String,
         description: "Options to pass to the brew command during installation."
 
-      property :install_cask, [TrueClass, FalseClass],
-        description: "Automatically install the Homebrew cask tap, if necessary.",
-        default: true
-
       property :homebrew_path, String,
         description: "The path to the Homebrew binary."
 
@@ -53,37 +49,27 @@ class Chef
         default_description: "Calculated default username"\
 
       action :install, description: "Install an application that is packaged as a Homebrew cask." do
-        if new_resource.install_cask
-          homebrew_tap "homebrew/cask" do
-            homebrew_path homebrew_bin_path(new_resource.homebrew_path)
-            owner new_resource.owner
-          end
-        end
-
         unless casked?
           converge_by("install cask #{new_resource.cask_name} #{new_resource.options}") do
-            shell_out!("#{homebrew_bin_path(new_resource.homebrew_path)} install --cask #{new_resource.cask_name} #{new_resource.options}",
-              user: new_resource.owner,
-              env:  { "HOME" => ::Dir.home(new_resource.owner), "USER" => new_resource.owner },
-              cwd: ::Dir.home(new_resource.owner))
+            execute "install cask #{new_resource.cask_name}" do
+              command "#{homebrew_bin_path(new_resource.homebrew_path)} install --cask #{new_resource.cask_name} #{new_resource.options}"
+              user new_resource.owner
+              cwd ::Dir.home(new_resource.owner)
+              login true
+            end
           end
         end
       end
 
       action :remove, description: "Remove an application that is packaged as a Homebrew cask." do
-        if new_resource.install_cask
-          homebrew_tap "homebrew/cask" do
-            homebrew_path homebrew_bin_path(new_resource.homebrew_path)
-            owner new_resource.owner
-          end
-        end
-
         if casked?
           converge_by("uninstall cask #{new_resource.cask_name}") do
-            shell_out!("#{homebrew_bin_path(new_resource.homebrew_path)} uninstall --cask #{new_resource.cask_name}",
-              user: new_resource.owner,
-              env:  { "HOME" => ::Dir.home(new_resource.owner), "USER" => new_resource.owner },
-              cwd: ::Dir.home(new_resource.owner))
+            execute "uninstall cask #{new_resource.cask_name}" do
+              command "#{homebrew_bin_path(new_resource.homebrew_path)} uninstall --cask #{new_resource.cask_name}"
+              user new_resource.owner
+              cwd ::Dir.home(new_resource.owner)
+              login true
+            end
           end
         end
       end
@@ -98,10 +84,13 @@ class Chef
         # @return [Boolean]
         def casked?
           unscoped_name = new_resource.cask_name.split("/").last
-          shell_out!("#{homebrew_bin_path(new_resource.homebrew_path)} list --cask 2>/dev/null",
+          shell_out!(
+            "#{homebrew_bin_path(new_resource.homebrew_path)} list --cask 2>/dev/null",
             user: new_resource.owner,
             env:  { "HOME" => ::Dir.home(new_resource.owner), "USER" => new_resource.owner },
-            cwd: ::Dir.home(new_resource.owner)).stdout.split.include?(unscoped_name)
+            cwd: ::Dir.home(new_resource.owner),
+            login: true
+          ).stdout.split.include?(unscoped_name)
         end
       end
     end

data/lib/chef/resource/homebrew_tap.rb

@@ -18,7 +18,7 @@
 #
 
 require_relative "../resource"
-require_relative "../mixin/homebrew_user"
+require_relative "../mixin/homebrew"
 
 class Chef
   class Resource
@@ -29,7 +29,7 @@ class Chef
       description "Use the **homebrew_tap** resource to add additional formula repositories to the Homebrew package manager."
       introduced "14.0"
 
-      include Chef::Mixin::HomebrewUser
+      include Chef::Mixin::Homebrew
 
       property :tap_name, String,
         description: "An optional property to set the tap name if it differs from the resource block's name.",
@@ -51,10 +51,13 @@ class Chef
       action :tap, description: "Add a Homebrew tap." do
         unless tapped?(new_resource.tap_name)
           converge_by("tap #{new_resource.tap_name}") do
-            shell_out!("#{homebrew_bin_path(new_resource.homebrew_path)} tap #{new_resource.tap_name} #{new_resource.url || ""}",
-              user: new_resource.owner,
-              env:  { "HOME" => ::Dir.home(new_resource.owner), "USER" => new_resource.owner },
-              cwd: ::Dir.home(new_resource.owner))
+            execute "tap #{new_resource.tap_name}" do
+              command "#{homebrew_bin_path(new_resource.homebrew_path)} tap #{new_resource.tap_name} #{new_resource.url || ""}"
+              user new_resource.owner
+              default_env true
+              cwd ::Dir.home(new_resource.owner)
+              login true
+            end
           end
         end
       end
@@ -62,21 +65,33 @@ class Chef
       action :untap, description: "Remove a Homebrew tap." do
         if tapped?(new_resource.tap_name)
           converge_by("untap #{new_resource.tap_name}") do
-            shell_out!("#{homebrew_bin_path(new_resource.homebrew_path)} untap #{new_resource.tap_name}",
-              user: new_resource.owner,
-              env:  { "HOME" => ::Dir.home(new_resource.owner), "USER" => new_resource.owner },
-              cwd: ::Dir.home(new_resource.owner))
+            execute "untap #{new_resource.tap_name}" do
+              command "#{homebrew_bin_path(new_resource.homebrew_path)} untap #{new_resource.tap_name}"
+              user new_resource.owner
+              default_env true
+              cwd ::Dir.home(new_resource.owner)
+              login true
+            end
           end
         end
       end
 
-      # Is the passed tap already tapped
-      #
-      # @return [Boolean]
-      def tapped?(name)
-        base_path = ["#{::File.dirname(which("brew"))}/../homebrew", "#{::File.dirname(which("brew"))}/../Homebrew", "/opt/homebrew", "/usr/local/Homebrew", "/home/linuxbrew/.linuxbrew"].uniq.select { |x| Dir.exist?(x) }.first
-        tap_dir = name.gsub("/", "/homebrew-")
-        ::File.directory?("#{base_path}/Library/Taps/#{tap_dir}")
+      action_class do
+        # Is the passed tap already tapped
+        #
+        # @return [Boolean]
+        def tapped?(name)
+          brew_path = ::File.dirname(homebrew_bin_path(new_resource.homebrew_path))
+          base_path = [
+            "#{brew_path}/../homebrew",
+            "#{brew_path}/../Homebrew",
+            "/opt/homebrew",
+            "/usr/local/Homebrew",
+            "/home/linuxbrew/.linuxbrew",
+          ].filter_map { |x| x if Dir.exist?(x) }.first
+          tap_dir = name.gsub("/", "/homebrew-")
+          ::File.directory?("#{base_path}/Library/Taps/#{tap_dir}")
+        end
       end
     end
   end

data/lib/chef/resource/homebrew_update.rb

@@ -19,13 +19,13 @@
 #
 
 require_relative "../resource"
-require_relative "../mixin/homebrew_user"
+require_relative "../mixin/homebrew"
 require "chef-utils/dist" unless defined?(ChefUtils::Dist)
 
 class Chef
   class Resource
     class HomebrewUpdate < Chef::Resource
-      include Chef::Mixin::HomebrewUser
+      include Chef::Mixin::Homebrew
 
       provides(:homebrew_update) { true }
 
@@ -78,9 +78,9 @@ class Chef
           end
 
           execute "brew update" do
-            command %w{brew update}
-            default_env true
+            command "#{homebrew_bin_path} update"
             user find_homebrew_uid
+            login true
             notifies :touch, "file[#{BREW_STAMP}]", :immediately
           end
         end

data/lib/chef/resource/hostname.rb

@@ -24,7 +24,7 @@ class Chef
     # @since 14.0.0
     class Hostname < Chef::Resource
 
-      provides :hostname, target_mode: true
+      provides :hostname
 
       description "Use the **hostname** resource to set the system's hostname, configure hostname and hosts config file, and re-run the Ohai hostname plugin so the hostname will be available in subsequent cookbooks."
       introduced "14.0"
@@ -99,7 +99,7 @@ class Chef
 
       action_class do
         def append_replacing_matching_lines(path, regex, string)
-          text = TargetIO::IO.read(path).split("\n")
+          text = IO.read(path).split("\n")
           text.reject! { |s| s =~ regex }
           text += [ string ]
           file path do
@@ -107,7 +107,7 @@ class Chef
             owner "root"
             group node["root_group"]
             mode "0644"
-            not_if { TargetIO::IO.read(path).split("\n").include?(string) }
+            not_if { IO.read(path).split("\n").include?(string) }
           end
         end
 
@@ -181,14 +181,14 @@ class Chef
             end
           when linux?
             case
-            when ::TargetIO::File.exist?("/usr/bin/hostnamectl") && !docker?
+            when ::File.exist?("/usr/bin/hostnamectl") && !docker?
               # use hostnamectl whenever we find it on linux (as systemd takes over the world)
               # this must come before other methods like /etc/hostname and /etc/sysconfig/network
               execute "hostnamectl set-hostname #{new_resource.hostname}" do
                 notifies :reload, "ohai[reload hostname]"
                 not_if { shell_out!("hostnamectl status", returns: [0, 1]).stdout =~ /Static hostname:\s*#{new_resource.hostname}\s*$/ }
               end
-            when ::TargetIO::File.exist?("/etc/hostname")
+            when ::File.exist?("/etc/hostname")
               # debian family uses /etc/hostname
               # arch also uses /etc/hostname
               # the "platform: iox_xr, platform_family: wrlinux, os: linux" platform also hits this
@@ -201,10 +201,10 @@ class Chef
                 group node["root_group"]
                 mode "0644"
               end
-            when ::TargetIO::File.file?("/etc/sysconfig/network")
+            when ::File.file?("/etc/sysconfig/network")
               # older non-systemd RHEL/Fedora derived
               append_replacing_matching_lines("/etc/sysconfig/network", /^HOSTNAME\s*=/, "HOSTNAME=#{new_resource.hostname}")
-            when ::TargetIO::File.exist?("/etc/HOSTNAME")
+            when ::File.exist?("/etc/HOSTNAME")
               # SuSE/openSUSE uses /etc/HOSTNAME
               file "/etc/HOSTNAME" do
                 content "#{new_resource.hostname}\n"
@@ -212,7 +212,7 @@ class Chef
                 group node["root_group"]
                 mode "0644"
               end
-            when ::TargetIO::File.exist?("/etc/conf.d/hostname")
+            when ::File.exist?("/etc/conf.d/hostname")
               # Gentoo
               file "/etc/conf.d/hostname" do
                 content "hostname=\"#{new_resource.hostname}\"\n"
@@ -226,7 +226,7 @@ class Chef
               # that manage sysctls on linux.
               append_replacing_matching_lines("/etc/sysctl.conf", /^\s+kernel\.hostname\s+=/, "kernel.hostname=#{new_resource.hostname}")
             end
-          when ::TargetIO::File.exist?("/etc/rc.conf")
+          when ::File.exist?("/etc/rc.conf")
             # *BSD systems with /etc/rc.conf + /etc/myname
             append_replacing_matching_lines("/etc/rc.conf", /^\s+hostname\s+=/, "hostname=#{new_resource.hostname}")
 
@@ -236,7 +236,7 @@ class Chef
               group node["root_group"]
               mode "0644"
             end
-          when ::TargetIO::File.exist?("/usr/sbin/svccfg") # solaris 5.11
+          when ::File.exist?("/usr/sbin/svccfg") # solaris 5.11
             execute "svccfg -s system/identity:node setprop config/nodename=\'#{new_resource.hostname}\'" do
               notifies :run, "execute[svcadm refresh]", :immediately
               notifies :run, "execute[svcadm restart]", :immediately
@@ -259,7 +259,6 @@ class Chef
           WINDOWS_EC2_CONFIG = 'C:\Program Files\Amazon\Ec2ConfigService\Settings\config.xml'.freeze
 
           raise "Windows hostnames cannot contain a period." if new_resource.hostname.include?(".")
-          raise "Windows not supported in Target Mode" if ChefConfig::Config.target_mode?
 
           # suppress EC2 config service from setting our hostname
           if ::File.exist?(WINDOWS_EC2_CONFIG)

data/lib/chef/resource/http_request.rb

@@ -23,7 +23,7 @@ class Chef
   class Resource
     class HttpRequest < Chef::Resource
 
-      provides :http_request, target_mode: true
+      provides :http_request
 
       description "Use the **http_request** resource to send an HTTP request (`GET`, `PUT`, `POST`, `DELETE`, `HEAD`, or `OPTIONS`) with an arbitrary message. This resource is often useful when custom callbacks are necessary."
 

data/lib/chef/resource/ifconfig.rb

@@ -23,7 +23,7 @@ class Chef
   class Resource
     class Ifconfig < Chef::Resource
 
-      provides :ifconfig, target_mode: true
+      provides :ifconfig
 
       description "Use the **ifconfig** resource to manage interfaces on Unix and Linux systems. Note: This resource requires the ifconfig binary to be present on the system and may require additional packages to be installed first. On Ubuntu 18.04 or later you will need to install the `ifupdown` package, which disables the built in Netplan functionality. Warning: This resource will not work with Fedora release 33 or later."
       examples <<~DOC

data/lib/chef/resource/inspec_input.rb

@@ -19,7 +19,7 @@ require_relative "../resource"
 class Chef
   class Resource
     class InspecInput < Chef::Resource
-      provides :inspec_input, target_mode: true
+      provides :inspec_input
 
       description "Use the **inspec_input** resource to add an input to the Compliance Phase."
       introduced "17.5"
@@ -105,8 +105,6 @@ class Chef
         def build_source
           return new_resource.source unless new_resource.source.nil?
           return nil unless new_resource.input.count(::File::SEPARATOR) > 0 || (::File::ALT_SEPARATOR && new_resource.input.count(::File::ALT_SEPARATOR) > 0 )
-
-          # InSpec gets processed locally, so no TargetIO
           return nil unless ::File.exist?(new_resource.input)
 
           new_resource.input

data/lib/chef/resource/inspec_waiver.rb

@@ -19,7 +19,7 @@ require_relative "../resource"
 class Chef
   class Resource
     class InspecWaiver < Chef::Resource
-      provides :inspec_waiver, target_mode: true
+      provides :inspec_waiver
 
       description "Use the **inspec_waiver** resource to add a waiver to the Compliance Phase."
       introduced "17.5"

data/lib/chef/resource/inspec_waiver_file_entry.rb

@@ -23,7 +23,7 @@ require "chef-utils/dist" unless defined?(ChefUtils::Dist)
 class Chef
   class Resource
     class InspecWaiverFileEntry < Chef::Resource
-      provides :inspec_waiver_file_entry, target_mode: true
+      provides :inspec_waiver_file_entry
 
       description "Use the **inspec_waiver_file_entry** resource to add or remove entries from an InSpec waiver file. This can be used in conjunction with the Compliance Phase."
       introduced "17.1"

data/lib/chef/resource/ips_package.rb

@@ -23,8 +23,8 @@ class Chef
   class Resource
     class IpsPackage < ::Chef::Resource::Package
 
-      provides :ips_package, target_mode: true
-      provides :package, os: "solaris2", target_mode: true
+      provides :ips_package
+      provides :package, os: "solaris2"
 
       description "Use the **ips_package** resource to manage packages (using Image Packaging System (IPS)) on the Solaris 11 platform."
 

data/lib/chef/resource/kernel_module.rb

@@ -12,7 +12,7 @@ class Chef
   class Resource
     class KernelModule < Chef::Resource
 
-      provides :kernel_module, target_mode: true
+      provides :kernel_module
 
       description "Use the **kernel_module** resource to manage kernel modules on Linux systems. This resource can load, unload, blacklist, disable, enable, install, and uninstall modules."
       introduced "14.3"
@@ -232,7 +232,7 @@ class Chef
         # see if the module is listed in /proc/modules or not
         # @return [Boolean]
         def module_loaded?
-          /^#{new_resource.modname}/.match?(::TargetIO::File.read("/proc/modules"))
+          /^#{new_resource.modname}/.match?(::File.read("/proc/modules"))
         end
       end
     end

data/lib/chef/resource/link.rb

@@ -25,7 +25,7 @@ class Chef
     class Link < Chef::Resource
       include Chef::Mixin::Securable
 
-      provides :link, target_mode: true
+      provides :link
 
       description "Use the **link** resource to create symbolic or hard links.\n\n"\
                   "A symbolic link—sometimes referred to as a soft link—is a directory entry"\

data/lib/chef/resource/locale.rb

@@ -21,7 +21,7 @@ require "chef-utils/dist" unless defined?(ChefUtils::Dist)
 class Chef
   class Resource
     class Locale < Chef::Resource
-      provides :locale, target_mode: true
+      provides :locale
 
       description "Use the **locale** resource to set the system's locale on Debian and Windows systems. Windows support was added in Chef Infra Client 16.0"
       introduced "14.5"
@@ -79,7 +79,7 @@ class Chef
           lang get_system_locale_windows
         else
           begin
-            old_content = ::TargetIO::File.read(LOCALE_CONF)
+            old_content = ::File.read(LOCALE_CONF)
             locale_values = Hash[old_content.split("\n").map { |v| v.split("=") }]
             lang locale_values["LANG"]
           rescue Errno::ENOENT => e

data/lib/chef/resource/mount.rb

@@ -24,7 +24,7 @@ class Chef
     class Mount < Chef::Resource
       description "Use the **mount** resource to manage a mounted file system."
 
-      provides :mount, target_mode: true
+      provides :mount
 
       default_action :mount
       allowed_actions :mount, :umount, :unmount, :remount, :enable, :disable

data/lib/chef/resource/notify_group.rb

@@ -19,7 +19,7 @@ require_relative "../resource"
 class Chef
   class Resource
     class NotifyGroup < Chef::Resource
-      provides :notify_group, target_mode: true
+      provides :notify_group
 
       description "The notify_group resource does nothing, and always fires notifications which are set on it.  Use it to DRY blocks of notifications that are common to multiple resources, and provide a single target for other resources to notify.  Unlike most resources, its default action is :nothing."
       introduced "15.8"

data/lib/chef/resource/ohai.rb

@@ -26,7 +26,7 @@ class Chef
   class Resource
     class Ohai < Chef::Resource
 
-      provides :ohai, target_mode: true
+      provides :ohai
 
       description "Use the **ohai** resource to reload the Ohai configuration on a node. This allows recipes that change system attributes (like a recipe that adds a user) to refer to those attributes later on during the #{ChefUtils::Dist::Infra::PRODUCT} run."
 

data/lib/chef/resource/ohai_hint.rb

@@ -21,7 +21,7 @@ class Chef
   class Resource
     class OhaiHint < Chef::Resource
 
-      provides(:ohai_hint, target_mode: true) { true }
+      provides(:ohai_hint) { true }
 
       description "Use the **ohai_hint** resource to aid in configuration detection by passing hint data to Ohai."
       introduced "14.0"

data/lib/chef/resource/openbsd_package.rb

@@ -25,8 +25,8 @@ require_relative "../provider/package/openbsd"
 class Chef
   class Resource
     class OpenbsdPackage < Chef::Resource::Package
-      provides :openbsd_package, target_mode: true
-      provides :package, os: "openbsd", target_mode: true
+      provides :openbsd_package
+      provides :package, os: "openbsd"
 
       description "Use the **openbsd_package** resource to manage packages for the OpenBSD platform."
       introduced "12.1"

data/lib/chef/resource/package.rb

@@ -22,7 +22,7 @@ require_relative "../resource"
 class Chef
   class Resource
     class Package < Chef::Resource
-      provides :package, target_mode: true
+      provides :package
 
       description "Use the **package** resource to manage packages. When the package is"\
                   " installed from a local file (such as with RubyGems, dpkg, or RPM"\

data/lib/chef/resource/pacman_package.rb

@@ -22,7 +22,7 @@ class Chef
   class Resource
     class PacmanPackage < Chef::Resource::Package
 
-      provides :pacman_package, target_mode: true
+      provides :pacman_package
 
       allowed_actions :install, :upgrade, :remove, :purge
 

data/lib/chef/resource/paludis_package.rb

@@ -23,7 +23,7 @@ class Chef
   class Resource
     class PaludisPackage < Chef::Resource::Package
 
-      provides :paludis_package, target_mode: true
+      provides :paludis_package
 
       description "Use the **paludis_package** resource to manage packages for the Paludis platform."
       introduced "12.1"

data/lib/chef/resource/portage_package.rb

@@ -22,7 +22,7 @@ class Chef
   class Resource
     class PortagePackage < Chef::Resource::Package
 
-      provides :portage_package, target_mode: true
+      provides :portage_package
 
       description "Use the **portage_package** resource to manage packages for the Gentoo platform."
 

data/lib/chef/resource/reboot.rb

@@ -23,7 +23,7 @@ class Chef
   class Resource
     class Reboot < Chef::Resource
 
-      provides :reboot, target_mode: true
+      provides :reboot
 
       description "Use the **reboot** resource to reboot a node, a necessary step with some"\
                   " installations on certain platforms. This resource is supported for use on"\

data/lib/chef/resource/registry_key.rb

@@ -41,6 +41,19 @@ class Chef
       end
       ```
 
+      ```ruby
+      **Suppress reporting the sibling values of the values being updated in a registry key**
+      registry 'HKEY_LOCAL_MACHINE\SYSTEM\CurrentControlSet\CONTROL\Session Manager' do
+        values [{
+          name: 'ProtectionMode',
+          type: :dword,
+          data: 1
+        }]
+        only_record_changes true
+        action :create
+      end
+      ```
+
       **Create a registry key with binary data: "\x01\x02\x03"**:
 
       ```ruby
@@ -50,6 +63,7 @@ class Chef
           :type => :binary,
           :data => [0, 1, 2].map(&:chr).join
         }]
+
         action :create
       end
       ```
@@ -150,6 +164,10 @@ class Chef
       }
       property :recursive, [TrueClass, FalseClass], default: false
       property :architecture, Symbol, default: :machine, equal_to: %i{machine x86_64 i386}
+      property :only_record_changes, [TrueClass, FalseClass],
+               default: true,
+               introduced: "19.0",
+               description: "Suppress reporting of the current value of sibling values in a registry key. Setting this to false may result in a large number of values reported."
 
       # Some registry key data types may not be safely reported as json.
       # Example (CHEF-5323):

data/lib/chef/resource/remote_directory.rb

@@ -26,7 +26,7 @@ class Chef
     class RemoteDirectory < Chef::Resource::Directory
       include Chef::Mixin::Securable
 
-      provides :remote_directory, target_mode: true
+      provides :remote_directory
 
       description "Use the **remote_directory** resource to incrementally transfer a directory from a cookbook to a node. The directory that is copied from the cookbook should be located under `COOKBOOK_NAME/files/default/REMOTE_DIRECTORY`. The `remote_directory` resource will obey file specificity."
 

data/lib/chef/resource/remote_file.rb

@@ -29,7 +29,7 @@ class Chef
     class RemoteFile < Chef::Resource::File
       include Chef::Mixin::Securable
 
-      provides :remote_file, target_mode: true
+      provides :remote_file
 
       description "Use the **remote_file** resource to transfer a file from a remote location using file specificity. This resource is similar to the **file** resource. Note: Fetching files from the `files/` directory in a cookbook should be done with the **cookbook_file** resource."
 

data/lib/chef/resource/rhsm_errata.rb

@@ -20,7 +20,7 @@ require_relative "../resource"
 class Chef
   class Resource
     class RhsmErrata < Chef::Resource
-      provides(:rhsm_errata, target_mode: true) { true }
+      provides(:rhsm_errata) { true }
 
       description "Use the **rhsm_errata** resource to install packages associated with a given Red Hat Subscription Manager Errata ID. This is helpful if packages to mitigate a single vulnerability must be installed on your hosts."
       introduced "14.0"

data/lib/chef/resource/rhsm_errata_level.rb

@@ -20,7 +20,7 @@ require_relative "../resource"
 class Chef
   class Resource
     class RhsmErrataLevel < Chef::Resource
-      provides(:rhsm_errata_level, target_mode: true) { true }
+      provides(:rhsm_errata_level) { true }
 
       description "Use the **rhsm_errata_level** resource to install all packages of a specified errata level from the Red Hat Subscription Manager. For example, you can ensure that all packages associated with errata marked at a 'Critical' security level are installed."
       introduced "14.0"

data/lib/chef/resource/rhsm_register.rb

@@ -22,7 +22,7 @@ require "shellwords" unless defined?(Shellwords)
 class Chef
   class Resource
     class RhsmRegister < Chef::Resource
-      provides(:rhsm_register, target_mode: true) { true }
+      provides(:rhsm_register) { true }
 
       description "Use the **rhsm_register** resource to register a node with the Red Hat Subscription Manager or a local Red Hat Satellite server."
       introduced "14.0"

data/lib/chef/resource/rhsm_repo.rb

@@ -21,7 +21,7 @@ class Chef
   class Resource
     class RhsmRepo < Chef::Resource
 
-      provides(:rhsm_repo, target_mode: true) { true }
+      provides(:rhsm_repo) { true }
 
       description "Use the **rhsm_repo** resource to enable or disable Red Hat Subscription Manager repositories that are made available via attached subscriptions."
       introduced "14.0"
@@ -65,8 +65,9 @@ class Chef
 
       action_class do
         def repo_enabled?(repo)
-          # FIXME: Add `env` support
-          cmd = shell_out("subscription-manager repos --list-enabled", env: { LANG: "en_US" })
+          # FIXME: use shell_out()
+          cmd = Mixlib::ShellOut.new("subscription-manager repos --list-enabled", env: { LANG: "en_US" })
+          cmd.run_command
           repo == "*" || !cmd.stdout.match(/Repo ID:\s+#{repo}$/).nil?
         end
       end