chef 18.4.2 → 18.5.0

data/lib/chef/provider/directory.rb

@@ -27,12 +27,12 @@ class Chef
   class Provider
     class Directory < Chef::Provider::File
 
-      provides :directory
+      provides :directory, target_mode: true
 
       def load_current_resource
         @current_resource = Chef::Resource::Directory.new(new_resource.name)
         current_resource.path(new_resource.path)
-        if ::File.exist?(current_resource.path) && @action != :create_if_missing
+        if ::TargetIO::File.exist?(current_resource.path) && @action != :create_if_missing
           load_resource_attributes_from_file(current_resource)
         end
         current_resource
@@ -50,15 +50,15 @@ class Chef
             if new_resource.recursive
               does_parent_exist = lambda do |base_dir|
                 base_dir = ::File.dirname(base_dir)
-                if ::File.exist?(base_dir)
-                  ::File.directory?(base_dir)
+                if ::TargetIO::File.exist?(base_dir)
+                  ::TargetIO::File.directory?(base_dir)
                 else
                   does_parent_exist.call(base_dir)
                 end
               end
               does_parent_exist.call(new_resource.path)
             else
-              ::File.directory?(parent_directory)
+              ::TargetIO::File.directory?(parent_directory)
             end
           end
           a.failure_message(Chef::Exceptions::EnclosingDirectoryDoesNotExist, "Parent directory #{parent_directory} does not exist, cannot create #{new_resource.path}")
@@ -73,7 +73,7 @@ class Chef
               # make sure we have write permissions to that directory
               is_parent_writable = lambda do |base_dir|
                 base_dir = ::File.dirname(base_dir)
-                if ::File.exist?(base_dir)
+                if ::TargetIO::File.exist?(base_dir)
                   if Chef::FileAccessControl.writable?(base_dir)
                     true
                   elsif Chef::Util::PathHelper.is_sip_path?(base_dir, node)
@@ -89,7 +89,7 @@ class Chef
             else
               # in why run mode & parent directory does not exist no permissions check is required
               # If not in why run, permissions must be valid and we rely on prior assertion that dir exists
-              if !whyrun_mode? || ::File.exist?(parent_directory)
+              if !whyrun_mode? || ::TargetIO::File.exist?(parent_directory)
                 if Chef::FileAccessControl.writable?(parent_directory)
                   true
                 elsif Chef::Util::PathHelper.is_sip_path?(parent_directory, node)
@@ -108,8 +108,8 @@ class Chef
 
         requirements.assert(:delete) do |a|
           a.assertion do
-            if ::File.exist?(new_resource.path)
-              ::File.directory?(new_resource.path) && Chef::FileAccessControl.writable?(new_resource.path)
+            if ::TargetIO::File.exist?(new_resource.path)
+              ::TargetIO::File.directory?(new_resource.path) && Chef::FileAccessControl.writable?(new_resource.path)
             else
               true
             end
@@ -122,12 +122,12 @@ class Chef
       end
 
       action :create, description: "Create a directory. If a directory already exists (but does not match), update that directory to match." do
-        unless ::File.exist?(new_resource.path)
+        unless ::TargetIO::File.exist?(new_resource.path)
           converge_by("create new directory #{new_resource.path}") do
             if new_resource.recursive == true
-              ::FileUtils.mkdir_p(new_resource.path)
+              ::TargetIO::FileUtils.mkdir_p(new_resource.path)
             else
-              ::Dir.mkdir(new_resource.path)
+              ::TargetIO::Dir.mkdir(new_resource.path)
             end
             logger.info("#{new_resource} created directory #{new_resource.path}")
           end
@@ -138,15 +138,15 @@ class Chef
       end
 
       action :delete, description: "Delete a directory." do
-        if ::File.exist?(new_resource.path)
+        if ::TargetIO::File.exist?(new_resource.path)
           converge_by("delete existing directory #{new_resource.path}") do
             if new_resource.recursive == true
               # we don't use rm_rf here because it masks all errors, including
               # IO errors or permission errors that would prevent the deletion
