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

Weird behavior when migrating from Github to gitea.com #11705

Closed
1 of 3 tasks
j-be opened this issue May 31, 2020 · 4 comments · Fixed by #13127
Closed
1 of 3 tasks

Weird behavior when migrating from Github to gitea.com #11705

j-be opened this issue May 31, 2020 · 4 comments · Fixed by #13127

Comments

@j-be
Copy link

j-be commented May 31, 2020

  • Gitea version (or commit ref): 1.13.0+dev-30-gb02d2c377
  • Git version: whatever gitea.com runs on
  • Operating system: whatever gitea.com runs on
  • Database (use [x]): whatever gitea.com runs on
  • Can you reproduce the bug at https://try.gitea.io:
    • Yes (provide example URL)
    • No
    • Not relevant

Description

While working on #8380 I migrated https://github.com/go-gitea/test_repo to https://gitea.com/j-be/test_repo/ and took that for unit tests. I am now a bit confused on how some stuff is handled when migrating:

  • Reactions:

    • lunny is taken over as UserID: 1, UserName: "lunny", as that user exists on Gitea too
    • mrsdizzie as user in Reactions is migrated as UserID: -1, UserName: "mrsdizzie(github)", which looks really good in the UI, but mrsdizzie would exist on Gitea as well - I guess OIDC is involved here, but how does it know? Do I need to take into account something here as well?
  • Review - here is where it gets really confusing:

    • lunny and jolheiser are both taken over as ReviewerID: 0, even though the accounts would exist on gitea.com
    • Reviews from both zeripath and lafriks are suddenly mine (ReviewerID: 5610, ReviewerName: "j-be"), again, even though these users exist on gitea.com
  • Release, PullRequest, and Comments thereof are all PublisherID: 5610, PublisherName: "j-be" and PosterID: 5610, PosterName: "j-be", even though they should still be owned by their initial authors.

@lunny
Copy link
Member

lunny commented Jun 1, 2020

Because lunny on gitea.com (that's me :)) has binding github login, so we can replace the external account as an internal account, but mrsdizzie hasn't binding github login, so It displayed as external account.

@lunny
Copy link
Member

lunny commented Jun 1, 2020

图片

It should be some bug.

@lunny lunny added the type/bug label Jun 1, 2020
a1012112796 added a commit to a1012112796/gitea that referenced this issue Jun 1, 2020
As title, maybe fix one of issue in go-gitea#11705

Signed-off-by: a1012112796 <[email protected]>
a1012112796 added a commit to a1012112796/gitea that referenced this issue Jun 1, 2020
fix one part of go-gitea#11705

Signed-off-by: a1012112796 <[email protected]>
techknowlogick added a commit that referenced this issue Jun 3, 2020
As title, maybe fix one of issue in #11705

Signed-off-by: a1012112796 <[email protected]>

Co-authored-by: techknowlogick <[email protected]>
j-be added a commit to j-be/gitea that referenced this issue Jun 6, 2020
Telling from the response on go-gitea#11705 this is expected.

Signed-off-by: Juri Berlanda <[email protected]>
j-be added a commit to j-be/gitea that referenced this issue Jun 6, 2020
Telling from the response on go-gitea#11705 this is expected.

Signed-off-by: Juri Berlanda <[email protected]>
@stale
Copy link

stale bot commented Jul 31, 2020

This issue has been automatically marked as stale because it has not had recent activity. I am here to help clear issues left open even if solved or waiting for more insight. This issue will be closed if no further activity occurs during the next 2 weeks. If the issue is still valid just add a comment to keep it alive. Thank you for your contributions.

@stale stale bot added the issue/stale label Jul 31, 2020
ydelafollye pushed a commit to ydelafollye/gitea that referenced this issue Jul 31, 2020
…#11721)

As title, maybe fix one of issue in go-gitea#11705

Signed-off-by: a1012112796 <[email protected]>

Co-authored-by: techknowlogick <[email protected]>
@stale
Copy link

stale bot commented Aug 16, 2020

This issue has been automatically closed because of inactivity. You can re-open it if needed.

@stale stale bot closed this as completed Aug 16, 2020
a1012112796 added a commit to a1012112796/gitea that referenced this issue Oct 13, 2020
follow go-gitea#12039, show original author's reviews by other way.
fix go-gitea#11705.

Signed-off-by: a1012112796 <[email protected]>
zeripath pushed a commit that referenced this issue Oct 14, 2020
follow #12039, show original author's reviews by other way.
fix #11705.

Signed-off-by: a1012112796 <[email protected]>
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants