Skip to content

ci: isolate S3 upload workspace#144

Merged
CodeNinjaEvan merged 1 commit into
tronprotocol:release_0.8.30from
yanghang8612:ci/isolate-s3-upload-workspace
Jul 24, 2026
Merged

ci: isolate S3 upload workspace#144
CodeNinjaEvan merged 1 commit into
tronprotocol:release_0.8.30from
yanghang8612:ci/isolate-s3-upload-workspace

Conversation

@yanghang8612

Copy link
Copy Markdown

Summary

  • download release artifacts into a run- and attempt-specific directory under RUNNER_TEMP
  • run permission restoration, packaging, and S3 upload from that isolated directory
  • clean the temporary workspace with an always() step

Root cause

The self-hosted for-linux runner reuses its repository workspace. A root-owned github/ directory was recreated there before the upload job started, so the java-tron runner user could not extract solc-windows.exe. This caused actions/download-artifact to retry five times and fail before the S3 upload step.

Impact

The upload job no longer depends on the ownership or contents of the persistent repository workspace. Each workflow run attempt uses ${RUNNER_TEMP}/solidity-${GITHUB_RUN_ID}-${GITHUB_RUN_ATTEMPT} and removes it on success or failure.

This addresses the failure observed in https://github.com/tronprotocol/solidity/actions/runs/30004204683.

Validation

  • parsed .github/workflows/build.yml successfully as YAML
  • ran git diff --check
  • ran actionlint with the repository's custom for-linux runner label ignored
  • confirmed the commit changes only .github/workflows/build.yml

@github-actions

Copy link
Copy Markdown

Thank you for your contribution to the Solidity compiler! A team member will follow up shortly.

If you haven't read our contributing guidelines and our review checklist before, please do it now, this makes the reviewing process and accepting your contribution smoother.

If you have any questions or need our help, feel free to post them in the PR or talk to us directly on the #solidity-dev channel on Matrix.

@yanghang8612
yanghang8612 marked this pull request as ready for review July 24, 2026 04:12
@CodeNinjaEvan
CodeNinjaEvan merged commit b520850 into tronprotocol:release_0.8.30 Jul 24, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants