#717 Fix rules broken due to bodhi 4 release
Merged by laxathom. Opened by misc.
misc/fedora-badges fix_bodhi  into  master

Download 717.patch

See https://github.com/fedora-infra/bodhi/pull/3144

people in sysadmin-badges are getting a ton of mail due to that.

See https://pagure.io/fedora-badges/pull-request/691

Namely "how do the conditions work in order to sum up the old and the new messages?"

when was msg.update.user.name introduced ? Cause maybe the new format is sufficient. Since the url I linked on the breakage speak of bodhi 1.0, I suspect the update.user.name was added in bodhi 2.0.1 in september 2015.

The badges date back to 2013, so I suspect that indeed, we would need to do some magic to get the information in 2 formats.

in fact no. Since we are using the msg.update.user.name from the current message, we do not need to take anything else in account.

Take for example rules/bodhi-binary-star.yml. We do a query on users based on the current message, in "users". So that part is already handled. And we do substitute for recipient, and again, that part is done for the current message.

So to me, that should be good.

Is there anybody who can confirm or test that assumption? Because I certainly don't.

We can also decide to bite the bullet and merge this.

I'm ok w/ merging it

Pull-Request has been merged by laxathom

Metadata