#188 callback fixup
Merged by mikem. Opened by mikeb.
mikeb/koji callback-fixup  into  master

Download 188.patch

The first commit makes the postImport callback more consistent (build info is always passed in via the build argument).

The second commit changes the arguments passed to the pre/postRPMSign callback. The callback previously passed unnecessary arguments (attribute, old) and put everything else into the "info" argument. This made it ambiguous as to what the fields in info were referring to. The new arguments are more clear, and provide direct access to all information a callback handler is most likely to be interested in.

It would have been possible to make this an additive-only change by pushing the build information into the info dict. However, this would have retained the unnecessary arguments, and made the ambiguity of the name/version/release fields in the info dict worse. This API change cleans things up, and should be minimally disruptive.

Looks fine, but let's give folks a few days to respond on the list

Fix in fedmsg.meta to handle the format change here: https://github.com/fedora-infra/fedmsg_meta_fedora_infrastructure/pull/403

I've seen no objections. Could we go ahead and merge this?

Pull-Request has been merged by mikem

Metadata