freshmaker

#98 Do not print whole message in case BaseEvent was not created - print just message id instead.
Merged by jkaluza. Opened by jkaluza.
jkaluza/freshmaker consumer-too-verbose  into  master

Download 98.patch

This makes logs much easier to read...

I recommend message.get('msg_id') just to be on the safe side.

From my point of view, BaseEvent not created for message is a little confused. Should it be "No event is defined for and can parse message with id %s" or something like?

@ralph: It does not show the whole code, but the code right above this part is:

        if 'msg_id' not in message:
            raise ValueError(
                'Received message does not contain "msg_id" or "message-id": '
                '%r' % (message))

It does not show the whole code, but the code right above this part
Gotcha. Thanks!

rebased onto d82a0129a4037b8db2d85383a15f8f68a71d3ad0

Pull-Request has been merged by jkaluza
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.