move save_fileinfo work back inside of a converge_by() block

1 files changed, 9 insertions, 7 deletions

diff --git a/lib/chef/provider/remote_file.rb b/lib/chef/provider/remote_file.rb
index 057f46d3c3..b5131e5d1e 100644
--- a/lib/chef/provider/remote_file.rb
+++ b/lib/chef/provider/remote_file.rb
@@ -55,13 +55,15 @@ class Chef
       end
 
       def save_fileinfo(source)
-        cache = Hash.new
-        cache["etag"] = @new_resource.etag
-        cache["last_modified"] = @new_resource.last_modified
-        cache["src"] = source
-        cache["checksum"] = @new_resource.checksum
-        cache_path = new_resource.name.sub(/^([A-Za-z]:)/, "")  # strip drive letter on Windows
-        Chef::FileCache.store("remote_file/#{cache_path}", cache.to_json)
+        converge_by("update saved etag/last_modified information on file") do
+          cache = Hash.new
+          cache["etag"] = @new_resource.etag
+          cache["last_modified"] = @new_resource.last_modified
+          cache["src"] = source
+          cache["checksum"] = @new_resource.checksum
+          cache_path = new_resource.name.sub(/^([A-Za-z]:)/, "")  # strip drive letter on Windows
+          Chef::FileCache.store("remote_file/#{cache_path}", cache.to_json)
+        end
       end
     end
   end