Skip to content

Commit 253ffbf

Browse files
fix(test): the Connect-only sync test owns its containers
`codefly publish` runs the source tests below the agent process, so the test binary inherits the agent's CODEFLY_CONTAINER_RECOVERY_SCOPE marker, which names a pid that is neither the test binary nor its parent. Core refuses to create a container under it, so TestConnectOnlyServiceSyncs (4c35bc0, #113) failed release-grade agent CI with "container recovery marker does not belong to this process or its launching parent" and aborted the 0.0.122 publish. GitHub CI sets no marker, so main was green. The test now calls ownTestContainers, as every other container-backed test here does, and takes a scope of its own. Verified: with a foreign marker in the environment the test fails with that refusal before this change; after it, the refusal is gone in both the marker and no-marker runs. Both runs then stop at Buf's BSR rate limit ("resource_exhausted: too many requests", no BSR login on this machine), and the unchanged test fails the same way with no marker, so that stop is environmental and not this change. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
1 parent c43d3c3 commit 253ffbf

1 file changed

Lines changed: 1 addition & 0 deletions

File tree

‎connect_only_test.go‎

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -142,6 +142,7 @@ func TestConnectOnlyServiceSyncs(t *testing.T) {
142142
if err := exec.Command("docker", "info").Run(); err != nil {
143143
t.Skipf("docker is not available, Buf cannot run: %v", err)
144144
}
145+
ownTestContainers(t)
145146
ctx := context.Background()
146147
created, root := createConnectOnlyService(t)
147148
identity, err := created.Identity.Proto()

0 commit comments

Comments
 (0)