diff options
author | Phil Dibowitz <phil@ipom.com> | 2021-03-06 19:10:18 -0800 |
---|---|---|
committer | Phil Dibowitz <phil@ipom.com> | 2021-03-06 19:24:59 -0800 |
commit | a1e0630be7f505b99ecba23c1a9994809c964618 (patch) | |
tree | 103a09bed505bdf0c5b9a1e3a2c95d229c4aac54 /lib/chef/provider/subversion.rb | |
parent | 228291d27b8f1827f98911b6b0192d1efe06e4a3 (diff) | |
download | chef-a1e0630be7f505b99ecba23c1a9994809c964618.tar.gz |
Move idempotency logs to debug
This was discussed in the last Chef Triage meeting.
In the great move to trace logging, which is great, idempotency check
logs moved to trace, which defeats the purpose. The goal is to keep
"internal chef" logging in trace so that when users are trying to figure
out why their cookbook isn't acting as expected they can look at debug
without having to see all the other cruft. But "here's why I didn't
install the package you said to install" is table-steaks "I'm trying to
debug my cookbook."
This moves those logs back to debug. There were a variety of cases where
logging was pretty nonstandard in other ways, but to keep this PR
scoped, I didn't change those.
I did fix a few related logging issues:
* A few warns that should have been debug in windows_task
* A few places where there was basically no logging whatsoever, I put a
few of the "I did X" logs to 'debug' (from trace). It coudl be argued
some of those should be 'info', but that's a bigger change that I
didn't want to put into this PR
Signed-off-by: Phil Dibowitz <phil@ipom.com>
Diffstat (limited to 'lib/chef/provider/subversion.rb')
-rw-r--r-- | lib/chef/provider/subversion.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/chef/provider/subversion.rb b/lib/chef/provider/subversion.rb index 18fc9d3a3c..2cc4cb7ab3 100644 --- a/lib/chef/provider/subversion.rb +++ b/lib/chef/provider/subversion.rb @@ -61,7 +61,7 @@ class Chef shell_out!(checkout_command, run_options) end else - logger.trace "#{new_resource} checkout destination #{new_resource.destination} already exists or is a non-empty directory - nothing to do" + logger.debug "#{new_resource} checkout destination #{new_resource.destination} already exists or is a non-empty directory - nothing to do" end end @@ -69,7 +69,7 @@ class Chef if target_dir_non_existent_or_empty? action_force_export else - logger.trace "#{new_resource} export destination #{new_resource.destination} already exists or is a non-empty directory - nothing to do" + logger.debug "#{new_resource} export destination #{new_resource.destination} already exists or is a non-empty directory - nothing to do" end end |