Update https://data.forgejo.org/actions/setup-forgejo action to v3.1.3 #1331
No reviewers
Labels
No labels
FreeBSD
Kind/Breaking
Kind/Bug
Kind/Chore
Kind/DependencyUpdate
Kind/Documentation
Kind/Enhancement
Kind/Feature
Kind/Security
Kind/Testing
Priority
Critical
Priority
High
Priority
Low
Priority
Medium
Reviewed
Confirmed
Reviewed
Duplicate
Reviewed
Invalid
Reviewed
Won't Fix
Status
Abandoned
Status
Blocked
Status
Need More Info
Windows
linux-powerpc64le
linux-riscv64
linux-s390x
run-end-to-end-tests
run-forgejo-tests
No milestone
No project
No assignees
4 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
forgejo/runner!1331
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "renovate/https-data.forgejo.org-actions-setup-forgejo-3.x"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
This PR contains the following updates:
v3.1.1→v3.1.3Release Notes
actions/setup-forgejo (https://data.forgejo.org/actions/setup-forgejo)
v3.1.3Compare Source
v3.1.2Compare Source
Configuration
📅 Schedule: Branch creation - Between 12:00 AM and 03:59 AM ( * 0-3 * * * ) (UTC), Automerge - Between 12:00 AM and 03:59 AM ( * 0-3 * * * ) (UTC).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR has been generated by Renovate Bot.
cascading-pr updated at actions/setup-forgejo#857
test failure
https://code.forgejo.org/forgejo/runner/actions/runs/15801/jobs/0/attempt/1#jobstep-3-3563
🤔
I haven't been able to track this down, but here's some artifacts from investigation so far.
This error originates from:
throw new TypeError(`Input does not meet YAML 1.2 "Core Schema" specification: ${name}\n` +`Support boolean input list: \`true | True | TRUE | false | False | FALSE\``);This is part of the
${{ hashfiles(...) }}routine.build-release.ymlfrom this branch (unmodified, though).docker/setup-buildx-action@36590ad0c1 (diff-311d855470)keep-stateinput, which is noted in the error message.So... it seems like something is calling
hashfileson the action, which I can't find. The actual contents in setup-buildx-action look like they should be valid, as well.I'll try to perform similar simulations of
build-release.ymlin the near future and see if I can reproduce this problem on a local Forgejo/Runner, which would allow sprinkling it with some logging to see what is happening. At the moment it's a bit obscure, but at least I can see relevant changes being around, just not connecting up.I don't know if it actually says there's a problem in Runner v12.6.1. It's possible it's only a problem in the repo's workflows due to some of these action bumps. I'm not sure though.
I've identified the problem. The
keep-stateinput it sent to the node process ofsetup-buildxby an environment variable,INPUT_KEEP-STATE.As part of replacing
lxc-enterwithnsenter, I used a small shell script with/bin/shto change the current process's cgroup to.lxc, matching thelxc-enterbehaviour:"/bin/sh","-c",`echo $$ > /sys/fs/cgroup/.lxc/cgroup.procs 2>/dev/null || true; exec $@`,"/bin/sh",Experimentally, it seems that
/bin/shis stripping out theINPUT_KEEP-STATEenvironment variable. This may be considered consistent with its documentation;/bin/shin Debian is implemented by dash, does this: "When starting up, the shell turns all the environment variables into shell variables.", and the POSIX standard for environment variables are that they consist solely of uppercase letters, digits, and underscores, resulting in/bin/shstripping outINPUT_KEEP-STATEbecause it has a dash in it.That causes
setup-buildxto fail because it's looking forINPUT_KEEP-STATEto have a value and it's a required field. I have a fix in hand to replace/bin/shwith/bin/bashwhich addresses the bug, and I'm testing it.At the moment, runner v12.6.0 & v12.6.1 are impacted and will fail to pass environment variables and inputs to actions which include a
-in the name, when using the LXC environment.Update https://data.forgejo.org/actions/setup-forgejo action to v3.1.2to Update https://data.forgejo.org/actions/setup-forgejo action to v3.1.3d3abf081767285b1b66ecascading-pr updated at actions/setup-forgejo#857