diff options
author | Lamont Granquist <lamont@scriptkiddie.org> | 2019-03-11 14:52:50 -0700 |
---|---|---|
committer | Lamont Granquist <lamont@scriptkiddie.org> | 2019-03-11 14:52:50 -0700 |
commit | 5df52d2bbffedd6d010c0365957dca134f20ff35 (patch) | |
tree | 3d6a1296f19d4709f94765a7efdba9be8d435cb0 | |
parent | 97c889fff07528b22745b2f59f64e29b43746644 (diff) | |
download | chef-5df52d2bbffedd6d010c0365957dca134f20ff35.tar.gz |
more bizarre cruft removal in Chef::Client
Signed-off-by: Lamont Granquist <lamont@scriptkiddie.org>
-rw-r--r-- | lib/chef/client.rb | 27 |
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 # |