-              FileUtils.rm_r(new_resource.path)
+              ::TargetIO::FileUtils.rm_r(new_resource.path)
               logger.info("#{new_resource} deleted #{new_resource.path} recursively")
             else
-              ::Dir.delete(new_resource.path)
+              ::TargetIO::Dir.delete(new_resource.path)
               logger.info("#{new_resource} deleted #{new_resource.path}")
             end
           end

data/lib/chef/provider/file.rb

@@ -29,6 +29,7 @@ require_relative "../mixin/file_class"
 require_relative "../mixin/enforce_ownership_and_permissions"
 require_relative "../resource/file/verification/json"
 require_relative "../resource/file/verification/yaml"
+require_relative "../target_io"
 require_relative "../util/backup"
 require_relative "../util/diff"
 require_relative "../util/selinux"
@@ -53,7 +54,7 @@ class Chef
       include Chef::Util::Selinux
       include Chef::Mixin::FileClass
 
-      provides :file
+      provides :file, target_mode: true
 
       attr_reader :deployment_strategy
 
@@ -75,7 +76,7 @@ class Chef
 
         # true if there is a non-file thing in the way that we need to unlink first
         @needs_unlinking =
-          if ::File.exist?(new_resource.path)
+          if ::TargetIO::File.exist?(new_resource.path)
             if managing_symlink?
               !symlink_to_real_file?(new_resource.path)
             else
@@ -86,7 +87,7 @@ class Chef
           end
 
         # true if we are going to be creating a new file
-        @needs_creating = !::File.exist?(new_resource.path) || needs_unlinking?
+        @needs_creating = !::TargetIO::File.exist?(new_resource.path) || needs_unlinking?
 
         # Let children resources override constructing the current_resource
         @current_resource ||= Chef::Resource::File.new(new_resource.name)
@@ -96,7 +97,11 @@ class Chef
           # we are updating an existing file
           if managing_content?
             logger.trace("#{new_resource} checksumming file at #{new_resource.path}.")
-            current_resource.checksum(checksum(current_resource.path))
+
+            resource_to_check = current_resource.path
+            resource_to_check = ::TargetIO::File.open(current_resource.path) if ChefConfig::Config.target_mode?
+
+            current_resource.checksum(checksum(resource_to_check))
           else
             # if the file does not exist or is not a file, then the checksum is invalid/pointless
             current_resource.checksum(nil)
@@ -114,15 +119,15 @@ class Chef
         # Make sure the parent directory exists, otherwise fail.  For why-run assume it would have been created.
         requirements.assert(:create, :create_if_missing, :touch) do |a|
           parent_directory = ::File.dirname(new_resource.path)
-          a.assertion { ::File.directory?(parent_directory) }
+          a.assertion { ::TargetIO::File.directory?(parent_directory) }
           a.failure_message(Chef::Exceptions::EnclosingDirectoryDoesNotExist, "Parent directory #{parent_directory} does not exist.")
           a.whyrun("Assuming directory #{parent_directory} would have been created")
         end
 
         # Make sure the file is deletable if it exists, otherwise fail.
-        if ::File.exist?(new_resource.path)
+        if ::TargetIO::File.exist?(new_resource.path)
           requirements.assert(:delete) do |a|
-            a.assertion { ::File.writable?(new_resource.path) }
+            a.assertion { ::TargetIO::File.writable?(new_resource.path) }
             a.failure_message(Chef::Exceptions::InsufficientPermissions, "File #{new_resource.path} exists but is not writable so it cannot be deleted")
           end
         end
@@ -151,7 +156,7 @@ class Chef
       end
 
       action :create_if_missing do
-        unless ::File.exist?(new_resource.path)
+        unless ::TargetIO::File.exist?(new_resource.path)
           action_create
         else
           logger.debug("#{new_resource} exists at #{new_resource.path} taking no action.")
@@ -159,10 +164,10 @@ class Chef
       end
 
       action :delete do
-        if ::File.exist?(new_resource.path)
+        if ::TargetIO::File.exist?(new_resource.path)
           converge_by("delete file #{new_resource.path}") do
             do_backup unless file_class.symlink?(new_resource.path)
