Skip to content

ci: cache the license-eye build in the Check License Headers job - #7357

Draft
Amer-Mukhtar wants to merge 1 commit into
apache:mainfrom
Amer-Mukhtar:ci/cache-license-eye-build
Draft

ci: cache the license-eye build in the Check License Headers job#7357
Amer-Mukhtar wants to merge 1 commit into
apache:mainfrom
Amer-Mukhtar:ci/cache-license-eye-build

Conversation

@Amer-Mukhtar

Copy link
Copy Markdown
Contributor

What changes were proposed in this PR?

The Check License Headers job uses the apache/skywalking-eyes action, which sets
up Go and builds the license-eye tool from source on every run. Building the tool
takes about as long as the scan it exists to do.

This PR keeps the action but caches the binary it installs. On an empty cache
nothing changes: the action builds the tool, installs it, and scans, and the
binary is saved at the end. On later runs the binary is restored, the action is
skipped, and the job goes straight to the scan.

The cache key is the hash of the workflow file. The action version is pinned in
that same file, so bumping the pin changes the key and the tool is rebuilt. If
the key held a copy of the pinned commit instead, a Renovate bump would leave it
stale and later runs would keep using the old binary.

The action is still referenced with a uses line rather than replaced by a direct
go install, because Renovate only tracks dependencies written that way.

Cold runs cost the same as before, so the saving shows up only on warm runs.

Any related issues, documentation, discussions?

Closes #7136

How was this PR tested?

Workflow files have no test framework, so the check is CI on this pull request.
The first run is a cache miss and should behave exactly as it does today.
Re-running the job takes the warm path, and its log should show the scan with no
Go setup or build step before it. Both runs need to pass.

Was this PR authored or co-authored using generative AI tooling?

Generated-by: Claude Code

@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Automated Reviewer Suggestions

Based on the git blame history of the changed files, we recommend the following reviewers:

  • No candidates found from git blame history.

@github-actions github-actions Bot added the ci changes related to CI label Aug 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci changes related to CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Cache the license-eye build in the Check License Headers job

1 participant