chef 18.4.2 → 18.5.0

data/lib/chef/provider/package/zypper.rb

@@ -30,8 +30,8 @@ class Chef
         use_package_name_for_source
         allow_nils
 
-        provides :package, platform_family: "suse"
-        provides :zypper_package
+        provides :package, platform_family: "suse", target_mode: true
+        provides :zypper_package, target_mode: true
 
         def define_resource_requirements
           super
@@ -91,7 +91,7 @@ class Chef
         # @return [Boolean] True if all sources exist
         def source_files_exist?
           if !new_resource.source.nil?
-            resolved_source_array.all? { |s| s && ::File.exist?(s) }
+            resolved_source_array.all? { |s| s && ::TargetIO::File.exist?(s) }
           else
             true
           end
@@ -100,7 +100,7 @@ class Chef
         # Helper to return all the names of the missing sources for error messages.
         # @return [Array<String>] Array of missing sources
         def missing_sources
-          resolved_source_array.select { |s| s.nil? || !::File.exist?(s) }
+          resolved_source_array.select { |s| s.nil? || !::TargetIO::File.exist?(s) }
         end
 
         def resolve_source_to_version
@@ -146,7 +146,6 @@ class Chef
             if md = line.match(/^(\S*)\s+\|\s+(\S+)\s+\|\s+(\S+)\s+\|\s+(\S+)\s+\|\s+(\S+)\s+\|\s+(.*)$/)
               (status, name, type, version, arch, repo) = [ md[1], md[2], md[3], md[4], md[5], md[6] ]
               next if version == "Version" # header
-              next if name != package_name
 
               # sometimes even though we request a specific version in the search string above and have match exact, we wind up
               # with other versions in the output, particularly getting the installed version when downgrading.

data/lib/chef/provider/package.rb

@@ -648,7 +648,7 @@ class Chef
           source_array.each_with_index.map do |source, i|
             package_name = package_name_array[i]
             # we require at least one '/' in the package_name to avoid [XXX_]package 'foo' breaking due to a random 'foo' file in cwd