-            ::File.delete(new_resource.path)
+            ::TargetIO::File.delete(new_resource.path)
             logger.info("#{new_resource} deleted file at #{new_resource.path}")
           end
         end
@@ -172,7 +177,7 @@ class Chef
         action_create
         converge_by("update utime on file #{new_resource.path}") do
           time = Time.now
-          ::File.utime(time, time, new_resource.path)
+          ::TargetIO::File.utime(time, time, new_resource.path)
           logger.info("#{new_resource} updated atime and mtime to #{time}")
         end
       end
@@ -254,7 +259,7 @@ class Chef
       # If any of the above apply, returns a 3-tuple of Exception class,
       # exception message, whyrun message; otherwise returns a 3-tuple of nil.
       def verify_symlink_sanity(path)
-        real_path = ::File.realpath(path)
+        real_path = ::TargetIO::File.realpath(path)
         if real_file?(real_path)
           [nil, nil, nil]
         else
@@ -283,15 +288,15 @@ class Chef
 
       def file_type_string(path)
         case
-        when ::File.blockdev?(path)
+        when ::TargetIO::File.blockdev?(path)
           "block device"
-        when ::File.chardev?(path)
+        when ::TargetIO::File.chardev?(path)
           "char device"
-        when ::File.directory?(path)
+        when ::TargetIO::File.directory?(path)
           "directory"
-        when ::File.pipe?(path)
+        when ::TargetIO::File.pipe?(path)
           "pipe"
-        when ::File.socket?(path)
+        when ::TargetIO::File.socket?(path)
           "socket"
         when file_class.symlink?(path)
           "symlink"
@@ -301,12 +306,12 @@ class Chef
       end
 
       def real_file?(path)
-        !file_class.symlink?(path) && ::File.file?(path)
+        !file_class.symlink?(path) && ::TargetIO::File.file?(path)
       end
 
       # like real_file? that follows (sane) symlinks
       def symlink_to_real_file?(path)
-        real_file?(::File.realpath(path))
+        real_file?(::TargetIO::File.realpath(path))
       rescue Errno::ELOOP, Errno::ENOENT
         false
       end
@@ -314,15 +319,15 @@ class Chef
       # Similar to File.exist?, but also returns true in the case that the
       # named file is a broken symlink.
       def l_exist?(path)
-        ::File.exist?(path) || file_class.symlink?(path)
+        ::TargetIO::File.exist?(path) || file_class.symlink?(path)
       end
 
       def unlink(path)
         # Directories can not be unlinked. Remove them using FileUtils.
-        if ::File.directory?(path)
-          FileUtils.rm_rf(path)
+        if ::TargetIO::File.directory?(path)
+          TargetIO::FileUtils.rm_rf(path)
         else
-          ::File.unlink(path)
+          ::TargetIO::File.unlink(path)
         end
       end
 
@@ -344,8 +349,8 @@ class Chef
           new_resource.verify.each do |v|
             unless v.verify(tempfile.path)
               backupfile = "#{Chef::Config[:file_cache_path]}/failed_validations/#{::File.basename(tempfile.path)}"
-              FileUtils.mkdir_p ::File.dirname(backupfile)
-              FileUtils.cp tempfile.path, backupfile
+              TargetIO::FileUtils.mkdir_p ::File.dirname(backupfile)
+              TargetIO::FileUtils.cp tempfile.path, backupfile
               raise Chef::Exceptions::ValidationFailed.new "Proposed content for #{new_resource.path} failed verification #{new_resource.sensitive ? "[sensitive]" : "#{v}\n#{v.output}"}\nTemporary file moved to #{backupfile}"
             end
           end
@@ -374,6 +379,11 @@ class Chef
       end
 
       def do_backup(file = nil)
+        if ChefConfig::Config.target_mode?
+          Chef::Log.info("#{@new_resource} backup skipped in Target Mode")
+          return
+        end
+
         Chef::Util::Backup.new(new_resource, file).backup!
       end
 
@@ -383,11 +393,14 @@ class Chef
 
       def update_file_contents
         do_backup unless needs_creating?
