[CI] If one test failed cancle all #4275

New Issue

GiteaMirror · 2025-11-02T05:44:31-06:00

GiteaMirror commented

2025-11-02 05:44:31 -06:00

Originally created by @6543 on GitHub (Nov 9, 2019).

I often see CI tasks wich already failed but still need recources

Originally created by @6543 on GitHub (Nov 9, 2019). I often see CI tasks wich already failed but still need recources

GiteaMirror closed this issue

2025-11-02 05:44:32 -06:00

GiteaMirror commented

2025-11-02 05:44:32 -06:00

@mrsdizzie commented on GitHub (Nov 9, 2019):

It should show all failures imo -- if there are 10 failures it doesn't make sense to stop at the first one, fix it, run again, see the 2nd one, fix it, etc...

A majority of the failed builds/resources are by people not running even unit tests locally first as recommended

@mrsdizzie commented on GitHub (Nov 9, 2019): It should show all failures imo -- if there are 10 failures it doesn't make sense to stop at the first one, fix it, run again, see the 2nd one, fix it, etc... A majority of the failed builds/resources are by people not running even unit tests locally first as recommended

GiteaMirror commented

2025-11-02 05:44:32 -06:00

@6543 commented on GitHub (Nov 9, 2019):

@mrsdizzie thought about this point ... problem is make test fail multible times - on local because of missing test settings/requirements ...

...ces/mirror/mirror.go:200:runSync() [E] Failed to synchronize tags to releases for repository: GetTagCommitID: exit status 128 - fatal:
models/repo.go:548:mustOwnerName() [E] Error loading repository owner name: user does not exist [uid: 0, name: , keyid: 0]
...s/repo_permission.go:154:func1 -> expected: []int64{} actual : []int64(nil)

.... but I'm looking forward to drone.gitea.com wich more people can cancle manualy ...

@6543 commented on GitHub (Nov 9, 2019): @mrsdizzie thought about this point ... problem is `make test` fail multible times - on local because of missing test settings/requirements ... * `...ces/mirror/mirror.go:200:runSync() [E] Failed to synchronize tags to releases for repository: GetTagCommitID: exit status 128 - fatal:` * `models/repo.go:548:mustOwnerName() [E] Error loading repository owner name: user does not exist [uid: 0, name: , keyid: 0] ` * `...s/repo_permission.go:154:func1` -> `expected: []int64{} actual : []int64(nil) ` .... but I'm looking forward to drone.gitea.com wich more people can cancle manualy ...

GiteaMirror commented

2025-11-02 05:44:33 -06:00

@zeripath commented on GitHub (Nov 9, 2019):

The issue with make test is that it runs with tests logging at trace level .

We need to resurrect the use gitea.com log module and then we can initialise the logger with Json

@zeripath commented on GitHub (Nov 9, 2019): The issue with make test is that it runs with tests logging at trace level . We need to resurrect the use gitea.com log module and then we can initialise the logger with Json

GiteaMirror referenced this issue

2025-11-02 12:32:20 -06:00

[PR #4275] [MERGED] Change parsing of postgresql settings #17312

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/gitea#4275