From c054f150f3fe9405208ff72232069eb4ac33d0cc Mon Sep 17 00:00:00 2001 From: Ralph Bean Date: Sat, 26 Sep 2015 15:56:29 +0000 Subject: [PATCH] Remove the failed flag on the fedmsg callback. It was born of confusion during the batcave migration. --- callback_plugins/fedmsg_callback.py | 8 -------- 1 file changed, 8 deletions(-) diff --git a/callback_plugins/fedmsg_callback.py b/callback_plugins/fedmsg_callback.py index dc48692cac..5dd5509089 100644 --- a/callback_plugins/fedmsg_callback.py +++ b/callback_plugins/fedmsg_callback.py @@ -46,19 +46,13 @@ class CallbackModule(object): # It seems like recursive playbooks call this over and over again and # fedmsg doesn't like to be initialized more than once. So, here, just # catch that and ignore it. - self.failed = False try: fedmsg.init(**config) except ValueError: - import traceback - traceback.print_exc() - self.failed = True pass def playbook_on_play_start(self, pattern): - if self.failed: - return # This gets called once for each play.. but we just issue a message once # for the first one. One per "playbook" play = getattr(self, 'play', None) @@ -85,8 +79,6 @@ class CallbackModule(object): self.playbook_path = path def playbook_on_stats(self, stats): - if self.failed: - return if not self.playbook_path: return