-
Notifications
You must be signed in to change notification settings - Fork 47
Use msg2long_form api for bodhi backend stuff #96
Comments
Here's an example of what bodhi needs sent: https://github.com/fedora-infra/bodhi/blob/master/bodhi/mail.py#L273 |
This will eventually be used by bodhi and FMN to send out bigger, more informative blocks of text. For fedora-infra/fedmsg_meta_fedora_infrastructure#96
Noting here that we now have the msg2long_form API and we use it many different places (git commits, github events, koji builds, copr builds, etc..). But we still need to do the implementation of producing those long bits of text for the bodhi backend process. |
@ralphbean , i would like to work on this. And the line to replace would be Let me know if i am going wrong. |
Yeah! We never quite figured out how this was supposed to work with bodhi2. There were two options we were tossing around, if I remember correctly:
@lmacken, do you have an opinion either way on this one? |
So in bodhi2 I basically ported the old mail module over for the update announcement emails that go to the package-announce-list. These are big templates that require a bunch of koji calls to generate, and that doesn't seem like something we want to do in the meta package. https://github.com/fedora-infra/bodhi/blob/pyramid/bodhi/mail.py I'm open to migrating some of those misc messages over to meta/FMN somehow. |
@ralphbean , so i guess the method 2 is what we will be implementing. In that case please guide me on how i should go ahead with this fix . |
@ghost-script would you feel comfortable getting into some of the bodhi2 code? It seems like those templates are going to need to be filled out with data from koji calls and then sent along with the data in the |
@ralphbean , i am comfortable with working with bodhi2 code. So in that case we need to implement msg2long_form api to fill the email templates with the data then push it into fedmsg. |
Not quite. I don't think it will actually make any use of the long_form or msg2* stuff on the bodhi2 side of things. I think it'll just be cut-and-dry python there. i.e. this big template will need to be filled in and stuffed into the fedmsg messages that get published from there: https://github.com/fedora-infra/bodhi/blob/pyramid/bodhi/mail.py#L237 |
Fix #96 "in advance of" should read "newer than"
We would use it for bodhi when the masher finishes in order to generate a big templated email.
We could use mako templates for it behind the scenes.
The text was updated successfully, but these errors were encountered: