Skip to content

ci: run integration tests with the remote multi-node worker#3724

Merged
tonistiigi merged 2 commits intodocker:masterfrom
crazy-max:tests-multi-node
Mar 12, 2026
Merged

ci: run integration tests with the remote multi-node worker#3724
tonistiigi merged 2 commits intodocker:masterfrom
crazy-max:tests-multi-node

Conversation

@crazy-max
Copy link
Member

follow-up #3716 (comment)

This adds a dedicated remote+multinode integration worker backed by the docker test environment. The new worker provisions two docker-container builders for different platforms and exposes them as a single appended remote builder. This gives the integration suite a stable multi-node topology.

@crazy-max
Copy link
Member Author

crazy-max commented Mar 12, 2026

Following tests fail with multi-node:

 === Failed
=== FAIL: tests TestIntegration/TestHistoryBuildName/worker=remote+multinode/bake_git (0.44s)
    history.go:291: 
        	Error Trace:	/src/tests/history.go:291
        	Error:      	Received unexpected error:
        	            	exit status 1
        	Test:       	TestIntegration/TestHistoryBuildName/worker=remote+multinode/bake_git
        --- FAIL: TestIntegration/TestHistoryBuildName/worker=remote+multinode/bake_git (0.44s)

=== FAIL: tests TestIntegration/TestHistoryBuildName/worker=remote+multinode (2.30s)
    --- FAIL: TestIntegration/TestHistoryBuildName/worker=remote+multinode (2.30s)

=== FAIL: tests TestIntegration/TestHistoryLs/worker=remote+multinode (1.50s)
    history.go:85: 
        	Error Trace:	/src/tests/history.go:85
        	            				/src/vendor/github.com/moby/buildkit/util/testutil/integration/run.go:103
        	            				/src/vendor/github.com/moby/buildkit/util/testutil/integration/run.go:251
        	Error:      	Received unexpected error:
        	            	exit status 1
        	Test:       	TestIntegration/TestHistoryLs/worker=remote+multinode
    --- FAIL: TestIntegration/TestHistoryLs/worker=remote+multinode (1.50s)

Slightly relates to #3716, skipping for now.

@crazy-max crazy-max requested a review from tonistiigi March 12, 2026 10:33
@crazy-max crazy-max marked this pull request as ready for review March 12, 2026 10:33
Signed-off-by: CrazyMax <1951866+crazy-max@users.noreply.github.com>
Signed-off-by: CrazyMax <1951866+crazy-max@users.noreply.github.com>
@tonistiigi tonistiigi merged commit 18c3dc2 into docker:master Mar 12, 2026
161 checks passed
@crazy-max crazy-max deleted the tests-multi-node branch March 12, 2026 18:39
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