scm_request - migrate ticket handling to Fedora Forge #429

Merged
zlopez merged 5 commits from jnsamyak/toddlers:migrate_scm_requests into main 2026-08-05 12:48:55 +00:00
Owner

Please check issue for more details: releng/tickets#13370 (comment)

Please check issue for more details: https://forge.fedoraproject.org/releng/tickets/issues/13370#issuecomment-1062204
Signed-off-by: Samyak Jain <samyak.jn11@gmail.com>
Signed-off-by: Samyak Jain <samyak.jn11@gmail.com>
[migration2forgejo]feat: process Forgejo tickets via forgejo util
Some checks failed
CI / mypy (pull_request) Has been cancelled
CI / format (pull_request) Has been cancelled
CI / flake8 (pull_request) Has been cancelled
CI / python3.11 (pull_request) Has been cancelled
CI / python3.12 (pull_request) Has been cancelled
6bbae261e7
Signed-off-by: Samyak Jain <samyak.jn11@gmail.com>
jnsamyak changed title from WIP: scm_request - migrate ticket handling to Fedora Forge to scm_request - migrate ticket handling to Fedora Forge 2026-07-14 14:29:38 +00:00
jnsamyak force-pushed migrate_scm_requests from 6bbae261e7
Some checks failed
CI / mypy (pull_request) Has been cancelled
CI / format (pull_request) Has been cancelled
CI / flake8 (pull_request) Has been cancelled
CI / python3.11 (pull_request) Has been cancelled
CI / python3.12 (pull_request) Has been cancelled
to e9478125cc
Some checks failed
CI / format (pull_request) Failing after 2m31s
CI / flake8 (pull_request) Successful in 2m36s
CI / mypy (pull_request) Failing after 2m49s
CI / python3.11 (pull_request) Failing after 2m55s
CI / python3.12 (pull_request) Failing after 1m32s
2026-07-14 14:42:33 +00:00
Compare
[migration2forgejo]fix: make Forgejo SCM CI green (black, mypy, coverage)
All checks were successful
CI / flake8 (pull_request) Successful in 2m33s
CI / mypy (pull_request) Successful in 2m48s
CI / format (pull_request) Successful in 2m57s
CI / python3.11 (pull_request) Successful in 2m57s
CI / python3.12 (pull_request) Successful in 1m33s
c6d6a8624c
Signed-off-by: Samyak Jain <samyak.jn11@gmail.com>
Author
Owner

Added a set of more tests to make CI happy, and had some formatting issues.

Added a set of more tests to make CI happy, and had some formatting issues.
Author
Owner

I think for reviews, the tests/ can be ignored for now, and we can jump back on the actual code to see if I'm missing any race conditions.

I think for reviews, the tests/ can be ignored for now, and we can jump back on the actual code to see if I'm missing any race conditions.
Author
Owner

But this change is to wait for fedpkg changes happening at releng/tickets#13369, I want to make sure to have things ready so we can test it out once we have fedpkg changes in.

But this change is to wait for fedpkg changes happening at https://forge.fedoraproject.org/releng/tickets/issues/13369, I want to make sure to have things ready so we can test it out once we have fedpkg changes in.
Author
Owner

Merging #430 for Anton, I think this PR will grow because I haven't accounted for unretirement toddler changes also, but I'll let it sit for one set of review.

Merging #430 for Anton, I think this PR will grow because I haven't accounted for unretirement toddler changes also, but I'll let it sit for one set of review.
[migration2forgejo] handle unretirement tickets via Forgejo
Some checks failed
CI / python3.11 (pull_request) Failing after 2m37s
CI / flake8 (pull_request) Failing after 2m57s
CI / mypy (pull_request) Successful in 3m2s
CI / python3.12 (pull_request) Failing after 1m36s
CI / format (pull_request) Failing after 2m57s
304bd230ec
Move unretire_packages ticket lifecycle off pagure.io onto Forge
(forgejo.issues), keeping dist-git/koji/bodhi unchanged.

Signed-off-by: Samyak Jain <samyak.jn11@gmail.com>
Author
Owner

I pushed changes for unretirement toddler as well, so unretirement tickets on releng/fedora-scm-requests are handled via Forgejo (forgejo.issues) instead of pagure.io. Dist-git, koji, and bodhi are unchanged. @amedvede would be nice if you could take a look at it.

I pushed changes for unretirement toddler as well, so unretirement tickets on releng/fedora-scm-requests are handled via Forgejo (forgejo.issues) instead of pagure.io. Dist-git, koji, and bodhi are unchanged. @amedvede would be nice if you could take a look at it.
jnsamyak force-pushed migrate_scm_requests from 304bd230ec
Some checks failed
CI / python3.11 (pull_request) Failing after 2m37s
CI / flake8 (pull_request) Failing after 2m57s
CI / mypy (pull_request) Successful in 3m2s
CI / python3.12 (pull_request) Failing after 1m36s
CI / format (pull_request) Failing after 2m57s
to b76f19fb00
All checks were successful
CI / flake8 (pull_request) Successful in 2m27s
CI / mypy (pull_request) Successful in 2m38s
CI / format (pull_request) Successful in 2m37s
CI / python3.11 (pull_request) Successful in 1m49s
CI / python3.12 (pull_request) Successful in 1m54s
2026-07-20 13:44:07 +00:00
Compare
Author
Owner

Added a test, and cosmetic changes to make CI green, let's see

Added a test, and cosmetic changes to make CI green, let's see
zlopez approved these changes 2026-08-04 12:13:14 +00:00
zlopez left a comment

I went through the changes and they look good to me.

I went through the changes and they look good to me.
Author
Owner

Thanks, I'll merge this up. @zlopez should I just cherry-pick these to staging and production directly or open PRs?

Thanks, I'll merge this up. @zlopez should I just cherry-pick these to staging and production directly or open PRs?
Author
Owner

Also just merge this, or squash out the commits?

Also just merge this, or squash out the commits?
Owner

@jnsamyak wrote in #429 (comment):

Thanks, I'll merge this up. @zlopez should I just cherry-pick these to staging and production directly or open PRs?

I think that the staging and production branch should now be at the same head as master, so there should be no need for cherry pick.

@jnsamyak wrote in #429 (comment):

Also just merge this, or squash out the commits?

Let me just merge this.

@jnsamyak wrote in https://forge.fedoraproject.org/apps/toddlers/pulls/429#issuecomment-1081017: > Thanks, I'll merge this up. @zlopez should I just cherry-pick these to staging and production directly or open PRs? I think that the staging and production branch should now be at the same head as master, so there should be no need for cherry pick. @jnsamyak wrote in https://forge.fedoraproject.org/apps/toddlers/pulls/429#issuecomment-1081018: > Also just merge this, or squash out the commits? Let me just merge this.
zlopez merged commit b76f19fb00 into main 2026-08-05 12:48:55 +00:00
Sign in to join this conversation.
No reviewers
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
apps/toddlers!429
No description provided.