-        deployment_strategy.deploy(tempfile.path, ::File.realpath(new_resource.path).force_encoding(Chef::Config[:ruby_encoding]))
+        deployment_strategy.deploy(tempfile.path, ::TargetIO::File.realpath(new_resource.path).force_encoding(Chef::Config[:ruby_encoding]))
         logger.info("#{new_resource} updated file contents #{new_resource.path}")
         if managing_content?
           # save final checksum for reporting.
-          new_resource.final_checksum = checksum(new_resource.path)
+          resource_to_check = new_resource.path
+          resource_to_check = ::TargetIO::File.open(resource_to_check) if ChefConfig::Config.target_mode?
+
+          new_resource.final_checksum = checksum(resource_to_check)
         end
       end
 
@@ -400,7 +413,7 @@ class Chef
         end
 
         # the file? on the next line suppresses the case in why-run when we have a not-file here that would have otherwise been removed
-        if ::File.file?(new_resource.path) && contents_changed?
+        if ::TargetIO::File.file?(new_resource.path) && contents_changed?
           description = [ "update content in file #{new_resource.path} from \
 #{short_cksum(current_resource.checksum)} to #{short_cksum(tempfile_checksum)}" ]
 
@@ -432,7 +445,7 @@ class Chef
         if resource_updated? && Chef::Config[:enable_selinux_file_permission_fixup]
           if selinux_enabled?
             converge_by("restore selinux security context") do
-              restore_security_context(::File.realpath(new_resource.path), recursive)
+              restore_security_context(::TargetIO::File.realpath(new_resource.path), recursive)
             end
           else
             logger.trace "selinux utilities can not be found. Skipping selinux permission fixup."

data/lib/chef/provider/git.rb

@@ -26,7 +26,7 @@ class Chef
     class Git < Chef::Provider
 
       extend Forwardable
-      provides :git
+      provides :git, target_mode: true
 
       GIT_VERSION_PATTERN = Regexp.compile("git version (\\d+\\.\\d+.\\d+)")
 
@@ -58,7 +58,7 @@ class Chef
         # Parent directory of the target must exist.
         requirements.assert(:checkout, :sync) do |a|
           dirname = ::File.dirname(cwd)
-          a.assertion { ::File.directory?(dirname) }
+          a.assertion { ::TargetIO::File.directory?(dirname) }
           a.whyrun("Directory #{dirname} does not exist, this run will fail unless it has been previously created. Assuming it would have been created.")
           a.failure_message(Chef::Exceptions::MissingParentDirectory,
                             "Cannot clone #{new_resource} to #{cwd}, the enclosing directory #{dirname} does not exist")
@@ -101,7 +101,7 @@ class Chef
       action :export do
         action_checkout
         converge_by("complete the export by removing #{cwd}.git after checkout") do
-          FileUtils.rm_rf(::File.join(cwd, ".git"))
+          TargetIO::FileUtils.rm_rf(::File.join(cwd, ".git"))
         end
       end
 
@@ -138,16 +138,16 @@ class Chef
       end
 
       def existing_git_clone?
-        ::File.exist?(::File.join(cwd, ".git"))
+        ::TargetIO::File.exist?(::File.join(cwd, ".git"))
       end
 
       def target_dir_non_existent_or_empty?
-        !::File.exist?(cwd) || Dir.entries(cwd).sort == [".", ".."]
+        !::TargetIO::File.exist?(cwd) || TargetIO::Dir.entries(cwd).sort == [".", ".."]
       end
 
       def find_current_revision
         logger.trace("#{new_resource} finding current git revision")
-        if ::File.exist?(::File.join(cwd, ".git"))
+        if ::TargetIO::File.exist?(::File.join(cwd, ".git"))
           # 128 is returned when we're not in a git repo. this is fine
           result = git("rev-parse", "HEAD", cwd: cwd, returns: [0, 128]).stdout.strip
         end
@@ -413,9 +413,9 @@ class Chef
         require "etc" unless defined?(Etc)
         case user
         when Integer
-          Etc.getpwuid(user).dir
+          TargetIO::Etc.getpwuid(user).dir
         else
