{"action":"created","issue":{"url":"https://api.github.com/repos/chef/chef/issues/3207","labels_url":"https://api.github.com/repos/chef/chef/issues/3207/labels{/name}","comments_url":"https://api.github.com/repos/chef/chef/issues/3207/comments","events_url":"https://api.github.com/repos/chef/chef/issues/3207/events","html_url":"https://github.com/chef/chef/pull/3207","id":67332016,"number":3207,"title":"patch to always run exception handlers","user":{"login":"Igorshp","id":3381045,"avatar_url":"https://avatars.githubusercontent.com/u/3381045?v=3","gravatar_id":"","url":"https://api.github.com/users/Igorshp","html_url":"https://github.com/Igorshp","followers_url":"https://api.github.com/users/Igorshp/followers","following_url":"https://api.github.com/users/Igorshp/following{/other_user}","gists_url":"https://api.github.com/users/Igorshp/gists{/gist_id}","starred_url":"https://api.github.com/users/Igorshp/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Igorshp/subscriptions","organizations_url":"https://api.github.com/users/Igorshp/orgs","repos_url":"https://api.github.com/users/Igorshp/repos","events_url":"https://api.github.com/users/Igorshp/events{/privacy}","received_events_url":"https://api.github.com/users/Igorshp/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/chef/chef/labels/Signed%20CLA","name":"Signed CLA","color":"ededed"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":13,"created_at":"2015-04-09T10:33:55Z","updated_at":"2015-04-13T22:55:45Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/chef/chef/pulls/3207","html_url":"https://github.com/chef/chef/pull/3207","diff_url":"https://github.com/chef/chef/pull/3207.diff","patch_url":"https://github.com/chef/chef/pull/3207.patch"},"body":"Patch for issue https://github.com/chef/chef/issues/3180\r\n\r\nActual problem:\r\nmissing recipe present in run_list - exception handlers work as expected\r\nmissing role present in run_list - exception handlers do not fire.\r\n\r\nException handlers require @run_status to be set.\r\nMoving the declaration to initialisation block ensures it's always available.\r\n\r\n'node' is nil at that stage and is set again here https://github.com/chef/chef/blob/master/lib/chef/client.rb#L244\r\n\r\nbig thanks to @jonlives, Coderanger and @thommay your help"},"comment":{"url":"https://api.github.com/repos/chef/chef/issues/comments/92525701","html_url":"https://github.com/chef/chef/pull/3207#issuecomment-92525701","issue_url":"https://api.github.com/repos/chef/chef/issues/3207","id":92525701,"user":{"login":"Igorshp","id":3381045,"avatar_url":"https://avatars.githubusercontent.com/u/3381045?v=3","gravatar_id":"","url":"https://api.github.com/users/Igorshp","html_url":"https://github.com/Igorshp","followers_url":"https://api.github.com/users/Igorshp/followers","following_url":"https://api.github.com/users/Igorshp/following{/other_user}","gists_url":"https://api.github.com/users/Igorshp/gists{/gist_id}","starred_url":"https://api.github.com/users/Igorshp/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Igorshp/subscriptions","organizations_url":"https://api.github.com/users/Igorshp/orgs","repos_url":"https://api.github.com/users/Igorshp/repos","events_url":"https://api.github.com/users/Igorshp/events{/privacy}","received_events_url":"https://api.github.com/users/Igorshp/received_events","type":"User","site_admin":false},"created_at":"2015-04-13T22:55:44Z","updated_at":"2015-04-13T22:55:44Z","body":"@lamont-granquist \r\nI've done some tests to see what the value of @subscribers inside the @events is at various points and you are indeed right - the contents of @events inside main loop is same as inside @run_status. There is no need to set events again.\r\n\r\nI'll update the PR shortly"}}