summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authordanielsdeleo <dan@chef.io>2017-04-10 13:01:14 -0700
committerdanielsdeleo <dan@chef.io>2017-04-10 13:01:14 -0700
commit08fbff8cac3af16abc3d5fc344603218792dc370 (patch)
treeeb8f22dcf25a60226774d9455803bf568d57bfe8
parentc6b4ed4b4726dfc9375b5acb0a06286f2f21262e (diff)
downloadchef-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.rb3
-rw-r--r--spec/unit/client_spec.rb2
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