-          Etc.getpwnam(user.to_s).dir
+          TargetIO::Etc.getpwnam(user.to_s).dir
         end
       end
     end

data/lib/chef/provider/group/aix.rb

@@ -22,7 +22,7 @@ class Chef
   class Provider
     class Group
       class Aix < Chef::Provider::Group::Groupadd
-        provides :group, platform: "aix"
+        provides :group, platform: "aix", target_mode: true
 
         def required_binaries
           [ "/usr/bin/mkgroup",

data/lib/chef/provider/group/dscl.rb

@@ -21,7 +21,7 @@ class Chef
     class Group
       class Dscl < Chef::Provider::Group
 
-        provides :group, os: "darwin"
+        provides :group, os: "darwin", target_mode: true
 
         def dscl(*args)
           argdup = args.dup

data/lib/chef/provider/group/gpasswd.rb

@@ -22,7 +22,7 @@ class Chef
   class Provider
     class Group
       class Gpasswd < Chef::Provider::Group::Groupadd
-        provides :group
+        provides :group, target_mode: true
 
         def load_current_resource
           super
@@ -31,7 +31,7 @@ class Chef
         def define_resource_requirements
           super
           requirements.assert(:all_actions) do |a|
-            a.assertion { ::File.exist?("/usr/bin/gpasswd") }
+            a.assertion { ::TargetIO::File.exist?("/usr/bin/gpasswd") }
             a.failure_message Chef::Exceptions::Group, "Could not find binary /usr/bin/gpasswd for #{new_resource}"
             # No whyrun alternative: this component should be available in the base install of any given system that uses it
           end

data/lib/chef/provider/group/groupadd.rb

@@ -35,7 +35,7 @@ class Chef
           super
           required_binaries.each do |required_binary|
             requirements.assert(:all_actions) do |a|
-              a.assertion { ::File.exist?(required_binary) }
+              a.assertion { ::TargetIO::File.exist?(required_binary) }
               a.failure_message Chef::Exceptions::Group, "Could not find binary #{required_binary} for #{new_resource}"
               # No whyrun alternative: this component should be available in the base install of any given system that uses it
             end

data/lib/chef/provider/group/groupmod.rb

@@ -21,12 +21,12 @@ class Chef
     class Group
       class Groupmod < Chef::Provider::Group
 
-        provides :group, os: "netbsd"
+        provides :group, os: "netbsd", target_mode: true
 
         def load_current_resource
           super
           %w{group user}.each do |binary|
-            raise Chef::Exceptions::Group, "Could not find binary /usr/sbin/#{binary} for #{new_resource}" unless ::File.exist?("/usr/sbin/#{binary}")
+            raise Chef::Exceptions::Group, "Could not find binary /usr/sbin/#{binary} for #{new_resource}" unless ::TargetIO::File.exist?("/usr/sbin/#{binary}")
           end
         end
 

data/lib/chef/provider/group/pw.rb

@@ -20,7 +20,7 @@ class Chef
   class Provider
     class Group
       class Pw < Chef::Provider::Group
-        provides :group, platform: "freebsd"
+        provides :group, platform: "freebsd", target_mode: true
 
         def load_current_resource
           super
@@ -30,7 +30,7 @@ class Chef
           super
 
           requirements.assert(:all_actions) do |a|
-            a.assertion { ::File.exist?("/usr/sbin/pw") }
+            a.assertion { ::TargetIO::File.exist?("/usr/sbin/pw") }
             a.failure_message Chef::Exceptions::Group, "Could not find binary /usr/sbin/pw for #{new_resource}"
             # No whyrun alternative: this component should be available in the base install of any given system that uses it
           end

data/lib/chef/provider/group/solaris.rb

@@ -25,7 +25,7 @@ class Chef
 
         # this provides line is setup to only catch the solaris2 platform, but
         # NOT other platforms in the Solaris platform_family. (See usermod provider.)
-        provides :group, platform: "solaris2"
+        provides :group, platform: "solaris2", target_mode: true
 
         def load_current_resource
           super
@@ -35,7 +35,7 @@ class Chef
           super
 
           requirements.assert(:all_actions) do |a|
-            a.assertion { ::File.exist?("/usr/sbin/usermod") && ::File.exist?("/usr/sbin/groupmod") }
+            a.assertion { ::TargetIO::File.exist?("/usr/sbin/usermod") && ::TargetIO::File.exist?("/usr/sbin/groupmod") }
             a.failure_message Chef::Exceptions::Group, "Could not find binary /usr/sbin/usermod or /usr/sbin/groupmod for #{new_resource}"
             # No whyrun alternative: this component should be available in the base install of any given system that uses it
           end

data/lib/chef/provider/group/usermod.rb

@@ -23,7 +23,7 @@ class Chef
     class Group
       class Usermod < Chef::Provider::Group::Groupadd
 
-        provides :group, os: %w{openbsd solaris2}
+        provides :group, os: %w{openbsd solaris2}, target_mode: true
 
         def load_current_resource
           super
@@ -33,7 +33,7 @@ class Chef
           super
 
           requirements.assert(:all_actions) do |a|
-            a.assertion { ::File.exist?("/usr/sbin/usermod") }
+            a.assertion { ::TargetIO::File.exist?("/usr/sbin/usermod") }
             a.failure_message Chef::Exceptions::Group, "Could not find binary /usr/sbin/usermod for #{new_resource}"
             # No whyrun alternative: this component should be available in the base install of any given system that uses it
           end

data/lib/chef/provider/group.rb

@@ -36,7 +36,7 @@ class Chef
 
         group_info = nil
         begin
-          group_info = Etc.getgrnam(new_resource.group_name)
+          group_info = TargetIO::Etc.getgrnam(new_resource.group_name)
         rescue ArgumentError
           @group_exists = false
           logger.trace("#{new_resource} group does not exist")

data/lib/chef/provider/http_request.rb

@@ -23,12 +23,12 @@ class Chef
   class Provider
     class HttpRequest < Chef::Provider
 
-      provides :http_request
+      provides :http_request, target_mode: true
 
       attr_writer :http
 
       def http
-        @http ||= Chef::HTTP::Simple.new(new_resource.url)
+        @http ||= TargetIO::HTTP.new(new_resource.url)
       end
 
       def load_current_resource; end
@@ -60,7 +60,6 @@ class Chef
             new_resource.headers
           )
           logger.info("#{new_resource} GET to #{new_resource.url} successful")
