Fix Koji owner sync for unretirements (update package key to repo and separate branch handling) #13364

Open
opened 2026-05-25 09:13:52 +00:00 by amedvede · 3 comments
Owner

Following up on the issue identified in #13270. The post-close automation script in fedora-scm-requests currently fails to sync the package owner with Koji when an unretirement ticket is closed.

The Problem:

  • Key Mismatch: The script currently expects the package name value under the name key, but the correct key in the payload is repo.

  • Branch Processing: The script currently conflates dist-git branches with Koji tags, which causes processing errors during the ublocking tags.

Proposed Solution:

  • Update the script to fetch the package name using the repo key instead of name.

  • Refactor the branch handling logic to cleanly divide and process Koji tags and dist-git branches separately to ensure the correct values are passed to the Koji API.

Following up on the issue identified in https://forge.fedoraproject.org/releng/tickets/issues/13270. The post-close automation script in fedora-scm-requests currently fails to sync the package owner with Koji when an unretirement ticket is closed. The Problem: - Key Mismatch: The script currently expects the package name value under the `name` key, but the correct key in the payload is `repo`. - Branch Processing: The script currently conflates dist-git branches with Koji tags, which causes processing errors during the ublocking tags. Proposed Solution: - Update the script to fetch the package name using the `repo` key instead of `name`. - Refactor the branch handling logic to cleanly divide and process Koji tags and dist-git branches separately to ensure the correct values are passed to the Koji API.
Author
Owner

I changed the key for unretirement request, and separated branches on fedpkg side: PR https://pagure.io/fedpkg/pull-request/626#
also adjusted toddlers functionality to process new keys: PR apps/toddlers#428 (comment)

I changed the key for unretirement request, and separated branches on fedpkg side: PR https://pagure.io/fedpkg/pull-request/626# also adjusted toddlers functionality to process new keys: PR https://forge.fedoraproject.org/apps/toddlers/pulls/428#issuecomment-759887
Author
Owner

toddlers changes are merged and ready; fedpkg changes are also merged, waiting for a new patch of fedpkg with recent changes

`toddlers` changes are merged and ready; `fedpkg` changes are also merged, waiting for a new patch of `fedpkg` with recent changes
Owner

@amedvede left a comment at toddler PR, please take a look, and let me know what is blocking and left. Toddlers changes are not seemed to be merged, seems like a open PR.

@amedvede left a comment at toddler PR, please take a look, and let me know what is blocking and left. Toddlers changes are not seemed to be merged, seems like a open PR.
Sign in to join this conversation.
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
releng/tickets#13364
No description provided.