Skip to content

fix(ci): sui binary caching#20539

Merged
erikburt merged 1 commit intodevelopfrom
fix/setup-sui-cache
Dec 5, 2025
Merged

fix(ci): sui binary caching#20539
erikburt merged 1 commit intodevelopfrom
fix/setup-sui-cache

Conversation

@erikburt
Copy link
Collaborator

@erikburt erikburt commented Dec 5, 2025

Changes

  • Fix the installation path for the sui binary so it can be cached

Motivation

There were installation issues which were fixed in #20533, but it broke the SUI binary cache.

For example: https://github.com/smartcontractkit/chainlink/actions/runs/19968771608/job/57267681721#step:38:3

Warning: Path Validation Error: Path(s) specified in the action for caching do(es) not exist, hence no cache is being saved.

Testing

@erikburt erikburt self-assigned this Dec 5, 2025
@erikburt erikburt force-pushed the fix/setup-sui-cache branch from f9ef7bc to d78e087 Compare December 5, 2025 18:41
@erikburt erikburt changed the title test: sui bin location fix(ci): sui bin caching Dec 5, 2025
@erikburt erikburt changed the title fix(ci): sui bin caching fix(ci): sui binary caching Dec 5, 2025
@erikburt erikburt marked this pull request as ready for review December 5, 2025 18:58
@erikburt erikburt requested review from a team as code owners December 5, 2025 18:58
Copilot AI review requested due to automatic review settings December 5, 2025 18:58
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR fixes the Sui binary caching mechanism in GitHub Actions by correcting the installation path used for caching. The previous path (~/.suiup/bin) no longer exists after changes in #20533, causing cache validation errors. The fix updates the cache path to the actual installation location (~/.local/bin/sui) and removes the obsolete path from the PATH environment variable.

Key Changes:

  • Updated cache path from ~/.suiup/bin to ~/.local/bin/sui to match the actual installation location
  • Removed ~/.suiup/bin from the PATH environment variable since it's no longer used

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@erikburt erikburt requested review from a team December 5, 2025 19:08
@cl-sonarqube-production
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@erikburt erikburt enabled auto-merge December 5, 2025 19:08
@erikburt erikburt added this pull request to the merge queue Dec 5, 2025
Merged via the queue into develop with commit d77c815 Dec 5, 2025
111 checks passed
@erikburt erikburt deleted the fix/setup-sui-cache branch December 5, 2025 19:39
@trunk-io
Copy link

trunk-io bot commented Dec 5, 2025

Static BadgeStatic BadgeStatic BadgeStatic Badge

View Full Report ↗︎Docs

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