cascading-pr from https://codeberg.org/forgejo/forgejo refs/pull/3059/head to forgejo/forgejo-3059 #146

Closed
cascading-pr wants to merge 6 commits from cascading-pr:forgejo/forgejo-3059 into main
First-time contributor

cascading-pr from https://codeberg.org/forgejo/forgejo refs/pull/3059/head

verify runs-on variable expansion happens on scheduled workflows

  • Adds the TEST_SCHEDULE_RUNSON=docker variable
  • Use it in a scheduled job
  • Run this test only with Forgejo versions greater or equal to 8.0
    otherwise it is not implemented yet

image

cascading-pr from https://codeberg.org/forgejo/forgejo refs/pull/3059/head verify runs-on variable expansion happens on scheduled workflows * Adds the TEST_SCHEDULE_RUNSON=docker variable * Use it in a scheduled job * Run this test only with Forgejo versions greater or equal to 8.0 otherwise it is not implemented yet ![image](/attachments/b96f6b31-efeb-4b51-b9d8-5e3995d48bb4)
cascading-pr added 1 commit 2024-04-06 07:21:57 +00:00
cascading-pr update
Some checks failed
/ build (pull_request) Successful in 8m36s
/ packages (pull_request) Successful in 18m17s
/ actions-docs (map[branch:next forgejo:https://codeberg.org owner:forgejo-experimental version:7.0]) (pull_request) Has been skipped
/ actions-docs (map[forgejo:https://codeberg.org owner:forgejo version:1.21]) (pull_request) Has been skipped
/ actions (pull_request) Has been cancelled
/ upgrade and storage (pull_request) Has been cancelled
b119e3fbc2
earl-warren added 2 commits 2024-04-06 07:44:01 +00:00
DEBUG REMOVE THIS COMMIT BEFORE MERGING
Some checks failed
/ build (pull_request) Successful in 4m11s
/ actions (pull_request) Failing after 6m38s
825866d16b
actions: add example-runs-on-expression
Some checks failed
/ build (pull_request) Successful in 11m12s
/ actions (pull_request) Failing after 17m7s
7aa5967047
verify runs on value is evaluated when it contains an expression
earl-warren force-pushed forgejo/forgejo-3059 from 7aa5967047 to 825866d16b 2024-04-06 08:26:57 +00:00 Compare
earl-warren added 1 commit 2024-04-06 09:39:39 +00:00
actions: add example-7-0-schedule
Some checks failed
/ build (pull_request) Successful in 10m8s
/ actions (pull_request) Failing after 9m21s
5653e706dd
verify runs-on variable expansion happens on scheduled workflows

* Adds the TEST_SCHEDULE_RUNSON=docker variable
* Use it in a scheduled job
* Run this test only with Forgejo versions greater or equal to 8.0
  otherwise it is not implemented yet
earl-warren force-pushed forgejo/forgejo-3059 from 5653e706dd to a62c82c36e 2024-04-06 09:59:18 +00:00 Compare
earl-warren force-pushed forgejo/forgejo-3059 from a62c82c36e to 6a9502dfe2 2024-04-06 10:04:16 +00:00 Compare
Owner

@thefox it looks like this error is relevant.

2024/04/06 10:21:42 ...rvices/cron/tasks.go:93:1() [E] PANIC whilst running task: start_schedule_tasks Value: runtime error: invalid memory address or nil pointer dereference

is relevant.

@thefox it looks like [this error](https://code.forgejo.org/forgejo/end-to-end/actions/runs/1018/jobs/1#jobstep-4-1013) is relevant. ``` 2024/04/06 10:21:42 ...rvices/cron/tasks.go:93:1() [E] PANIC whilst running task: start_schedule_tasks Value: runtime error: invalid memory address or nil pointer dereference ``` is relevant.
Member

My guess is that the repo wasn't loaded. I'll look into it, maybe the non-scheduled way had an init before which I missed.
EDIT:
That indeed was the case

My guess is that the repo wasn't loaded. I'll look into it, maybe the non-scheduled way had an init before which I missed. EDIT: That indeed was the case
cascading-pr added 1 commit 2024-04-06 14:54:42 +00:00
cascading-pr update
All checks were successful
/ build (pull_request) Successful in 3m20s
/ actions (pull_request) Successful in 3m44s
53d4e289d0
Owner

@thefox now that it passed and the PR has been merged, could you make a new PR out of this one to add the test so it runs on 8.0-test & 8.0-dev?

@thefox now that it passed and the PR has been merged, could you make a new PR out of this one to add the test so it runs on 8.0-test & 8.0-dev?
cascading-pr added 1 commit 2024-04-06 19:58:22 +00:00
cascading-pr update
All checks were successful
/ build (pull_request) Successful in 4m11s
/ actions (pull_request) Successful in 6m29s
5ad13238b1
cascading-pr added 1 commit 2024-04-06 20:03:16 +00:00
cascading-pr update
All checks were successful
/ build (pull_request) Successful in 6m8s
/ actions (pull_request) Successful in 9m12s
569b424462
earl-warren closed this pull request 2024-04-13 10:08:03 +00:00
Some checks are pending
/ build (pull_request) Successful in 6m8s
Required
Details
/ actions (pull_request) Successful in 9m12s
Required
Details
/ packages (pull_request)
Required
/ upgrade and storage (pull_request)
Required

Pull request closed

Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
3 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: forgejo/end-to-end#146
No description provided.