mirror of
https://codeberg.org/forgejo/forgejo.git
synced 2026-07-25 10:57:37 +00:00
fa5a2501d0
If a Forgejo Actions job was run more than once, Forgejo would not display previous attempts if no `ActionTask` existed for the latest attempt. That is the case when a job is cancelled or skipped before having been dispatched to a runner or while it is waiting for a runner. This is fixed by always loading all existing attempts. Resolves #12626. ## Checklist The [contributor guide](https://forgejo.org/docs/next/contributor/) contains information that will be helpful to first time contributors. All work and communication must conform to Forgejo's [AI Agreement](https://codeberg.org/forgejo/governance/src/branch/main/AIAgreement.md). There also are a few [conditions for merging Pull Requests in Forgejo repositories](https://codeberg.org/forgejo/governance/src/branch/main/PullRequestsAgreement.md). You are also welcome to join the [Forgejo development chatroom](https://matrix.to/#/#forgejo-development:matrix.org). ### Tests for Go changes - I added test coverage for Go changes... - [x] in their respective `*_test.go` for unit tests. - [x] in the `tests/integration` directory if it involves interactions with a live Forgejo server. - I ran... - [x] `make pr-go` before pushing ### Documentation - [ ] I created a pull request [to the documentation](https://codeberg.org/forgejo/docs) to explain to Forgejo users how to use this change. - [x] I did not document these changes and I do not expect someone else to do it. ### Release notes - [x] This change will be noticed by a Forgejo user or admin (feature, bug fix, performance, etc.). I suggest to include a release note for this change. - [ ] This change is not visible to a Forgejo user or admin (refactor, dependency upgrade, etc.). I think there is no need to add a release note for this change. *The decision if the pull request will be shown in the release notes is up to the mergers / release team.* The content of the `release-notes/<pull request number>.md` file will serve as the basis for the release notes. If the file does not exist, the title of the pull request will be used instead. Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/12779 Reviewed-by: Mathieu Fenniak <mfenniak@noreply.codeberg.org> Reviewed-by: 0ko <0ko@noreply.codeberg.org>
222 lines
4.1 KiB
YAML
222 lines
4.1 KiB
YAML
-
|
|
id: 192
|
|
run_id: 791
|
|
repo_id: 4
|
|
owner_id: 1
|
|
commit_sha: c2d72f548424103f01ee1dc02889c1e2bff816b0
|
|
is_fork_pull_request: false
|
|
name: job_2
|
|
attempt: 3
|
|
job_id: job_2
|
|
task_id: 47
|
|
status: 1
|
|
started: 1683636528
|
|
stopped: 1683636626
|
|
-
|
|
id: 193
|
|
run_id: 792
|
|
repo_id: 4
|
|
owner_id: 1
|
|
commit_sha: c2d72f548424103f01ee1dc02889c1e2bff816b0
|
|
is_fork_pull_request: false
|
|
name: job_2
|
|
attempt: 1
|
|
job_id: job_2
|
|
task_id: 48
|
|
status: 1
|
|
started: 1683636528
|
|
stopped: 1683636626
|
|
-
|
|
id: 194
|
|
run_id: 793
|
|
repo_id: 4
|
|
owner_id: 1
|
|
commit_sha: c2d72f548424103f01ee1dc02889c1e2bff816b0
|
|
is_fork_pull_request: false
|
|
name: job1 (1)
|
|
attempt: 1
|
|
job_id: job1
|
|
task_id: 49
|
|
status: 1
|
|
started: 1683636528
|
|
stopped: 1683636626
|
|
-
|
|
id: 195
|
|
run_id: 793
|
|
repo_id: 4
|
|
owner_id: 1
|
|
commit_sha: c2d72f548424103f01ee1dc02889c1e2bff816b0
|
|
is_fork_pull_request: false
|
|
name: job1 (2)
|
|
attempt: 1
|
|
job_id: job1
|
|
task_id: 50
|
|
status: 1
|
|
started: 1683636528
|
|
stopped: 1683636626
|
|
-
|
|
id: 196
|
|
run_id: 793
|
|
repo_id: 4
|
|
owner_id: 1
|
|
commit_sha: c2d72f548424103f01ee1dc02889c1e2bff816b0
|
|
is_fork_pull_request: false
|
|
name: job2
|
|
attempt: 1
|
|
job_id: job2
|
|
needs: [job1]
|
|
task_id: 51
|
|
status: 5
|
|
started: 1683636528
|
|
stopped: 1683636626
|
|
-
|
|
id: 197
|
|
run_id: 895
|
|
repo_id: 4
|
|
owner_id: 1
|
|
commit_sha: c2d72f548424103f01ee1dc02889c1e2bff816b0
|
|
is_fork_pull_request: false
|
|
name: job1 (1)
|
|
attempt: 0
|
|
job_id: job1
|
|
task_id: 54
|
|
status: 2 # failure
|
|
runs_on: '["postmarketOS"]'
|
|
started: 1683636528
|
|
stopped: 1683636626
|
|
-
|
|
id: 198
|
|
run_id: 895
|
|
repo_id: 4
|
|
owner_id: 1
|
|
commit_sha: c2d72f548424103f01ee1dc02889c1e2bff816b0
|
|
is_fork_pull_request: false
|
|
name: job1 (2)
|
|
attempt: 0
|
|
job_id: job1
|
|
task_id: 55
|
|
status: 6 # running
|
|
runs_on: '["postmarketOS"]'
|
|
started: 1683636528
|
|
stopped: 1683636626
|
|
-
|
|
id: 199
|
|
run_id: 896
|
|
repo_id: 4
|
|
owner_id: 1
|
|
commit_sha: c2d72f548424103f01ee1dc02889c1e2bff816b0
|
|
is_fork_pull_request: false
|
|
name: job1 (1)
|
|
attempt: 0
|
|
job_id: job1
|
|
task_id: 56
|
|
status: 2 # failure
|
|
runs_on: '["postmarketOS"]'
|
|
started: 1683636528
|
|
stopped: 1683636626
|
|
-
|
|
id: 200
|
|
run_id: 896
|
|
repo_id: 4
|
|
owner_id: 1
|
|
commit_sha: c2d72f548424103f01ee1dc02889c1e2bff816b0
|
|
is_fork_pull_request: false
|
|
name: job1 (2)
|
|
attempt: 0
|
|
job_id: job1
|
|
task_id: 57
|
|
status: 1 # success
|
|
runs_on: '["postmarketOS"]'
|
|
started: 1683636528
|
|
stopped: 1683636626
|
|
-
|
|
id: 292
|
|
run_id: 891
|
|
repo_id: 1
|
|
owner_id: 1
|
|
commit_sha: 985f0301dba5e7b34be866819cd15ad3d8f508ee
|
|
is_fork_pull_request: false
|
|
name: job_2
|
|
attempt: 1
|
|
job_id: job_2
|
|
task_id: 47
|
|
status: 1
|
|
started: 1683636528
|
|
stopped: 1683636626
|
|
-
|
|
id: 393
|
|
run_id: 891
|
|
repo_id: 1
|
|
owner_id: 1
|
|
commit_sha: 985f0301dba5e7b34be866819cd15ad3d8f508ee
|
|
is_fork_pull_request: false
|
|
name: job_2
|
|
attempt: 1
|
|
handle: 18e9cf40-c2f6-409f-b832-b945ea7dc79b
|
|
job_id: job_2
|
|
task_id: 47
|
|
status: 5
|
|
runs_on: '["ubuntu-latest"]'
|
|
started: 1683636528
|
|
stopped: 1683636626
|
|
-
|
|
id: 394
|
|
run_id: 891
|
|
repo_id: 1
|
|
owner_id: 2
|
|
commit_sha: 985f0301dba5e7b34be866819cd15ad3d8f508ee
|
|
is_fork_pull_request: false
|
|
name: job_2
|
|
attempt: 2
|
|
handle: a723d3e3-49a1-4e6b-947f-e987e60bfbd6
|
|
job_id: job_2
|
|
task_id: 47
|
|
status: 5
|
|
runs_on: '["debian-latest"]'
|
|
started: 1683636528
|
|
stopped: 1683636626
|
|
-
|
|
id: 395
|
|
run_id: 891
|
|
repo_id: 1
|
|
owner_id: 3
|
|
commit_sha: 985f0301dba5e7b34be866819cd15ad3d8f508ee
|
|
is_fork_pull_request: false
|
|
name: job_2
|
|
attempt: 1
|
|
handle: 40317a2f-2f00-4a82-8cc4-57347989a493
|
|
job_id: job_2
|
|
task_id: 47
|
|
status: 5
|
|
runs_on: '["fedora"]'
|
|
started: 1683636528
|
|
stopped: 1683636626
|
|
-
|
|
id: 396
|
|
run_id: 794
|
|
repo_id: 4
|
|
owner_id: 1
|
|
commit_sha: 985f0301dba5e7b34be866819cd15ad3d8f508ee
|
|
is_fork_pull_request: false
|
|
name: job_2
|
|
attempt: 1
|
|
job_id: job_2
|
|
task_id: null
|
|
status: 5
|
|
runs_on: '["fedora"]'
|
|
started: 1683636528
|
|
stopped: 1683636626
|
|
-
|
|
id: 397
|
|
run_id: 892
|
|
repo_id: 1
|
|
owner_id: 3
|
|
commit_sha: ea09cba7099f7e3bea80dee1a42f0b5d5e127f2e
|
|
is_fork_pull_request: false
|
|
name: job_3
|
|
attempt: 1
|
|
job_id: job_3
|
|
task_id: 48
|
|
status: 5
|
|
started: 1683636528
|
|
stopped: 1683636626
|