Skip to content

fix: pin docker/login-action to an ASF-approved commit SHA - #280

Merged
wu-sheng merged 1 commit into
mainfrom
fix/pin-docker-login-action
Jul 29, 2026
Merged

fix: pin docker/login-action to an ASF-approved commit SHA#280
wu-sheng merged 1 commit into
mainfrom
fix/pin-docker-login-action

Conversation

@wu-sheng

Copy link
Copy Markdown
Member

Problem

The publish-docker workflow has been failing on every push to main with
startup_failure — no jobs ever start, so there is no job log to inspect:

https://github.com/apache/skywalking-eyes/actions/runs/30420520851

Cause

.github/workflows/publish-docker.yaml referenced a third-party action by
version tag:

uses: docker/login-action@v1.10.0

The ASF GitHub Actions policy
requires every third-party action to be pinned to a reviewed commit SHA listed in
apache/infrastructure-actions/approved_patterns.yml.
Only the actions/*, apache/* and github/* namespaces are exempt — which is why
lint.yaml and license-eye-check.yaml are unaffected. A tag reference is rejected
before the run starts, hence the startup failure.

This was the only unpinned third-party action left in the repo.

Fix

Pin to 650006c6eb7dba73a995cc03b0b2d7f5ca915bee (v4.2.0), which is on the approved
list and is already the pin used by skywalking, skywalking-java, skywalking-cli,
skywalking-go, skywalking-python, skywalking-horizon-ui and skywalking-showcase.

The v1 → v4 bump is safe for this step: skywalking-cli runs the identical step shape
on this exact pin — same registry: ${{ env.HUB }} (a value carrying a path,
ghcr.io/apache/...), same github.actor / GITHUB_TOKEN credentials.

🤖 Generated with Claude Code

The publish-docker workflow failed at startup (startup_failure, no jobs
run) because `docker/login-action@v1.10.0` is referenced by version tag.
The ASF org-wide GitHub Actions policy requires every third-party action
to be pinned to a reviewed commit SHA present in
apache/infrastructure-actions/approved_patterns.yml; only the actions/*,
apache/* and github/* namespaces are exempt.

Pin to 650006c6eb7dba73a995cc03b0b2d7f5ca915bee (v4.2.0), which is on the
approved list and is the same pin already used by skywalking,
skywalking-java, skywalking-cli, skywalking-go and skywalking-python.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@wu-sheng wu-sheng added this to the 0.9.0 milestone Jul 29, 2026
@wu-sheng
wu-sheng merged commit 315732d into main Jul 29, 2026
2 checks passed
@wu-sheng
wu-sheng deleted the fix/pin-docker-login-action branch July 29, 2026 23:51
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.

2 participants