-          logger.trace("#{new_resource} GET request response: #{body}")
         end
       end
 

data/lib/chef/provider/ifconfig/aix.rb

@@ -22,7 +22,7 @@ class Chef
   class Provider
     class Ifconfig
       class Aix < Chef::Provider::Ifconfig
-        provides :ifconfig, platform: "aix"
+        provides :ifconfig, platform: "aix", target_mode: true
 
         def load_current_resource
           @current_resource = Chef::Resource::Ifconfig.new(new_resource.name)

data/lib/chef/provider/ifconfig/debian.rb

@@ -23,7 +23,7 @@ class Chef
   class Provider
     class Ifconfig
       class Debian < Chef::Provider::Ifconfig
-        provides :ifconfig, platform_family: %w{debian}
+        provides :ifconfig, platform_family: %w{debian}, target_mode: true
 
         INTERFACES_FILE = "/etc/network/interfaces".freeze
         INTERFACES_DOT_D_DIR = "/etc/network/interfaces.d".freeze
@@ -81,7 +81,7 @@ iface <%= new_resource.device %> <%= new_resource.family %> static
 
         def enforce_interfaces_dot_d_sanity
           # on ubuntu 18.04+ there's no interfaces file and it uses interfaces.d by default
-          return if ::File.directory?(INTERFACES_DOT_D_DIR) && !::File.exist?(INTERFACES_FILE)
+          return if ::TargetIO::File.directory?(INTERFACES_DOT_D_DIR) && !::TargetIO::File.exist?(INTERFACES_FILE)
 
           # create /etc/network/interfaces.d via dir if it's missing
           directory INTERFACES_DOT_D_DIR
