WIP: Add support to migrate from gogs

[lunny]

As title

[codecov-io]

Codecov Report

Merging #7181 into master will decrease coverage by 0.12%.
The diff coverage is 2.18%.

@@            Coverage Diff             @@
##           master    #7181      +/-   ##
==========================================
- Coverage   41.25%   41.13%   -0.13%     
==========================================
  Files         542      543       +1     
  Lines       69708    69879     +171     
==========================================
- Hits        28758    28744      -14     
- Misses      37269    37454     +185     
  Partials     3681     3681

Impacted Files	Coverage Δ
modules/migrations/migrate.go	21.13% <0%> (-1.78%)	⬇️
modules/migrations/gogs.go	2.56% <2.56%> (ø)
modules/sync/unique_queue.go	78.57% <0%> (-10.72%)	⬇️
modules/indexer/indexer.go	44.73% <0%> (-10.53%)	⬇️
models/unit.go	62.16% <0%> (-5.41%)	⬇️
models/repo_indexer.go	66.54% <0%> (-3.64%)	⬇️
routers/repo/view.go	37.71% <0%> (-0.88%)	⬇️
models/gpg_key.go	55.59% <0%> (+0.55%)	⬆️
modules/log/event.go	65.64% <0%> (+1.02%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fe7a6d9...9fe0b4a. Read the comment docs.

[zeripath]

Can we do this without having to add gogs-client to every install of gitea?

[lunny]

If we want to support migration from gogs, I think it's difficult except we rewrite a sdk for gogs.

[mrsdizzie]

What happens if try.gogs.io is down, or your repo gets remove...will all builds then fail? I know we already rely on github and other sites being available for building but maybe for these type of migration test it can skip them if the URL doesn't respond properly right away. Even now try.gogs.io is very slow for me and times out on some pages.

[silverwind]

Agree, Internet-dependent tests should ideally not run during regular test runs, but only when explicitely triggered e.g. make test-internet or similar. Could maybe add to release tasks to run those.

[adelowo]

Fix import

[mrsdizzie]

@lunny I think we should still have the test, it just shouldn't fail if try.gogs.io is down which would break Gitea CI. Maybe we can just try to ping/reach that site before running the test and skip/pass the test if it isn't available.

A best way would be to caputre timeout in the actual migration code and if the error is a timeout error not fail the test.

Another simple way is to have a check in the test file:

resp, err := http.Get("https://try.gogs.io/lunnytest/TESTREPO")
if err != nil {
   // skip and don't run test
} else {
    // check if response is 200 level
    // ok to run test
}

That should make sure that try.gogs.io is up and that your rest repo is still there.

Also now we can use original_author original_author_id and original_url when migrating these.

[lunny]

@mrsdizzie OK. I will add test back.

[stale]

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 months. Thank you for your contributions.

[6543]

pleace resolve confilcts

[Cat7373]

Why has this feature not been merged yet?

It has been waiting for more than a year.

[6543]

@Cat7373 the simple answer is nobody had time to finaly finish it

[lunny]

replaced by #14342

[Cat7373]

get excited.