PR#47: Reset the logs in packagers_without_bugzilla for every message - fedora-infra/toddlers

fedora-infra / toddlers

#47 Reset the logs in packagers_without_bugzilla for every message

Merged 3 years ago by pingou. Opened 3 years ago by pingou.

clean_logs into master

Reset the logs in packagers_without_bugzilla for every message

Pierre-Yves Chibon • 3 years ago

44fc13a

toddlers/plugins/packagers_without_bugzilla.py

file modified

		`@@ -88,6 +88,7 @@`

		`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"])`

pingou commented 3 years ago

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 pingou@pingoured.fr