summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLamont Granquist <lamont@scriptkiddie.org>2019-03-11 14:52:50 -0700
committerLamont Granquist <lamont@scriptkiddie.org>2019-03-11 14:52:50 -0700
commit5df52d2bbffedd6d010c0365957dca134f20ff35 (patch)
tree3d6a1296f19d4709f94765a7efdba9be8d435cb0
parent97c889fff07528b22745b2f59f64e29b43746644 (diff)
downloadchef-5df52d2bbffedd6d010c0365957dca134f20ff35.tar.gz
more bizarre cruft removal in Chef::Client
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
-rw-r--r--lib/chef/client.rb27
1 files changed, 7 insertions, 20 deletions
diff --git a/lib/chef/client.rb b/lib/chef/client.rb
index 466d785d00..bb80b024bd 100644
--- a/lib/chef/client.rb
+++ b/lib/chef/client.rb
@@ -265,10 +265,7 @@ class Chef
load_required_recipe(@rest, run_context) unless Chef::Config[:solo_legacy_mode]
- converge_error = converge_and_save(run_context)
-
- # Raise converge_error so run_failed reporters/events are processed.
- raise converge_error if converge_error
+ converge_and_save(run_context)
run_status.stop_clock
logger.info("Chef Run complete in #{run_status.elapsed_time} seconds")
@@ -299,10 +296,9 @@ class Chef
# Raise converge, and other errors here so that we exit
# with the proper exit status code and everything gets raised
# as a RunFailedWrappingError
- if run_error || converge_error
- error = run_error || converge_error
- Chef::Application.debug_stacktrace(error)
- raise error
+ if run_error
+ Chef::Application.debug_stacktrace(run_error)
+ raise run_error
end
true
@@ -656,7 +652,6 @@ class Chef
# @api private
#
def converge(run_context)
- converge_exception = nil
catch(:end_client_run_early) do
begin
events.converge_start(run_context)
@@ -666,10 +661,9 @@ class Chef
events.converge_complete
rescue Exception => e
events.converge_failed(e)
- converge_exception = e
+ raise e
end
end
- converge_exception
end
#
@@ -690,15 +684,8 @@ class Chef
# split this stuff up?
#
def converge_and_save(run_context)
- converge_exception = converge(run_context)
- unless converge_exception
- begin
- save_updated_node
- rescue Exception => e
- raise e
- end
- end
- converge_exception
+ converge(run_context)
+ save_updated_node
end
#