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

Missing mirror URL on project dashboard #28203

Open
somera opened this issue Nov 24, 2023 · 23 comments
Open

Missing mirror URL on project dashboard #28203

somera opened this issue Nov 24, 2023 · 23 comments
Labels

Comments

@somera
Copy link

somera commented Nov 24, 2023

Description

Since I upgraded to 1.21.0 I see more and more mirrors with no Mirror URL

image

The sync for this mirror is working.

In PostgreSQL I see the URL

image

I checked ~200 projects. In 24 the mirror url is not visible. some days ago I deleted one mirror and added it again and I saw the mirror url. But today I can't see it in project dashboard.

Gitea Version

1.21.0

Can you reproduce the bug on the Gitea demo site?

No

Log Gist

No response

Screenshots

No response

Git Version

2.43.0

Operating System

Ubuntu 20.04.x

How are you running Gitea?

gitea-1.21.0-linux-amd64

Database

PostgreSQL

@somera somera changed the title Missing mirror URL Missing mirror URL on project dashboard Nov 24, 2023
@lunny
Copy link
Member

lunny commented Nov 25, 2023

Could you find any error logs?

@somera
Copy link
Author

somera commented Nov 25, 2023

@lunny. No.

In browser I see this

image

@lunny
Copy link
Member

lunny commented Nov 25, 2023

I think it maybe a translation problem. Could you try English?

@somera
Copy link
Author

somera commented Nov 25, 2023

As I wrote. No -> I can't see any errors in the gitea logs.

The screenshot which I posted was just from the browser. This was probably confusing.

@lunny
Copy link
Member

lunny commented Nov 25, 2023

Yes. To confirm my guess. Could you try to switch English from the footer right corner and take a look again?

@somera
Copy link
Author

somera commented Nov 25, 2023

I try this later. I'm upgrading now to PostgreSQL 16 and Ubuntu 22.04.

@somera
Copy link
Author

somera commented Nov 25, 2023

I did it. Same problem
image

@somera
Copy link
Author

somera commented Nov 25, 2023

I found at the moment 44 projects with this issue. But I have no time to check all mirrors.

@somera
Copy link
Author

somera commented Nov 26, 2023

~80 mirrors now affected.

@somera
Copy link
Author

somera commented Nov 27, 2023

@lunny no changes here with 1.21.1

@somera somera closed this as completed Nov 27, 2023
@somera somera reopened this Nov 27, 2023
@somera
Copy link
Author

somera commented Nov 27, 2023

@lunny when I do this

[Mo Nov 27][19:23:44][xxx]@[xxx]:[~]$ curl http://xxx/git/api/v1/repos/skeeto/Spacefront | jq | grep original
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  2111    0  2111    0     0   223k      0 --:--:-- --:--:-- --:--:--  229k
  "original_url": "https://github.com/skeeto/Spacefront.git",

than I see the original_url. Looks good.

I restarted my memcached. But there was no change.

image

@somera
Copy link
Author

somera commented Nov 28, 2023

@lunny any updates here? The problem makes it a little difficult for me to work with Gitea.

@somera
Copy link
Author

somera commented Nov 29, 2023

@lunny I found the problem. ~17% mirrors are affected on my gitea instance. The problem is, that in the mirrors table the remote_address column is empty. I can fix it by setting the column. But there is an bug, cause I migrated new mirrors and some days later the url was empty.

The question is: why is this empty?

@somera
Copy link
Author

somera commented Dec 2, 2023

After my fix in the DB three days ago nothing changes. It's looks good.

@lunny if you think there is nothig to fix, than this issue can be closed.

@yp05327
Copy link
Contributor

yp05327 commented Dec 5, 2023

Maybe related to #26952

@lunny
Copy link
Member

lunny commented Dec 5, 2023

After my fix in the DB three days ago nothing changes. It's looks good.

@lunny if you think there is nothig to fix, than this issue can be closed.

Let's leave the issue open since it's possible a bug. And how did you do the workaround?

@yp05327
Copy link
Contributor

yp05327 commented Dec 5, 2023

It seems that this issue is caused by the migration, as remote_address column is empty but original_url is not.

@somera
Copy link
Author

somera commented Dec 5, 2023

Let's leave the issue open since it's possible a bug. And how did you do the workaround?

I set remote_address to the same value like original_url. Before I did this, I compared all te mirrors where remote_address was set.

@lunny
Copy link
Member

lunny commented Mar 4, 2024

Let's leave the issue open since it's possible a bug. And how did you do the workaround?

I set remote_address to the same value like original_url. Before I did this, I compared all te mirrors where remote_address was set.

After you fix it manually, does any mirrors remote_address become empty?

@wxiaoguang
Copy link
Contributor

There are not only one issue report for the RemoteAddress problem (eg: #29132)

I think " Add RemoteAddress to mirrors #26952 " is not robust enough.

@somera
Copy link
Author

somera commented Mar 4, 2024

After you fix it manually, does any mirrors remote_address become empty?

No. It works fine.

@lunny
Copy link
Member

lunny commented Mar 4, 2024

There are not only one issue report for the RemoteAddress problem (eg: #29132)

I think " Add RemoteAddress to mirrors #26952 " is not robust enough.

Yes, but maybe we can close this one or keep this open for some future refactor PRs about that?

@wxiaoguang
Copy link
Contributor

Yes, but maybe we can close this one or keep this open for some future refactor PRs about that?

But I just hope there could be a PR fixing the robust problem. It is critical and stops users from upgrading.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants