Skip to content

chore: bump CTF framework to fix Sui/Aptos cwd tar race#929

Merged
Fletch153 merged 1 commit intomainfrom
fix-sui-aptos-ctf-cwd-tar-race
Apr 15, 2026
Merged

chore: bump CTF framework to fix Sui/Aptos cwd tar race#929
Fletch153 merged 1 commit intomainfrom
fix-sui-aptos-ctf-cwd-tar-race

Conversation

@Fletch153
Copy link
Copy Markdown
Contributor

@Fletch153 Fletch153 commented Apr 14, 2026

Summary

Bumps github.com/smartcontractkit/chainlink-testing-framework/framework to v0.15.16 to pick up the upstream Sui/Aptos CTF provider cwd-tar-race fix.

Upstream PR: smartcontractkit/chainlink-testing-framework#2519 (merged as v0.15.16).

Root Cause

chain/sui/provider/ctf_provider.go and chain/aptos/provider/ctf_provider.go both build blockchain.Input with no ContractsDir. Upstream CTF's newSui/newAptos called filepath.Abs(in.ContractsDir) unconditionally — Go stdlib resolves filepath.Abs("") to the process cwd. That cwd was attached as a testcontainers.ContainerFile, tarring the host test working directory. Files being written concurrently (logs, db dumps) trip archive/tar.ErrWriteTooLong between os.Stat (header size) and io.Copy (body).

The retry.Attempts(10) wrapper in startContainer masks most occurrences. Test_CTFChainProvider_Initialize (which actually starts a container) hits this path in this repo's own CI.

Fix

Module bump only — no source changes. Pins framework to v0.15.16 (stable tag), which gates the Files: entry on ContractsDir != "".

Changeset included: .changeset/fix-sui-aptos-cwd-tar-race.md (patch).

