#117 Support to send messages to UMB
Merged by cqi. Opened by cqi.
cqi/freshmaker send-msg  into  master

Download 117.patch

Original code in messaging is refactored so that single entry method
publish can be called to publish message via different backend messaging
brokers according to configuration in freshmaker's config file.

Message body will contains complete event and associated builds information, each of them also contains URL that could be used to link back to freshmaker for another API call.

With this patch, a message can be sent in following ways

  • for fedmsg, publish('images.ready', {}) will send message to topic freshmaker.images.ready.
  • for rhmsg, publish('images.ready', {}) will send message to topic VirtualTopic.eng.freshmaker.images.ready

rebased onto 8862dc8ffe24256b056b33a2d52b12569096b119

1 new commit added

  • Send message when rebuilt images are found

Nice! Looks good here.

@mikeb, would you like to review too?

It looks good to me, I will wait for next week for mikeb to review this one before merging. mikeb is currently away for whole week on meetings.

FYI - this needs a rebase before it can be merged.

rebased onto 8edde38cc445ee8388011b479e8d6c1053ec7a69

+1 Looks good!

Rad. @cqi, it's yours to merge. :)

Pull-Request has been merged by cqi

Metadata