refactoring 2/2 #12978

Merged
jnsamyak merged 35 commits from refactoring_2 into main 2026-06-08 03:10:21 +00:00
Owner

This Pr is the second part of refactoring for the releng repository, it touches the scripts listed in this issue https://pagure.io/releng/issue/12935. Also, this PR will contain comments that will have the following look: name_of_script -> links on PR with changes for external repos.
Signed-off-by: Anton Medvedev amedvede@redhat.com

This Pr is the second part of refactoring for the releng repository, it touches the scripts listed in this issue https://pagure.io/releng/issue/12935. Also, this PR will contain comments that will have the following look: `name_of_script` -> links on PR with changes for external repos. Signed-off-by: Anton Medvedev <amedvede@redhat.com>
Author
Owner

orphaned-packages-proccess/all_files -> https://pagure.io/infra-docs-fpo/pull-request/418

`orphaned-packages-proccess/all_files` -> https://pagure.io/infra-docs-fpo/pull-request/418
Author
Owner

1 new commit added

  • ref: distgit/ also renaming some of them
**1 new commit added** * ``ref: distgit/ also renaming some of them``
Author
Owner

2 new commits added

  • ref: distgit/ also renaming some of them
  • ref: orhaned-packages-process/
**2 new commits added** * ``ref: distgit/ also renaming some of them`` * ``ref: orhaned-packages-process/``
Author
Owner

1 new commit added

  • ref: find_tagged_retired_epel_builds.py removed, also removed link on find_unblocked_orphans.py
**1 new commit added** * ``ref: find_tagged_retired_epel_builds.py removed, also removed link on find_unblocked_orphans.py``
Author
Owner

1 new commit added

  • ref: block_retired.py
**1 new commit added** * ``ref: block_retired.py``
Author
Owner

block_retired.py -> doc change: https://pagure.io/infra-docs-fpo/pull-request/420
also depending releng scripts:find_tagged_retired_epel_builds.py(removed) and build-functions.sh(part of patrik's scripts but it also will be removed)

`block_retired.py` -> doc change: https://pagure.io/infra-docs-fpo/pull-request/420 also depending releng scripts:`find_tagged_retired_epel_builds.py`(removed) and `build-functions.sh`(part of patrik's scripts but it also will be removed)
Author
Owner

1 new commit added

  • ref: check_epel_deps.py check_epel_deps.sh
**1 new commit added** * ``ref: check_epel_deps.py check_epel_deps.sh``
Author
Owner

1 new commit added

  • ref: check-latest-build.py check-testing-pre-GA.py check-upgrade-paths.py clean-overrides.py
**1 new commit added** * ``ref: check-latest-build.py check-testing-pre-GA.py check-upgrade-paths.py clean-overrides.py``
Author
Owner

1 new commit added

  • ref: clean-amis.py
**1 new commit added** * ``ref: clean-amis.py``
Author
Owner
`clean-amis.py` -> https://pagure.io/infra-docs-fpo/pull-request/421
Author
Owner

1 new commit added

  • ref: close-old-ftbfs-bugz.sh
**1 new commit added** * ``ref: close-old-ftbfs-bugz.sh``
Author
Owner

1 new commit added

  • ref: distgit-branch-unused.py
**1 new commit added** * ``ref: distgit-branch-unused.py``
Author
Owner
`distgit-branch-unused.py` -> infra-docs-fpo: https://pagure.io/infra-docs-fpo/pull-request/422 howtos: https://pagure.io/fedora-infra/howtos/pull-request/15
Author
Owner

1 new commit added

  • ref: fedmsg-functions.sh fedora-messaging-logger fedora-messaging.sh
**1 new commit added** * ``ref: fedmsg-functions.sh fedora-messaging-logger fedora-messaging.sh``
Author
Owner

fedmsg-functions.sh (removed) ->
fedora-pungi - exchanged by fedora_messaging.sh https://pagure.io/pungi-fedora/pull-request/1539
fedora-messaging.sh (renamed) ->
fedora-pungi - changed paths https://pagure.io/pungi-fedora/pull-request/1539
fedora-messaging-logger (renamed) ->
ansible - changed the name but not sure if needed https://pagure.io/fedora-infra/ansible/pull-request/2871

Also, made a misstake and created branch in pungi-fedora repo that called ref_36, need to remove it in future

`fedmsg-functions.sh` (removed) -> fedora-pungi - exchanged by fedora_messaging.sh https://pagure.io/pungi-fedora/pull-request/1539 `fedora-messaging.sh` (renamed) -> fedora-pungi - changed paths https://pagure.io/pungi-fedora/pull-request/1539 `fedora-messaging-logger` (renamed) -> ansible - changed the name but not sure if needed https://pagure.io/fedora-infra/ansible/pull-request/2871 Also, made a misstake and created branch in pungi-fedora repo that called `ref_36`, need to remove it in future
Author
Owner

1 new commit added

  • ref: fedora_torrent_ini.py
**1 new commit added** * ``ref: fedora_torrent_ini.py``
Author
Owner

fedora_torrent_ini.py ->
doc change https://pagure.io/infra-docs-fpo/pull-request/423

`fedora_torrent_ini.py` -> doc change https://pagure.io/infra-docs-fpo/pull-request/423
Author
Owner

1 new commit added

  • ref: find_failures.py
**1 new commit added** * ``ref: find_failures.py``
Author
Owner

find_failures.py ->
doc change: https://pagure.io/infra-docs-fpo/pull-request/424
Also need to notify Patric that mass_rebuild_file_bugs.py is importing this script and paths should be changed accordingly.

`find_failures.py` -> doc change: https://pagure.io/infra-docs-fpo/pull-request/424 Also need to notify Patric that `mass_rebuild_file_bugs.py` is importing this script and paths should be changed accordingly.
Author
Owner

1 new commit added

  • ref: find_FTBFS.py
**1 new commit added** * ``ref: find_FTBFS.py``
Author
Owner

1 new commit added

  • ref: find_bad_builds.py
**1 new commit added** * ``ref: find_bad_builds.py``
Author
Owner

1 new commit added

  • ref: find-hidden-packages.py
**1 new commit added** * ``ref: find-hidden-packages.py``
Author
Owner

1 new commit added

  • ref: ftbfs_weekly_reminder.py
**1 new commit added** * ``ref: ftbfs_weekly_reminder.py``
Author
Owner

ftbfs_weekly_reminder.py ->
ansible changes: https://pagure.io/fedora-infra/ansible/pull-request/2874

`ftbfs_weekly_reminder.py` -> ansible changes: https://pagure.io/fedora-infra/ansible/pull-request/2874
Author
Owner

1 new commit added

  • ref: get_retired_packages.sh
**1 new commit added** * ``ref: get_retired_packages.sh``
Author
Owner
`get_retired_packages.sh` -> docs change: https://pagure.io/infra-docs-fpo/pull-request/425 ansible crom job change: https://pagure.io/fedora-infra/ansible/pull-request/2880
Author
Owner

1 new commit added

  • ref: isolate-tag.py
**1 new commit added** * ``ref: isolate-tag.py``
Author
Owner

1 new commit added

  • ref: mass_tag.py
**1 new commit added** * ``ref: mass_tag.py``
Author
Owner
`mass_tag.py` -> docs change: https://pagure.io/infra-docs-fpo/pull-request/426
Author
Owner

1 new commit added

  • ref: need_rebuild.py
**1 new commit added** * ``ref: need_rebuild.py``
Author
Owner

need_rebuild.py ->
docs change: https://pagure.io/infra-docs-fpo/pull-request/427

`need_rebuild.py` -> docs change: https://pagure.io/infra-docs-fpo/pull-request/427
Author
Owner

1 new commit added

  • ref: prune-tag.py
**1 new commit added** * ``ref: prune-tag.py``
Author
Owner

1 new commit added

  • ref: secondary-mash-updates
**1 new commit added** * ``ref: secondary-mash-updates``
Author
Owner

1 new commit added

  • ref: secondary-sync.py
**1 new commit added** * ``ref: secondary-sync.py``
Author
Owner

1 new commit added

  • ref: srpm-excluded-arch.py
**1 new commit added** * ``ref: srpm-excluded-arch.py``
Author
Owner

1 new commit added

  • ref: srpm-excluded-arch.py
**1 new commit added** * ``ref: srpm-excluded-arch.py``
Author
Owner

1 new commit added

  • ref: sync-blocked-primary.py
**1 new commit added** * ``ref: sync-blocked-primary.py``
Author
Owner

1 new commit added

  • ref: sync-tagged-primary
**1 new commit added** * ``ref: sync-tagged-primary``
Author
Owner

1 new commit added

  • ref: sig_policy.py
**1 new commit added** * ``ref: sig_policy.py``
Author
Owner

27 new commits added

  • ref: sig_policy.py
  • ref: sync-tagged-primary
  • ref: sync-blocked-primary.py
  • ref: srpm-excluded-arch.py
  • ref: secondary-sync.py
  • ref: secondary-mash-updates
  • ref: prune-tag.py
  • ref: need_rebuild.py
  • ref: mass_tag.py
  • ref: isolate-tag.py
  • ref: get_retired_packages.sh
  • ref: ftbfs_weekly_reminder.py
  • ref: find-hidden-packages.py
  • ref: find_bad_builds.py
  • ref: find_FTBFS.py
  • ref: find_failures.py
  • ref: fedora_torrent_ini.py
  • ref: fedmsg-functions.sh fedora-messaging-logger fedora-messaging.sh
  • ref: distgit-branch-unused.py
  • ref: close-old-ftbfs-bugz.sh
  • ref: clean-amis.py
  • ref: check-latest-build.py check-testing-pre-GA.py check-upgrade-paths.py clean-overrides.py
  • ref: check_epel_deps.py check_epel_deps.sh
  • ref: block_retired.py
  • ref: find_tagged_retired_epel_builds.py removed, also removed link on find_unblocked_orphans.py
  • ref: distgit/ also renaming some of them
  • ref: orhaned-packages-process/
**27 new commits added** * ``ref: sig_policy.py`` * ``ref: sync-tagged-primary`` * ``ref: sync-blocked-primary.py`` * ``ref: srpm-excluded-arch.py`` * ``ref: secondary-sync.py`` * ``ref: secondary-mash-updates`` * ``ref: prune-tag.py`` * ``ref: need_rebuild.py`` * ``ref: mass_tag.py`` * ``ref: isolate-tag.py`` * ``ref: get_retired_packages.sh`` * ``ref: ftbfs_weekly_reminder.py`` * ``ref: find-hidden-packages.py`` * ``ref: find_bad_builds.py`` * ``ref: find_FTBFS.py`` * ``ref: find_failures.py`` * ``ref: fedora_torrent_ini.py`` * ``ref: fedmsg-functions.sh fedora-messaging-logger fedora-messaging.sh`` * ``ref: distgit-branch-unused.py`` * ``ref: close-old-ftbfs-bugz.sh`` * ``ref: clean-amis.py`` * ``ref: check-latest-build.py check-testing-pre-GA.py check-upgrade-paths.py clean-overrides.py`` * ``ref: check_epel_deps.py check_epel_deps.sh`` * ``ref: block_retired.py`` * ``ref: find_tagged_retired_epel_builds.py removed, also removed link on find_unblocked_orphans.py`` * ``ref: distgit/ also renaming some of them`` * ``ref: orhaned-packages-process/``
Author
Owner

don't be affraid I just rebase last two commits into one

don't be affraid I just rebase last two commits into one
Author
Owner

sig_policy.py ->
ansible cron job changes: https://pagure.io/fedora-infra/ansible/pull-request/2885

`sig_policy.py` -> ansible cron job changes: https://pagure.io/fedora-infra/ansible/pull-request/2885
Author
Owner

1 new commit added

  • ref: follow-policy.py
**1 new commit added** * ``ref: follow-policy.py``
Author
Owner

follow-policy ->
also created soft link for it

(.venv) amedvede@fedora:~/Work/releng$ ls -la scripts_new/infrastructure/fesco/follow_policy.py
lrwxrwxrwx. 1 amedvede amedvede 55 Oct  2 13:32 scripts_new/infrastructure/fesco/follow_policy.py -> scripts_new/release-process/bug-filing/follow_policy.py
`follow-policy` -> also created soft link for it ``` (.venv) amedvede@fedora:~/Work/releng$ ls -la scripts_new/infrastructure/fesco/follow_policy.py lrwxrwxrwx. 1 amedvede amedvede 55 Oct 2 13:32 scripts_new/infrastructure/fesco/follow_policy.py -> scripts_new/release-process/bug-filing/follow_policy.py ```
Author
Owner

Now there just a single PR with doc modifications: https://pagure.io/infra-docs-fpo/pull-request/428

Now there just a single PR with doc modifications: https://pagure.io/infra-docs-fpo/pull-request/428
Author
Owner

Single Ansible PR that contains Ansible changes: https://pagure.io/fedora-infra/ansible/pull-request/2888

Single Ansible PR that contains Ansible changes: https://pagure.io/fedora-infra/ansible/pull-request/2888
Author
Owner

1 new commit added

  • ref: critpath.py
**1 new commit added** * ``ref: critpath.py``
Author
Owner

single pungi-fedora PR with changes: https://pagure.io/pungi-fedora/pull-request/1546

single pungi-fedora PR with changes: https://pagure.io/pungi-fedora/pull-request/1546
Author
Owner

1 new commit added

  • ref: follow_policy.py
**1 new commit added** * ``ref: follow_policy.py``
Author
Owner

1 new commit added

  • ref: indroduction of post-release dir under release-process dir and moving failed_composes_cleanup.py there
**1 new commit added** * ``ref: indroduction of post-release dir under release-process dir and moving failed_composes_cleanup.py there``
Author
Owner

1 new commit added

  • ref: ga-cleanup rebuild
**1 new commit added** * ``ref: ga-cleanup rebuild``
Author
Owner

rebased onto 896d5284b9

rebased onto 896d5284b9621de4e6b4ee334cf10da07cc104d8
Member
https://pagure.io/fork/amedvede/releng/blob/5f72afcadc63d6b5bd049a1d8e8fa8aa7d0f5f77/f/scripts_new/packages/orphaned/orphaned-packages-process/entrypoint.sh#_10 needs to be updated to point to the new path.
Member

Also, orphaned-packages-process probably doesn't need to be its own subdirectory. Putting everything in packages/orphaned makes more sense IMO.

Also, orphaned-packages-process probably doesn't need to be its own subdirectory. Putting everything in `packages/orphaned` makes more sense IMO.
Author
Owner

Also, orphaned-packages-process probably doesn't need to be its own subdirectory. Putting everything in packages/orphaned makes more sense IMO.

will bring it up for discussion

> Also, orphaned-packages-process probably doesn't need to be its own subdirectory. Putting everything in `packages/orphaned` makes more sense IMO. will bring it up for discussion
Author
Owner

1 new commit added

  • ref: orpaned-packages-process scripts moved 1 dir above
**1 new commit added** * ``ref: orpaned-packages-process scripts moved 1 dir above``
Author
Owner

pagure.io/fork/amedvede/releng@5f72afcadc/f/scripts_new/packages/orphaned/orphaned-packages-process/entrypoint.sh (_10) needs to be updated to point to the new path.

fixed, thanks for pointing me out!

> https://pagure.io/fork/amedvede/releng/blob/5f72afcadc63d6b5bd049a1d8e8fa8aa7d0f5f77/f/scripts_new/packages/orphaned/orphaned-packages-process/entrypoint.sh#_10 needs to be updated to point to the new path. fixed, thanks for pointing me out!
Author
Owner
Fesco doc PR: https://pagure.io/fesco/fesco-docs/pull-request/112
Author
Owner

rebased onto 669640c91a

rebased onto 669640c91a9462a2eb91e21c62f8ab3551dcca26
Author
Owner

1 new commit added

  • ref: follow_policy.py
**1 new commit added** * ``ref: follow_policy.py``
Author
Owner

1 new commit added

  • ref: import paths update for mass_rebuild_file_bugs.py
**1 new commit added** * ``ref: import paths update for mass_rebuild_file_bugs.py``
Author
Owner

Pull-Request has been merged by amedvede

Pull-Request has been merged by amedvede
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
releng/tickets!12978
No description provided.