From 44fc13a601761317c8e710c1026a51beb3a3381b Mon Sep 17 00:00:00 2001 From: Pierre-Yves Chibon Date: Sep 17 2020 13:36:53 +0000 Subject: Reset the logs in packagers_without_bugzilla for every message Otherwise we keep in memory the logs from the previous runs which renders the email sent to the admins a little hard to read. For example on day one, you have user1 and user2 without a bugzilla account (then logs contain user1 and user2). Then on day two, user1 and user3 are also without a valid bugzilla account (logs = user1, user2, user1, user3) then on day three only user3 is left (logs = user1, user2, user1, user3, user3) and so on... Signed-off-by: Pierre-Yves Chibon --- diff --git a/toddlers/plugins/packagers_without_bugzilla.py b/toddlers/plugins/packagers_without_bugzilla.py index b1bc7cc..4a5e47b 100644 --- a/toddlers/plugins/packagers_without_bugzilla.py +++ b/toddlers/plugins/packagers_without_bugzilla.py @@ -88,6 +88,7 @@ class PackagersWithoutBugzilla(ToddlerBase): def process(self, config, message, send_email=True, username=None): """ Looks for packagers/groups without bugzilla email. """ + self.logs = [] try: email_overrides = toml.load(config["email_overrides_file"])