koji

#725 'postBuildStateChange' doesn't always contain consistent buildinfo
Closed: Fixed Opened by julian8628.

Closed: Fixed
Reopen Issue

The full buildinfo(with package_name,owner_name, etc) is obtained by get_build().
But in some places the buildinfo is assembled when creating, and then directly passed to callbacks.
In this case, some plugins like messagebus won't get the useful information like owner_name for instance.

After looking inside the code, I found here we might meet another problem that the buildinfo passed to preBuildStateChange and postBuildStateChange should be reconsidered.
Should it be a completely new information reflecting the build after the change?

@mikem any idea about this?

candidate PR #726

Metadata Update from @tkopecek:
- Issue set to the milestone: 1.16

Have we encountered actual issues with this, or is this something you noticed in the code?

Have we encountered actual issues with this, or is this something you noticed in the code?

The first one was filed by Brew user. He wants to get owner_name in CG build via UMB messages.

and the second one

After looking inside the code, I found here we might meet another problem that the buildinfo passed to preBuildStateChange and postBuildStateChange should be reconsidered.
Should it be a completely new information reflecting the build after the change?

is what I found in the code.

Commit 2b7b90a2 relates to this ticket

Commit 7c1865f9 relates to this ticket

Oops, that merge was supposed to close this

Metadata Update from @mikem:
- Issue status updated to: Closed (was: Open)

Metadata Update from @mikem:
- Issue close_status updated to: Fixed

This issue has been migrated to Fedora Forge:
https://forge.fedoraproject.org/koji/koji/issues/725

Please continue any further discussion there.
Metadata
None
None
None
None
None
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.