@@ -90,7 +90,7 @@ iface <%= new_resource.device %> <%= new_resource.family %> static
           interfaces_dot_d_for_regexp = INTERFACES_DOT_D_DIR.gsub(/\./, "\\.") # escape dots for the regexp
           regexp = %r{^\s*source\s+#{interfaces_dot_d_for_regexp}/\*\s*$}
 
-          return if ::File.exist?(INTERFACES_FILE) && regexp.match(IO.read(INTERFACES_FILE))
+          return if ::TargetIO::File.exist?(INTERFACES_FILE) && regexp.match(IO.read(INTERFACES_FILE))
 
           converge_by("modifying #{INTERFACES_FILE} to source #{INTERFACES_DOT_D_DIR}") do
             conf = Chef::Util::FileEdit.new(INTERFACES_FILE)

data/lib/chef/provider/ifconfig/redhat.rb

@@ -22,7 +22,7 @@ class Chef
   class Provider
     class Ifconfig
       class Redhat < Chef::Provider::Ifconfig
-        provides :ifconfig, platform_family: "fedora_derived"
+        provides :ifconfig, platform_family: "fedora_derived", target_mode: true
 
         def initialize(new_resource, run_context)
           super(new_resource, run_context)

data/lib/chef/provider/ifconfig.rb

@@ -31,7 +31,7 @@ class Chef
     #     device 'eth1'
     #   end
     class Ifconfig < Chef::Provider
-      provides :ifconfig
+      provides :ifconfig, target_mode: true
 
       attr_accessor :config_template
       attr_accessor :config_path

data/lib/chef/provider/link.rb

@@ -28,7 +28,7 @@ class Chef
   class Provider
     class Link < Chef::Provider
 
-      provides :link
+      provides :link, target_mode: true
 
       include Chef::Mixin::EnforceOwnershipAndPermissions
       include Chef::Mixin::FileClass
@@ -43,8 +43,8 @@ class Chef
           )
         else
           current_resource.link_type(:hard)
-          if ::File.exist?(current_resource.target_file)
-            if ::File.exist?(new_resource.to) &&
+          if ::TargetIO::File.exist?(current_resource.target_file)
+            if ::TargetIO::File.exist?(new_resource.to) &&
                 file_class.stat(current_resource.target_file).ino ==
                     file_class.stat(new_resource.to).ino
               current_resource.to(canonicalize(new_resource.to))
@@ -92,14 +92,14 @@ class Chef
             # However if the new symlink will point to a file and the current symlink is pointing at a
             # directory we want to throw an exception and calling ::File.unlink on the directory symlink
             # will throw the correct ones.
-            if ChefUtils.windows? && ::File.directory?(new_resource.to) &&
-                ::File.directory?(current_resource.target_file)
+            if ChefUtils.windows? && ::TargetIO::File.directory?(new_resource.to) &&
+                ::TargetIO::File.directory?(current_resource.target_file)
               converge_by("unlink existing windows symlink to dir at #{new_resource.target_file}") do
-                ::Dir.unlink(new_resource.target_file)
+                ::TargetIO::Dir.unlink(new_resource.target_file)
               end
             else
               converge_by("unlink existing symlink to file at #{new_resource.target_file}") do
-                ::File.unlink(new_resource.target_file)
+                ::TargetIO::File.unlink(new_resource.target_file)
               end
             end
           end
@@ -134,14 +134,14 @@ class Chef
 
       action :delete do
         if current_resource.to # Exists
-          if ChefUtils.windows? && ::File.directory?(current_resource.target_file)
+          if ChefUtils.windows? && ::TargetIO::File.directory?(current_resource.target_file)
             converge_by("delete link to dir at #{new_resource.target_file}") do
-              ::Dir.delete(new_resource.target_file)
+              ::TargetIO::Dir.delete(new_resource.target_file)
               logger.info("#{new_resource} deleted")
             end
           else
             converge_by("delete link to file at #{new_resource.target_file}") do
-              ::File.delete(new_resource.target_file)
+              ::TargetIO::File.delete(new_resource.target_file)
               logger.info("#{new_resource} deleted")
             end
           end