Safety

  • Callers passing a non-empty ContractsDir: identical behaviour.
  • Callers passing empty (both Sui and Aptos providers in this repo): req.Files now nil. testcontainers-go's hook iterates range files — zero iterations on nil, no-op (verified upstream in lifecycle.go:151).
  • Delta in framework/components/blockchain/ from previous pin (v0.15.13) to v0.15.16: 1 unrelated commit (#2517 update sui version) + the 2 fix commits. Minimal transitive risk.

Test plan

  • go build ./... clean
  • go mod tidy clean
  • CI green — 22 checks pass, 2 skipped, 0 failing

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Apr 14, 2026

🦋 Changeset detected

Latest commit: da4819e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
chainlink-deployments-framework Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Pulls in smartcontractkit/chainlink-testing-framework#2519. The Sui
and Aptos CTF providers in chain/{sui,aptos}/provider/ctf_provider.go
build blockchain.Input with no ContractsDir, which made upstream
newSui/newAptos call filepath.Abs("") = cwd and tar-stream the live
test working directory. Growing files trip archive/tar: write too long
intermittently, surfacing as retries in Test_CTFChainProvider_Initialize.

Module bump only — no source changes. Patch-level release.

Changeset: .changeset/fix-sui-aptos-cwd-tar-race.md
@Fletch153 Fletch153 force-pushed the fix-sui-aptos-ctf-cwd-tar-race branch from dbd2e69 to da4819e Compare April 14, 2026 18:55
@cl-sonarqube-production
Copy link
Copy Markdown

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

@Fletch153 Fletch153 marked this pull request as ready for review April 14, 2026 20:14
@Fletch153 Fletch153 requested a review from a team as a code owner April 14, 2026 20:14
@Fletch153 Fletch153 added this pull request to the merge queue Apr 15, 2026
Merged via the queue into main with commit 9d78329 Apr 15, 2026
24 checks passed
@Fletch153 Fletch153 deleted the fix-sui-aptos-ctf-cwd-tar-race branch April 15, 2026 16:52
This was referenced Apr 15, 2026
graham-chainlink pushed a commit that referenced this pull request Apr 16, 2026
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to ggoh/test-ci, this
PR will be updated.


# Releases
## chainlink-deployments-framework@0.95.1

### Patch Changes

-
[#929](#929)
[`9d78329`](9d78329)
Thanks [@Fletch153](https://github.com/Fletch153)! - Bump
chainlink-testing-framework/framework to pick up the Sui/Aptos CTF
provider cwd-tar-race fix (upstream
smartcontractkit/chainlink-testing-framework#2519). Resolves
intermittent `archive/tar: write too long` flakes in
`Test_CTFChainProvider_Initialize` for the Sui and Aptos providers.

## operations-gen@0.0.2

### Patch Changes

-
[`80d3475`](80d3475)
Thanks [@graham-chainlink](https://github.com/graham-chainlink)! - test

---------

Co-authored-by: app-token-issuer-engops[bot] <144731339+app-token-issuer-engops[bot]@users.noreply.github.com>
graham-chainlink pushed a commit that referenced this pull request Apr 16, 2026
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to ggoh/test-ci, this
PR will be updated.


# Releases
## chainlink-deployments-framework@0.95.1

### Patch Changes

-
[#929](#929)
[`9d78329`](9d78329)
Thanks [@Fletch153](https://github.com/Fletch153)! - Bump
chainlink-testing-framework/framework to pick up the Sui/Aptos CTF
provider cwd-tar-race fix (upstream
smartcontractkit/chainlink-testing-framework#2519). Resolves
intermittent `archive/tar: write too long` flakes in
`Test_CTFChainProvider_Initialize` for the Sui and Aptos providers.

## operations-gen@0.0.2

### Patch Changes

-
[`dbf0e57`](dbf0e57)
Thanks [@graham-chainlink](https://github.com/graham-chainlink)! - test

---------

Co-authored-by: app-token-issuer-engops[bot] <144731339+app-token-issuer-engops[bot]@users.noreply.github.com>
graham-chainlink pushed a commit that referenced this pull request Apr 16, 2026
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to ggoh/test-ci, this
PR will be updated.


# Releases
## chainlink-deployments-framework@0.95.1

### Patch Changes

-
[#929](#929)
[`9d78329`](9d78329)
Thanks [@Fletch153](https://github.com/Fletch153)! - Bump
chainlink-testing-framework/framework to pick up the Sui/Aptos CTF
provider cwd-tar-race fix (upstream
smartcontractkit/chainlink-testing-framework#2519). Resolves
intermittent `archive/tar: write too long` flakes in
`Test_CTFChainProvider_Initialize` for the Sui and Aptos providers.

## tools/operations-gen@0.0.2

### Patch Changes

-
[`dbf0e57`](dbf0e57)
Thanks [@graham-chainlink](https://github.com/graham-chainlink)! - test

---------

Co-authored-by: app-token-issuer-engops[bot] <144731339+app-token-issuer-engops[bot]@users.noreply.github.com>
github-merge-queue Bot pushed a commit that referenced this pull request Apr 16, 2026
This PR was opened by the [Changesets
release](https://github.com/changesets/action) GitHub action. When
you're ready to do a release, you can merge this and the packages will
be published to npm automatically. If you're not ready to do a release
yet, that's fine, whenever you add more changesets to main, this PR will
be updated.


# Releases
## chainlink-deployments-framework@0.95.1

### Patch Changes

-
[#929](#929)
[`9d78329`](9d78329)
Thanks [@Fletch153](https://github.com/Fletch153)! - Bump
chainlink-testing-framework/framework to pick up the Sui/Aptos CTF
provider cwd-tar-race fix (upstream
smartcontractkit/chainlink-testing-framework#2519). Resolves
intermittent `archive/tar: write too long` flakes in
`Test_CTFChainProvider_Initialize` for the Sui and Aptos providers.

-
[#938](#938)
[`9565eff`](9565eff)
Thanks [@AnieeG](https://github.com/AnieeG)! - Fix/update anvil startup
to use offical anvil image, adding --no-storage-caching option to start
up

---------

Co-authored-by: app-token-issuer-engops[bot] <144731339+app-token-issuer-engops[bot]@users.noreply.github.com>
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