Skip to content

ci: Set Docker API version explicitly - #62

Closed
qezz wants to merge 1 commit into
mainfrom
patch-set-docker-api-version
Closed

ci: Set Docker API version explicitly#62
qezz wants to merge 1 commit into
mainfrom
patch-set-docker-api-version

Conversation

@qezz

@qezz qezz commented Nov 17, 2025

Copy link
Copy Markdown
Contributor

Apparently Docker Engine v29 broke some compatibility with testcontainers, but setting this var should work.

Apparently Docker Engine v29 broke some compatibility with
testcontainers, but setting this var should work.
@qezz
qezz requested a review from a team November 17, 2025 13:57
@qezz

qezz commented Nov 17, 2025

Copy link
Copy Markdown
Contributor Author

If tests work, then it works as expected.

Broken tests were failing as:

--- FAIL: TestIsImagePresent (60.01s)
    util.go:112: 
        	Error Trace:	/home/runner/work/blazar/blazar/internal/pkg/testutils/util.go:112
        	            				/home/runner/work/blazar/blazar/internal/pkg/docker/docker_test.go:16
        	Error:      	Received unexpected error:
        	            	Error response from daemon: client version 1.43 is too old. Minimum supported API version is 1.44, please upgrade your client to a newer version
        	            	error during connect: Post "http://%2Fvar%2Frun%2Fdocker.sock/v1.43/build?buildargs=null&cachefrom=null&cgroupparent=&cpuperiod=0&cpuquota=0&cpusetcpus=&cpusetmems=&cpushares=0&dockerfile=Dockerfile&forcerm=1&labels=null&memory=0&memswap=0&networkmode=&rm=1&shmsize=0&t=img%3Atag&target=&ulimits=null&version=": io: read/write on closed pipe

@qezz

qezz commented Nov 17, 2025

Copy link
Copy Markdown
Contributor Author

tests failed

@qezz

qezz commented Nov 17, 2025

Copy link
Copy Markdown
Contributor Author

ok so setting docker version won't work, obviously, because it's testcontainers that got outdated, not docker

@qezz

qezz commented Nov 17, 2025

Copy link
Copy Markdown
Contributor Author

closing in favor of #63

@qezz qezz closed this Nov 17, 2025
@qezz
qezz deleted the patch-set-docker-api-version branch June 22, 2026 15:43
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