Skip to content

fix(aws.js): install actions/runner 2.338.0 - #2

Merged
psofiterol merged 2 commits into
mainfrom
fix/runner-2.338.0
Oct 14, 2026
Merged

psofiterol merged 2 commits into
mainfrom
fix/runner-2.338.0

Conversation

@psofiterol

@psofiterol psofiterol commented Oct 14, 2026 •

Copy link
Copy Markdown
Member

Summary

  • Since 2026-09-29, GitHub rejects self-hosted runners older than 2.329.0. This action installs 2.313.0, so the EC2 instances it starts never register, and every job waiting on them fails. The nil-stats nightly is one of the casualties (see NillionNetwork/nil-stats#22).
  • This bumps the runner download in src/aws.js to 2.338.0 (the current actions/runner release, 2026-10-06) and rebuilds dist/index.js.

Verification

  • npm ci && npm run package reproduces the committed dist/index.js byte-for-byte.
  • npm run lint is clean. The second commit fixes the 2 errors that were already on main and failed the PR lint check: an unused idx parameter, and SubnetId set twice with the same value. Neither changes behaviour.

GitHub has rejected runner registrations below 2.329.0 since 2026-09-29, so
instances started with 2.313.0 never come online and the jobs waiting for
them fail. Bump to 2.338.0, the current release.

dist/index.js is rebuilt with npm ci && npm run package.
buildUserDataScript took an `idx` parameter that no caller passes (the
`$idx` in the script is the bash loop variable), and RunInstances params set
SubnetId twice with the same value. Neither change alters behaviour.
@psofiterol
psofiterol merged commit d7121fd into main Oct 14, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant