Infrastructure/ansible
1
0
Fork 0
Code Issues Pull requests 6 Projects Releases Packages Wiki Activity Actions

koji: Fix sending the rpm.sign messages

Browse source 
fedmsg used to automatically serialize datetime object while
fedora-messaging doesn't.
This is fine for code-base one knows but for us on this code base
it has been a bit of a wake-a-mole approach and we were missing
one field (which this commit fixes).
In addition, we're logging the error to the console using python's
logging module, except that it looks like koji isn't quite setting
up handlers for the logs, so they were not showing anywhere :(
(We may need to add print() in the exception handling part in the
future if we run into more problems with this).

Signed-off-by: Pierre-Yves Chibon <pingou@pingoured.fr>
This commit is contained in:
Pierre-Yves Chibon 2019-09-25 21:24:45 +02:00
parent a7665c9fe1
commit 3456871b07
1 changed files with 1 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

2
roles/koji_hub/templates/fedmsg-koji-plugin.py
View file

@ -32,7 +32,7 @@ def camel_to_dots(name):
def serialize_datetime_in_task(task):
date_fields = [
"completion_time", "create_time", "start_time", "buildtime",
"creation_ts"
"creation_ts", "creation_time",
]
for date_key in date_fields:
if task.get(date_key) is None: