Skip to content

Comments

feat: Add support for tag migration#212

Merged
gridhead merged 2 commits intofedora-infra:mainfrom
sdglitched:feat/issue-187
Jul 15, 2025
Merged

feat: Add support for tag migration#212
gridhead merged 2 commits intofedora-infra:mainfrom
sdglitched:feat/issue-187

Conversation

@sdglitched
Copy link
Contributor

@sdglitched sdglitched commented Jul 5, 2025

Hi,

This PR introduces support for migration of tags from source repository to destination repository.


Migration of tags uses similar approach which was uitilised for migrating the branches along with tests for checking the functionality.

As the tests uses repository gitlab.com/gridhead/protop2g-test/ as destination for migrating the the branches, same is used for migrating the tags.

Tags form centos/rpms/systemd and fedoraproject/rpms/systemd are migrated to gitlab/gridhead/protop2g-test.

Note: As the source repository pagure/protop2g-test-srce didn't have any available tags in it, complete testing of migrating tags in modules test_repo_fedo and test_unit_repo were not possible.

@gridhead, will it be possible to create a couple of releases in pagure/protop2g-test-srce? This would help in testing the migration of tags from pagure.io. Having the source changed and changing all the test seems unfeasible.


The tests are passing locally (ref. the below snapshot).

image

But the same tests seems to be failing in the GitHub's CI.

The failures of this CI run seems very similar to the failure of CI run for PR which we believed was causing by the unavailability of the pagure services due to AI scraper attacks.

Not sure what the root cause might be now, as the pagure services seems to be working.


Thank you for reviewing the PR at your convenience.

Fixes: #187

@sdglitched sdglitched force-pushed the feat/issue-187 branch 3 times, most recently from 4b4fa4e to c971366 Compare July 6, 2025 17:10
@sdglitched sdglitched marked this pull request as draft July 7, 2025 00:52
@sdglitched sdglitched marked this pull request as ready for review July 7, 2025 03:00
 Fetch tags from source repo and migrate all/specified tags to the destination repo

Signed-off-by: Shounak Dey <shounakdey@ymail.com>
 Test the tag migration functionality

Signed-off-by: Shounak Dey <shounakdey@ymail.com>
@gridhead gridhead self-requested a review July 14, 2025 03:38
@gridhead gridhead self-assigned this Jul 14, 2025
Copy link
Member

@gridhead gridhead left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for working on modularizing the parts of the codebase on the top of the requested feature. I will now merge this because I suspect that the secrets are not made available to the contributions that come from outside the origin.

@gridhead gridhead merged commit e5bba01 into fedora-infra:main Jul 15, 2025
6 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Investigate migration of tags while migrating repositories

2 participants