Skip to content

[fix][ci] Update GraalVM setup action to ASF-approved version - #26571

Merged
merlimat merged 1 commit into
apache:masterfrom
lhotari:lh-fix-approved-graalvm-action
Sep 13, 2026
Merged

[fix][ci] Update GraalVM setup action to ASF-approved version#26571
merlimat merged 1 commit into
apache:masterfrom
lhotari:lh-fix-approved-graalvm-action

Conversation

@lhotari

@lhotari lhotari commented Sep 13, 2026

Copy link
Copy Markdown
Member

Motivation

Pulsar CI fails before starting any jobs because ASF no longer permits the pinned graalvm/setup-graalvm v1.5.4 commit. This blocks unrelated changes, including run 34751336550 and run 34770940645.

Modifications

Update graalvm/setup-graalvm to v1.6.6, pinned to 0426e2e191540e8514dff98dc52a5f5146a2a276, the latest version in ASF's approved actions list. Keep the existing GraalVM distribution and Java version settings.

Verifying this change

  • Make sure that the change passes the CI checks.

This change updates a CI action pin without adding tests.

  • Verified the SHA against the ASF allowlist and the upstream v1.6.6 tag, and checked that the configured inputs are supported.
  • git diff --check passes.
  • actionlint -shellcheck='' reports the same six existing matrix-property errors before and after this change, with no new diagnostics.
  • ./gradlew spotlessCheck checkstyleMain checkstyleTest passes.

Does this pull request potentially affect one of the following parts:

  • Dependencies (add or upgrade a dependency)
  • The public API
  • The schema
  • The default values of configurations
  • The threading model
  • The binary protocol
  • The REST endpoints
  • The admin CLI options
  • The metrics
  • Anything that affects deployment

Only the CI GraalVM setup action is upgraded.

@lhotari lhotari added this to the 5.0.0 milestone Sep 13, 2026
@lhotari

lhotari commented Sep 13, 2026

Copy link
Copy Markdown
Member Author

Follow-up: #26572 adds the ASF action allowlist check to the CI preconditions. Besides failing CI when scanned action refs are not approved, it emits non-failing warnings for approved pins expiring within 30 days, giving us advance notice to update them before they cause startup failures.

@merlimat
merlimat merged commit bd69f06 into apache:master Sep 13, 2026
48 checks 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.

3 participants