diff options
author | danielsdeleo <dan@chef.io> | 2017-04-10 13:01:14 -0700 |
---|---|---|
committer | danielsdeleo <dan@chef.io> | 2017-04-10 13:01:14 -0700 |
commit | 08fbff8cac3af16abc3d5fc344603218792dc370 (patch) | |
tree | eb8f22dcf25a60226774d9455803bf568d57bfe8 | |
parent | c6b4ed4b4726dfc9375b5acb0a06286f2f21262e (diff) | |
download | chef-08fbff8cac3af16abc3d5fc344603218792dc370.tar.gz |
Clarify log messaging for required recipe feature
The previous log message didn't have enough context and could sound
"scary" to users unfamiliar with the feature.
Signed-off-by: Daniel DeLeo <dan@chef.io>
-rw-r--r-- | lib/chef/client.rb | 3 | ||||
-rw-r--r-- | spec/unit/client_spec.rb | 2 |
2 files changed, 3 insertions, 2 deletions
diff --git a/lib/chef/client.rb b/lib/chef/client.rb index 06325e0a23..9f8a34e85e 100644 --- a/lib/chef/client.rb +++ b/lib/chef/client.rb @@ -532,6 +532,7 @@ class Chef # def load_required_recipe(rest, run_context) required_recipe_contents = rest.get("required_recipe") + Chef::Log.info("Required Recipe found, loading it") Chef::FileCache.store("required_recipe", required_recipe_contents) required_recipe_file = Chef::FileCache.load("required_recipe", false) @@ -552,7 +553,7 @@ class Chef rescue Net::HTTPServerException => e case e.response when Net::HTTPNotFound - Chef::Log.info("Required Recipe not found") + Chef::Log.debug("Required Recipe not configured on the server, skipping it") else raise end diff --git a/spec/unit/client_spec.rb b/spec/unit/client_spec.rb index 064876a82f..d348c24385 100644 --- a/spec/unit/client_spec.rb +++ b/spec/unit/client_spec.rb @@ -437,7 +437,7 @@ EOM end it "should log and continue on" do - expect(Chef::Log).to receive(:info) + expect(Chef::Log).to receive(:debug) client.load_required_recipe(rest, run_context) end end |