-            if use_package_name_for_source? && source.nil? && package_name.match(/#{::File::SEPARATOR}/) && ::File.exist?(package_name)
+            if use_package_name_for_source? && source.nil? && package_name.match(/#{::File::SEPARATOR}/) && ::TargetIO::File.exist?(package_name)
               logger.trace("No package source specified, but #{package_name} exists on filesystem, using #{package_name} as source.")
               package_name
             else

data/lib/chef/provider/remote_directory.rb

@@ -32,7 +32,7 @@ class Chef
       extend Forwardable
       include Chef::Mixin::FileClass
 
-      provides :remote_directory
+      provides :remote_directory, target_mode: true
 
       def_delegators :new_resource, :purge, :path, :source, :cookbook, :cookbook_name
       def_delegators :new_resource, :files_rights, :files_mode, :files_group, :files_owner, :files_backup
@@ -100,7 +100,7 @@ class Chef
       #
       def purge_unmanaged_files
         if purge
-          Dir.glob(::File.join(Chef::Util::PathHelper.escape_glob_dir(path), "**", "*"), ::File::FNM_DOTMATCH).sort!.reverse!.each do |file|
+          TargetIO::Dir.glob(::File.join(Chef::Util::PathHelper.escape_glob_dir(path), "**", "*"), ::File::FNM_DOTMATCH).sort!.reverse!.each do |file|
             # skip '.' and '..'
             next if [".", ".."].include?(Pathname.new(file).basename.to_s)
 
@@ -110,7 +110,7 @@ class Chef
             # Skip files that we've sync'd and their parent dirs
             next if managed_files.include?(file)
 
-            if ::File.directory?(file)
+            if ::TargetIO::File.directory?(file)
               if !ChefUtils.windows? && file_class.symlink?(file.dup)
                 # Unix treats dir symlinks as files
                 purge_file(file)
@@ -188,7 +188,7 @@ class Chef
       def create_cookbook_file(cookbook_file_relative_path)
         full_path = ::File.join(path, cookbook_file_relative_path)
 
-        ensure_directory_exists(::File.dirname(full_path))
+        ensure_directory_exists(::TargetIO::File.dirname(full_path))
 
         res = cookbook_file_resource(full_path, cookbook_file_relative_path)
         res.run_action(action_for_cookbook_file)
@@ -228,7 +228,7 @@ class Chef
       #
       def ensure_directory_exists(dir)
         # doing the check here and skipping the resource should be more performant
-        unless ::File.directory?(dir)
+        unless ::TargetIO::File.directory?(dir)
           res = directory_resource(dir)
           res.run_action(:create)
           new_resource.updated_by_last_action(true) if res.updated?

data/lib/chef/provider/remote_file/http.rb

@@ -17,7 +17,6 @@
 # limitations under the License.
 #
 
-require_relative "../../http/simple"
 require_relative "../../digester"
 require_relative "../remote_file"
 require_relative "cache_control_data"
@@ -62,7 +61,7 @@ class Chef
         end
 
         def fetch
-          http = Chef::HTTP::Simple.new(uri, http_client_opts)
+          http = TargetIO::HTTP.new(uri, http_client_opts)
           orig_tempfile = Chef::FileContentManagement::Tempfile.new(@new_resource).tempfile
           if want_progress?
             tempfile = http.streaming_request_with_progress(uri, headers, orig_tempfile) do |size, total|
@@ -97,7 +96,7 @@ class Chef
         end
 
         def want_progress?
-          events.formatter? && (Chef::Config[:show_download_progress] || !!new_resource.show_progress)
+          !ChefConfig::Config.target_mode? && events.formatter? && (Chef::Config[:show_download_progress] || !!new_resource.show_progress)
         end
 
         def progress_interval

data/lib/chef/provider/remote_file.rb

@@ -22,7 +22,7 @@ require_relative "file"
 class Chef
   class Provider
     class RemoteFile < Chef::Provider::File
-      provides :remote_file
+      provides :remote_file, target_mode: true
 
       def initialize(new_resource, run_context)
         @content_class = Chef::Provider::RemoteFile::Content

data/lib/chef/provider/route.rb

@@ -25,7 +25,7 @@ class Chef
   class Provider
     class Route < Chef::Provider
 
-      provides :route
+      provides :route, target_mode: true
 
       attr_accessor :is_running
 
@@ -98,7 +98,7 @@ class Chef
         # For linux, we use /proc/net/route file to read proc table info
         return unless linux?
 
-        route_file = ::File.open("/proc/net/route", "r")
+        route_file = ::TargetIO::File.open("/proc/net/route", "r")
 
         # Read all routes
         while (line = route_file.gets)
@@ -186,24 +186,24 @@ class Chef
               network_file_name = "/etc/sysconfig/network"
               converge_by("write route default route to #{network_file_name}") do
                 logger.trace("#{new_resource} writing default route #{new_resource.gateway} to #{network_file_name}")
-                if ::File.exist?(network_file_name)
+                if ::TargetIO::File.exist?(network_file_name)
                   network_file = ::Chef::Util::FileEdit.new(network_file_name)
                   network_file.search_file_replace_line(/^GATEWAY=/, "GATEWAY=#{new_resource.gateway}")
                   network_file.insert_line_if_no_match(/^GATEWAY=/, "GATEWAY=#{new_resource.gateway}")
                   network_file.write_file
                 else
-                  network_file = ::File.new(network_file_name, "w")
-                  network_file.puts("GATEWAY=#{new_resource.gateway}")
-                  network_file.close
+                  ::TargetIO::File.open(network_file_name, "w") do |network_file|
+                    network_file.puts("GATEWAY=#{new_resource.gateway}")
+                  end
                 end
               end
             else
               network_file_name = "/etc/sysconfig/network-scripts/route-#{k}"
               converge_by("write route route.#{k}\n#{conf[k]} to #{network_file_name}") do
-                network_file = ::File.new(network_file_name, "w")
-                network_file.puts(conf[k])
+                ::TargetIO::File.open(network_file_name, "w") do |network_file|
+                  network_file.puts(conf[k])
+                end
                 logger.trace("#{new_resource} writing route.#{k}\n#{conf[k]}")
-                network_file.close
               end
             end
           end

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

@@ -24,7 +24,7 @@ class Chef
       class Aix < Chef::Provider::Service
         attr_reader :status_load_success
 
-        provides :service, os: "aix"
+        provides :service, os: "aix", target_mode: true
 
         def initialize(new_resource, run_context)
           super

data/lib/chef/provider/service/aixinit.rb

@@ -57,7 +57,7 @@ class Chef
         end
 
         def enable_service
-          Dir.glob(["/etc/rc.d/rc2.d/[SK][0-9][0-9]#{@new_resource.service_name}", "/etc/rc.d/rc2.d/[SK]#{@new_resource.service_name}"]).each { |f| ::File.delete(f) }
+          TargetIO::Dir.glob(["/etc/rc.d/rc2.d/[SK][0-9][0-9]#{@new_resource.service_name}", "/etc/rc.d/rc2.d/[SK]#{@new_resource.service_name}"]).each { |f| ::File.delete(f) }
 
           if @new_resource.priority.is_a? Integer
             create_symlink(2, "S", @new_resource.priority)
@@ -72,7 +72,7 @@ class Chef
         end
 
         def disable_service
-          Dir.glob(["/etc/rc.d/rc2.d/[SK][0-9][0-9]#{@new_resource.service_name}", "/etc/rc.d/rc2.d/[SK]#{@new_resource.service_name}"]).each { |f| ::File.delete(f) }
+          TargetIO::Dir.glob(["/etc/rc.d/rc2.d/[SK][0-9][0-9]#{@new_resource.service_name}", "/etc/rc.d/rc2.d/[SK]#{@new_resource.service_name}"]).each { |f| ::File.delete(f) }
 
           if @new_resource.priority.is_a? Integer
             create_symlink(2, "K", 100 - @new_resource.priority)
@@ -86,13 +86,13 @@ class Chef
         end
 
         def create_symlink(run_level, status, priority)
-          ::File.symlink("/etc/rc.d/init.d/#{@new_resource.service_name}", "/etc/rc.d/rc#{run_level}.d/#{status}#{priority}#{@new_resource.service_name}")
+          ::TargetIO::File.symlink("/etc/rc.d/init.d/#{@new_resource.service_name}", "/etc/rc.d/rc#{run_level}.d/#{status}#{priority}#{@new_resource.service_name}")
         end
 
         def set_current_resource_attributes
           # assuming run level 2 for aix
           is_enabled = false
-          files = Dir.glob(["/etc/rc.d/rc2.d/[SK][0-9][0-9]#{@new_resource.service_name}", "/etc/rc.d/rc2.d/[SK]#{@new_resource.service_name}"])
+          files = TargetIO::Dir.glob(["/etc/rc.d/rc2.d/[SK][0-9][0-9]#{@new_resource.service_name}", "/etc/rc.d/rc2.d/[SK]#{@new_resource.service_name}"])
 
           priority = {}
 

data/lib/chef/provider/service/arch.rb

@@ -20,7 +20,7 @@ require_relative "init"
 
 class Chef::Provider::Service::Arch < Chef::Provider::Service::Init
 
-  provides :service, platform_family: "arch"
+  provides :service, platform_family: "arch", target_mode: true
 
   def self.supports?(resource, action)
     service_script_exist?(:etc_rcd, resource.service_name)
@@ -32,8 +32,8 @@ class Chef::Provider::Service::Arch < Chef::Provider::Service::Init
   end
 
   def load_current_resource
-    raise Chef::Exceptions::Service, "Could not find /etc/rc.conf" unless ::File.exist?("/etc/rc.conf")
-    raise Chef::Exceptions::Service, "No DAEMONS found in /etc/rc.conf" unless /DAEMONS=\((.*)\)/m.match?(::File.read("/etc/rc.conf"))
+    raise Chef::Exceptions::Service, "Could not find /etc/rc.conf" unless ::TargetIO::File.exist?("/etc/rc.conf")
+    raise Chef::Exceptions::Service, "No DAEMONS found in /etc/rc.conf" unless /DAEMONS=\((.*)\)/m.match?(::TargetIO::File.read("/etc/rc.conf"))
 
     super
 
@@ -50,7 +50,7 @@ class Chef::Provider::Service::Arch < Chef::Provider::Service::Init
   #   )
   def daemons
     entries = []
-    if ::File.read("/etc/rc.conf") =~ /DAEMONS=\((.*)\)/m
+    if ::TargetIO::File.read("/etc/rc.conf") =~ /DAEMONS=\((.*)\)/m
       entries += $1.gsub(/\\?[\r\n]/, " ").gsub(/# *[^ ]+/, " ").split(" ") if $1.length > 0
     end
 
@@ -61,8 +61,8 @@ class Chef::Provider::Service::Arch < Chef::Provider::Service::Init
 
   # FIXME: Multiple entries of DAEMONS will cause very bad results :)
   def update_daemons(entries)
-    content = ::File.read("/etc/rc.conf").gsub(/DAEMONS=\((.*)\)/m, "DAEMONS=(#{entries.join(" ")})")
-    ::File.open("/etc/rc.conf", "w") do |f|
+    content = ::TargetIO::File.read("/etc/rc.conf").gsub(/DAEMONS=\((.*)\)/m, "DAEMONS=(#{entries.join(" ")})")
+    ::TargetIO::File.open("/etc/rc.conf", "w") do |f|
       f.write(content)
     end
   end

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

@@ -22,7 +22,7 @@ class Chef
   class Provider
     class Service
       class Debian < Chef::Provider::Service::Init
-        provides :service, platform_family: "debian" do
+        provides :service, platform_family: "debian", target_mode: true do
           debianrcd?
         end
 
@@ -46,7 +46,7 @@ class Chef
           shared_resource_requirements
           requirements.assert(:all_actions) do |a|
             update_rcd = "/usr/sbin/update-rc.d"
-            a.assertion { ::File.exist? update_rcd }
+            a.assertion { ::TargetIO::File.exist? update_rcd }
             a.failure_message Chef::Exceptions::Service, "#{update_rcd} does not exist!"
             # no whyrun recovery - this is a base system component of debian
             # distros and must be present
@@ -73,10 +73,10 @@ class Chef
 
         # returns a list of levels that the service should be stopped or started on
         def parse_init_file(path)
-          return [] unless ::File.exist?(path)
+          return [] unless ::TargetIO::File.exist?(path)
 
           in_info = false
-          ::File.readlines(path).each_with_object([]) do |line, acc|
+          ::TargetIO::File.readlines(path).each_with_object([]) do |line, acc|
             if /^### BEGIN INIT INFO/.match?(line)
               in_info = true
             elsif /^### END INIT INFO/.match?(line)
@@ -95,7 +95,7 @@ class Chef
 
           levels = parse_init_file(@init_command)
           levels.each do |level|
-            rc_files.push Dir.glob("/etc/rc#{level}.d/[SK][0-9][0-9]#{current_resource.service_name}")
+            rc_files.push TargetIO::Dir.glob("/etc/rc#{level}.d/[SK][0-9][0-9]#{current_resource.service_name}")
           end
 
           rc_files.flatten.each do |line|

data/lib/chef/provider/service/freebsd.rb

@@ -26,7 +26,7 @@ class Chef
 
         attr_reader :enabled_state_found
 
-        provides :service, os: %w{freebsd netbsd}
+        provides :service, os: %w{freebsd netbsd}, target_mode: true
 
         include Chef::Mixin::ShellOut
 
@@ -34,9 +34,9 @@ class Chef
           super
           @enabled_state_found = false
           @init_command = nil
-          if ::File.exist?("/etc/rc.d/#{new_resource.service_name}")
+          if ::TargetIO::File.exist?("/etc/rc.d/#{new_resource.service_name}")
             @init_command = "/etc/rc.d/#{new_resource.service_name}"
-          elsif ::File.exist?("/usr/local/etc/rc.d/#{new_resource.service_name}")
+          elsif ::TargetIO::File.exist?("/usr/local/etc/rc.d/#{new_resource.service_name}")
             @init_command = "/usr/local/etc/rc.d/#{new_resource.service_name}"
           end
         end
@@ -118,11 +118,11 @@ class Chef
         private
 
         def read_rc_conf
-          ::File.open("/etc/rc.conf", "r", &:readlines)
+          ::TargetIO::File.open("/etc/rc.conf", "r", &:readlines)
         end
 
         def write_rc_conf(lines)
-          ::File.open("/etc/rc.conf", "w") do |file|
+          ::TargetIO::File.open("/etc/rc.conf", "w") do |file|
             lines.each { |line| file.puts(line) }
           end
         end
@@ -131,7 +131,7 @@ class Chef
         def service_enable_variable_name
           @service_enable_variable_name ||=
             if init_command
-              ::File.open(init_command) do |rcscript|
+              ::TargetIO::File.open(init_command) do |rcscript|
                 rcscript.each_line do |line|
                   if line =~ /^name="?(\w+)"?/
                     return $1 + "_enable"
@@ -150,7 +150,7 @@ class Chef
 
         def determine_enabled_status!
           var_name = service_enable_variable_name
-          if ::File.exist?("/etc/rc.conf") && var_name
+          if ::TargetIO::File.exist?("/etc/rc.conf") && var_name
             read_rc_conf.each do |line|
               case line
               when /^#{Regexp.escape(var_name)}="(\w+)"/

data/lib/chef/provider/service/gentoo.rb

@@ -22,7 +22,7 @@ require_relative "../../util/path_helper"
 
 class Chef::Provider::Service::Gentoo < Chef::Provider::Service::Init
 
-  provides :service, platform_family: "gentoo"
+  provides :service, platform_family: "gentoo", target_mode: true
 
   def load_current_resource
     supports[:status] = true if supports[:status].nil?
@@ -32,10 +32,10 @@ class Chef::Provider::Service::Gentoo < Chef::Provider::Service::Init
     super
 
     @current_resource.enabled(
-      Dir.glob("/etc/runlevels/**/#{Chef::Util::PathHelper.escape_glob_dir(@current_resource.service_name)}").any? do |file|
+      TargetIO::Dir.glob("/etc/runlevels/**/#{Chef::Util::PathHelper.escape_glob_dir(@current_resource.service_name)}").any? do |file|
         @found_script = true
-        exists = ::File.exist? file
-        readable = ::File.readable? file
+        exists = ::TargetIO::File.exist? file
+        readable = ::TargetIO::File.readable? file
         logger.trace "#{@new_resource} exists: #{exists}, readable: #{readable}"
         exists && readable
       end
@@ -47,7 +47,7 @@ class Chef::Provider::Service::Gentoo < Chef::Provider::Service::Init
 
   def define_resource_requirements
     requirements.assert(:all_actions) do |a|
-      a.assertion { ::File.exist?("/sbin/rc-update") }
+      a.assertion { ::TargetIO::File.exist?("/sbin/rc-update") }
       a.failure_message Chef::Exceptions::Service, "/sbin/rc-update does not exist"
       # no whyrun recovery -t his is a core component whose presence is
       # unlikely to be affected by what we do in the course of a chef run

data/lib/chef/provider/service/init.rb

@@ -26,7 +26,7 @@ class Chef
 
         attr_accessor :init_command
 
-        provides :service, os: "!windows"
+        provides :service, os: "!windows", target_mode: true
 
         def self.supports?(resource, action)
           service_script_exist?(:initd, resource.service_name)
@@ -42,7 +42,7 @@ class Chef
           shared_resource_requirements
           requirements.assert(:start, :stop, :restart, :reload) do |a|
             a.assertion do
-              custom_command_for_action?(action) || ::File.exist?(default_init_command)
+              custom_command_for_action?(action) || ::TargetIO::File.exist?(default_init_command)
             end
             a.failure_message(Chef::Exceptions::Service, "#{default_init_command} does not exist!")
             a.whyrun("Init script '#{default_init_command}' doesn't exist, assuming a prior action would have created it.") do

data/lib/chef/provider/service/insserv.rb

@@ -24,7 +24,7 @@ class Chef
     class Service
       class Insserv < Chef::Provider::Service::Init
 
-        provides :service, platform_family: %w{debian rhel fedora suse amazon} do
+        provides :service, platform_family: %w{debian rhel fedora suse amazon}, target_mode: true do
           insserv?
         end
 
@@ -38,7 +38,7 @@ class Chef
           # Look for a /etc/rc.*/SnnSERVICE link to signify that the service would be started in a runlevel
           service_name = Chef::Util::PathHelper.escape_glob_dir(current_resource.service_name)
 
-          if Dir.glob("/etc/rc*/**/S*#{service_name}").empty?
+          if TargetIO::Dir.glob("/etc/rc*/**/S*#{service_name}").empty?
             current_resource.enabled false
           else
             current_resource.enabled true

data/lib/chef/provider/service/invokercd.rb

@@ -23,7 +23,7 @@ class Chef
     class Service
       class Invokercd < Chef::Provider::Service::Init
 
-        provides :service, platform_family: "debian", override: true do
+        provides :service, platform_family: "debian", override: true, target_mode: true do
           invokercd?
         end
 

data/lib/chef/provider/service/openbsd.rb

@@ -24,7 +24,7 @@ class Chef
     class Service
       class Openbsd < Chef::Provider::Service::Init
 
-        provides :service, os: "openbsd"
+        provides :service, os: "openbsd", target_mode: true
 
         attr_reader :init_command, :rc_conf, :rc_conf_local, :enabled_state_found
 
@@ -33,9 +33,9 @@ class Chef
 
         def initialize(new_resource, run_context)
           super
-          @rc_conf = ::File.read(RC_CONF_PATH) rescue ""
-          @rc_conf_local = ::File.read(RC_CONF_LOCAL_PATH) rescue ""
-          @init_command = ::File.exist?(rcd_script_path) ? rcd_script_path : nil
+          @rc_conf = ::TargetIO::File.read(RC_CONF_PATH) rescue ""
+          @rc_conf_local = ::TargetIO::File.read(RC_CONF_LOCAL_PATH) rescue ""
+          @init_command = ::TargetIO::File.exist?(rcd_script_path) ? rcd_script_path : nil
           new_resource.status_command("#{default_init_command} check")
         end
 
@@ -129,8 +129,8 @@ class Chef
         end
 
         def update_rcl(value)
-          FileUtils.touch RC_CONF_LOCAL_PATH unless ::File.exist? RC_CONF_LOCAL_PATH
-          ::File.write(RC_CONF_LOCAL_PATH, value)
+          TargetIO::FileUtils.touch RC_CONF_LOCAL_PATH unless ::TargetIO::File.exist? RC_CONF_LOCAL_PATH
+          ::TargetIO::File.write(RC_CONF_LOCAL_PATH, value)
           @rc_conf_local = value
         end
 
@@ -139,7 +139,7 @@ class Chef
           @bsevn ||= begin
             result = nil
             if rcd_script_found?
-              ::File.open(init_command) do |rcscript|
+              ::TargetIO::File.open(init_command) do |rcscript|
                 if m = rcscript.read.match(/^# \$OpenBSD: (\w+)[(.rc),]?/)
                   result = m[1] + "_flags"
                 end

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

@@ -28,7 +28,7 @@ class Chef
         # @api private
         attr_accessor :current_run_levels
 
-        provides :service, platform_family: "rpm_based" do
+        provides :service, platform_family: "rpm_based", target_mode: true do
           redhatrcd?
         end
 
@@ -56,7 +56,7 @@ class Chef
 
           requirements.assert(:all_actions) do |a|
             chkconfig_file = "/sbin/chkconfig"
-            a.assertion { ::File.exist? chkconfig_file }
+            a.assertion { ::TargetIO::File.exist? chkconfig_file }
             a.failure_message Chef::Exceptions::Service, "#{chkconfig_file} does not exist!"
           end
 
@@ -80,7 +80,7 @@ class Chef
 
           super
 
-          if ::File.exist?("/sbin/chkconfig")
+          if ::TargetIO::File.exist?("/sbin/chkconfig")
             chkconfig = shell_out!("/sbin/chkconfig --list #{current_resource.service_name}", returns: [0, 1])
             unless run_levels.nil? || run_levels.empty?
               all_levels_match = true

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

@@ -25,7 +25,7 @@ class Chef
       class Solaris < Chef::Provider::Service
         attr_reader :maintenance
 
-        provides :service, os: "solaris2"
+        provides :service, os: "solaris2", target_mode: true
 
         def initialize(new_resource, run_context = nil)
           super
@@ -39,7 +39,7 @@ class Chef
           @current_resource.service_name(@new_resource.service_name)
 
           [@init_command, @status_command].each do |cmd|
-            unless ::File.executable? cmd
+            unless ::TargetIO::File.executable? cmd
               raise Chef::Exceptions::Service, "#{cmd} not executable!"
             end
           end

data/lib/chef/provider/service/systemd.rb

@@ -26,7 +26,7 @@ class Chef::Provider::Service::Systemd < Chef::Provider::Service::Simple
 
   include Chef::Mixin::Which
 
-  provides :service, os: "linux", target_mode: true do |node|
+  provides(:service, os: "linux", target_mode: true) do |node|
     systemd?
   end
 
@@ -106,7 +106,7 @@ class Chef::Provider::Service::Systemd < Chef::Provider::Service::Simple
     if new_resource.user
       raise NotImplementedError, "#{new_resource} does not support the user property on a target_mode host (yet)" if Chef::Config.target_mode?
 
-      uid = Etc.getpwnam(new_resource.user).uid
+      uid = TargetIO::Etc.getpwnam(new_resource.user).uid
       options = {
         environment: {
           "DBUS_SESSION_BUS_ADDRESS" => "unix:path=/run/user/#{uid}/bus",

data/lib/chef/provider/service/upstart.rb

@@ -124,9 +124,9 @@ class Chef
             end
           end
           # Get enabled/disabled state by reading job configuration file
-          if ::File.exist?("#{@upstart_job_dir}/#{@new_resource.service_name}#{@upstart_conf_suffix}")
+          if ::TargetIO::File.exist?("#{@upstart_job_dir}/#{@new_resource.service_name}#{@upstart_conf_suffix}")
             logger.trace("#{@new_resource} found #{@upstart_job_dir}/#{@new_resource.service_name}#{@upstart_conf_suffix}")
-            ::File.open("#{@upstart_job_dir}/#{@new_resource.service_name}#{@upstart_conf_suffix}", "r") do |file|
+            ::TargetIO::File.open("#{@upstart_job_dir}/#{@new_resource.service_name}#{@upstart_conf_suffix}", "r") do |file|
               while line = file.gets
                 case line
                 when /^start on/

data/lib/chef/provider/service/windows.rb

@@ -74,7 +74,6 @@ class Chef::Provider::Service::Windows < Chef::Provider::Service
       current_resource.run_as_user(config_info.service_start_name)    if config_info.service_start_name
       current_resource.display_name(config_info.display_name)         if config_info.display_name
       current_resource.delayed_start(current_delayed_start)           if current_delayed_start
-      current_resource.description(config_info.description)           if new_resource.description
     end
 
     current_resource

data/lib/chef/provider/subversion.rb

@@ -27,7 +27,7 @@ class Chef
   class Provider
     class Subversion < Chef::Provider
 
-      provides :subversion
+      provides :subversion, target_mode: true
 
       SVN_INFO_PATTERN = /^([\w\s]+): (.+)$/.freeze
 
@@ -48,7 +48,7 @@ class Chef
           # Make sure the parent dir exists, or else fail.
           # for why run, print a message explaining the potential error.
           parent_directory = ::File.dirname(new_resource.destination)
-          a.assertion { ::File.directory?(parent_directory) }
+          a.assertion { ::TargetIO::File.directory?(parent_directory) }
           a.failure_message(Chef::Exceptions::MissingParentDirectory,
             "Cannot clone #{new_resource} to #{new_resource.destination}, the enclosing directory #{parent_directory} does not exist")
           a.whyrun("Directory #{parent_directory} does not exist, assuming it would have been created")
@@ -81,7 +81,7 @@ class Chef
 
       action :sync, description: "Update the source to the specified version, or get a new clone or checkout. This action causes a hard reset of the index and working tree, discarding any uncommitted changes." do
         assert_target_directory_valid!
-        if ::File.exist?(::File.join(new_resource.destination, ".svn"))
+        if ::TargetIO::File.exist?(::File.join(new_resource.destination, ".svn"))
           current_rev = find_current_revision
           logger.trace "#{new_resource} current revision: #{current_rev} target revision: #{revision_int}"
           unless current_revision_matches_target_revision?
@@ -134,7 +134,7 @@ class Chef
       alias :revision_slug :revision_int
 
       def find_current_revision
-        return nil unless ::File.exist?(::File.join(new_resource.destination, ".svn"))
+        return nil unless ::TargetIO::File.exist?(::File.join(new_resource.destination, ".svn"))
 
         command = scm(:info)
         svn_info = shell_out!(command, **run_options(cwd: cwd, returns: [0, 1])).stdout
@@ -214,7 +214,7 @@ class Chef
       end
 
       def target_dir_non_existent_or_empty?
-        !::File.exist?(new_resource.destination) || Dir.entries(new_resource.destination).sort == [".", ".."]
+        !::TargetIO::File.exist?(new_resource.destination) || TargetIO::Dir.entries(new_resource.destination).sort == [".", ".."]
       end
 
       def svn_binary
@@ -224,7 +224,7 @@ class Chef
 
       def assert_target_directory_valid!
         target_parent_directory = ::File.dirname(new_resource.destination)
-        unless ::File.directory?(target_parent_directory)
+        unless ::TargetIO::File.directory?(target_parent_directory)
           msg = "Cannot clone #{new_resource} to #{new_resource.destination}, the enclosing directory #{target_parent_directory} does not exist"
           raise Chef::Exceptions::MissingParentDirectory, msg
         end
@@ -238,9 +238,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