TestAPIRepoMigrate fails when GitHub is down #10814

Closed
opened 2025-11-02 09:18:52 -06:00 by GiteaMirror · 7 comments
Owner

Originally created by @silverwind on GitHub (May 9, 2023).

https://github.com/go-gitea/gitea/actions/runs/4927752928/jobs/8805178261

    api_repo_test.go:360: 
        	Error Trace:	/home/runner/work/gitea/gitea/tests/integration/api_repo_test.go:360
        	Error:      	unexpected error '%v' on url '%s'
        	Test:       	TestAPIRepoMigrate
        	Messages:   	Migration failed: Clone: exit status 128 - fatal: unable to access 'https://github.com/go-gitea/test_repo.git/': The requested URL returned error: 503
        	            	.%!(EXTRA string=https://github.com/go-gitea/test_repo.git)

Originally created by @silverwind on GitHub (May 9, 2023). https://github.com/go-gitea/gitea/actions/runs/4927752928/jobs/8805178261 ``` api_repo_test.go:360: Error Trace: /home/runner/work/gitea/gitea/tests/integration/api_repo_test.go:360 Error: unexpected error '%v' on url '%s' Test: TestAPIRepoMigrate Messages: Migration failed: Clone: exit status 128 - fatal: unable to access 'https://github.com/go-gitea/test_repo.git/': The requested URL returned error: 503 .%!(EXTRA string=https://github.com/go-gitea/test_repo.git) ```
GiteaMirror added the type/testing label 2025-11-02 09:18:52 -06:00
Author
Owner

@wxiaoguang commented on GitHub (May 9, 2023):

At that time, GitHub is down ..........

@wxiaoguang commented on GitHub (May 9, 2023): At that time, GitHub is down ..........
Author
Owner

@silverwind commented on GitHub (May 9, 2023):

I think we should not have external dependencies in our tests so they work offline too. Otherwise, adding some failed test retries in gotestsum may help.

@silverwind commented on GitHub (May 9, 2023): I think we should not have external dependencies in our tests so they work offline too. Otherwise, adding some failed test retries in [gotestsum](https://github.com/go-gitea/gitea/pull/24528) may help.
Author
Owner

@wxiaoguang commented on GitHub (May 9, 2023):

Theoretically speaking, yes. But actually, that's the easiest way to test the GitHub migration.

@wxiaoguang commented on GitHub (May 9, 2023): Theoretically speaking, yes. But actually, that's the easiest way to test the GitHub migration.
Author
Owner

@silverwind commented on GitHub (May 9, 2023):

Right, so I guess to realistically test GH, it has to be GH. Might just go with the retry variant.

@silverwind commented on GitHub (May 9, 2023): Right, so I guess to realistically test GH, it has to be GH. Might just go with the retry variant.
Author
Owner

@wxiaoguang commented on GitHub (May 9, 2023):

But at that time, GitHub was outage for one hour ....

@wxiaoguang commented on GitHub (May 9, 2023): But at that time, GitHub was outage for one hour ....
Author
Owner

@silverwind commented on GitHub (May 9, 2023):

I guess we could make the test skip if it receives a status code in the 5xx range. Not safe from a complete outage, but I guess better than currently.

@silverwind commented on GitHub (May 9, 2023): I guess we could make the test skip if it receives a status code in the 5xx range. Not safe from a complete outage, but I guess better than currently.
Author
Owner

@silverwind commented on GitHub (May 9, 2023):

Closing for now as it is rather unactionable.

@silverwind commented on GitHub (May 9, 2023): Closing for now as it is rather unactionable.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#10814