Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sync issues, PRs, labels, milestones, comments from GitHub mirrors #20311

Open
wants to merge 136 commits into
base: main
Choose a base branch
from

Conversation

harryzcy
Copy link
Contributor

@harryzcy harryzcy commented Jul 10, 2022

This PR adds support for one-way syncing of topics, milestones, labels, issues, pull requests, comments, reviews, releases, and reactions from pull mirror (GitHub) to Gitea.
Closes #18369.

Downloader interface is extended with GetNewIssues, GetNewPullRequests, since new issues and pull requests should be appended/updated to the existing repository.

Uploader interface is extended with UpdateTopics, UpdateMilestones, UpdateLabels, PatchReleases, PatchComments, PatchIssues, and PatchPullRequests. All Update* should replace existing data and Patch* should append/update but should not delete existing data.

The above methods will be implement here for GitHub, or possibly also GitLab, in this pull request.

In order to make the updates work, we would need to uniquely identify a Topic, Milestone, Label, Release, Comments, Issue and PullRequest. The following list shows how they are identified:

  • Topic - Name (it's just a string, unrelated to any other things)
  • Milestone - OriginalID*
  • Label - OriginalID*
  • Release - OriginalID*
  • Comments - OriginalID*
  • Issue - IssueID and Index
  • PullRequest - IssueID and Index
  • Reviews - OriginalID*
  • Reactions - IssueID, CommentID and Type

OriginalID*: additional column added to database, used to track ID from the source

Screenshot of toggles to set which migration Items should be synced (the only change is now when mirror option is checked, the migration items are still available for supported platforms):

image

@lunny
Copy link
Member

lunny commented Jul 11, 2022

You need to implement UpdateIssues, UpdateReactions, UpdateComments and etc. functions. You need to find the unique external id to update them.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jul 11, 2022
@codecov-commenter
Copy link

codecov-commenter commented Jul 11, 2022

Codecov Report

Merging #20311 (dd20d7d) into main (f521e88) will decrease coverage by 0.43%.
The diff coverage is 20.20%.

❗ Current head dd20d7d differs from pull request most recent head 8fb9956. Consider uploading reports for the commit 8fb9956 to get more accurate results

@@            Coverage Diff             @@
##             main   #20311      +/-   ##
==========================================
- Coverage   47.14%   46.71%   -0.43%     
==========================================
  Files        1149      994     -155     
  Lines      151446   137597   -13849     
==========================================
- Hits        71397    64285    -7112     
+ Misses      71611    65414    -6197     
+ Partials     8438     7898     -540     
Impacted Files Coverage Δ
models/repo/mirror.go 66.07% <ø> (ø)
modules/migration/null_downloader.go 23.40% <0.00%> (-8.03%) ⬇️
modules/migration/null_uploader.go 0.00% <0.00%> (ø)
modules/notification/notification.go 93.02% <ø> (+10.01%) ⬆️
routers/api/v1/repo/migrate.go 53.43% <ø> (+2.24%) ⬆️
routers/web/repo/migrate.go 40.72% <ø> (+1.91%) ⬆️
services/migrations/dump.go 39.83% <ø> (+0.55%) ⬆️
services/migrations/migrate.go 30.01% <0.00%> (-18.85%) ⬇️
models/migrate.go 17.34% <4.26%> (-28.64%) ⬇️
models/issues/review.go 48.31% <15.00%> (-4.42%) ⬇️
... and 7 more

... and 1146 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

models/migrate.go Outdated Show resolved Hide resolved
models/issues/issue.go Outdated Show resolved Hide resolved
@delvh delvh dismissed their stale review February 24, 2024 10:44

Too much changes since my last review, have to re-review all of them carefully

@GiteaBot GiteaBot added lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. and removed lgtm/blocked A maintainer has reservations with the PR and thus it cannot be merged labels Feb 24, 2024
@delvh delvh changed the title Sync issues/PRs etc from pull mirror Sync issues, PRs, labels, milestones, comments from GitHub mirrors Feb 24, 2024
@lunny lunny modified the milestones: 1.22.0, 1.23.0 Mar 4, 2024
@rcarmo
Copy link

rcarmo commented Mar 8, 2024

Does this include mirroring release packages as well? Right now Gitea only mirrors the tarballs, and any artifacts (like .appimages) added to GitHub releases are lost.

@harryzcy
Copy link
Contributor Author

harryzcy commented Mar 8, 2024

Does this include mirroring release packages as well? Right now Gitea only mirrors the tarballs, and any artifacts (like .appimages) added to GitHub releases are lost.

Yes, included in this PR

@github-actions github-actions bot added modifies/go Pull requests that update Go code modifies/templates This PR modifies the template files modifies/js labels Apr 26, 2024
@siddarthkay
Copy link

This PR looks cool, I'll try a build from this branch locally to see if this fixes our needs for having a mirror set up for Github repositories that also syncs, issues, comments on issues and PRs.
Thanks @harryzcy
Looking forward to this PR getting vetted and merged in the future.

@harryzcy
Copy link
Contributor Author

@siddarthkay Thank you for helping testing this out.

@siddarthkay
Copy link

hmm @harryzcy : I used your branch url as the source of my docker image and now I see that the options to select Issues, PRs while setting up a mirror are disabled.

Is it intended to be that way?

image

Here is my docker-compose.yml Incase it helps to replicate the issue.

version: "3"

networks:
  gitea:
    external: false

services:
  server:
    build:
      context: https://github.com/harryzcy/gitea.git#sync-issue-pr-and-more
      dockerfile: Dockerfile
    container_name: gitea
    environment:
      - USER_UID=1000
      - USER_GID=1000
      - GITEA__database__DB_TYPE=mysql
      - GITEA__database__HOST=db:3306
      - GITEA__database__NAME=gitea
      - GITEA__database__USER=gitea
      - GITEA__database__PASSWD=gitea
    restart: always
    networks:
      - gitea
    volumes:
      - ./gitea:/data
      - /etc/timezone:/etc/timezone:ro
      - /etc/localtime:/etc/localtime:ro
    ports:
      - "3000:3000"
      - "222:22"
    depends_on:
      - db

  db:
    image: mysql:8
    restart: always
    environment:
      - MYSQL_ROOT_PASSWORD=gitea
      - MYSQL_USER=gitea
      - MYSQL_PASSWORD=gitea
      - MYSQL_DATABASE=gitea
    networks:
      - gitea
    volumes:
      - ./mysql:/var/lib/mysql

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. modifies/api This PR adds API routes or modifies them modifies/cli PR changes something on the CLI, i.e. gitea doctor or gitea admin modifies/go Pull requests that update Go code modifies/js modifies/templates This PR modifies the template files size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. topic/ui Change the appearance of the Gitea UI type/feature Completely new functionality. Can only be merged if feature freeze is not active.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sync/Mirror Issues and Pull-Requests on migrated repository