Skip to content

vendor: test moby dockerd integration tests with custom docker client#49459

Closed
jsternberg wants to merge 1 commit intomoby:masterfrom
jsternberg:buildkit-remove-docker-api-dependency
Closed

vendor: test moby dockerd integration tests with custom docker client#49459
jsternberg wants to merge 1 commit intomoby:masterfrom
jsternberg:buildkit-remove-docker-api-dependency

Conversation

@jsternberg
Copy link
Copy Markdown
Collaborator

- What I did
Moved a slim version of the docker api client to buildkit for its integration tests.

moby/buildkit#5731

- How I did it
Copied over the client from moby and then deleted sections until it used the subset needed by buildkit's integration tests.

- How to verify it
The tests should pass in this draft PR.

- A picture of a cute animal (not mandatory but encouraged)

@jsternberg jsternberg force-pushed the buildkit-remove-docker-api-dependency branch 4 times, most recently from 00e3925 to 11a008b Compare February 13, 2025 16:56
@jsternberg jsternberg force-pushed the buildkit-remove-docker-api-dependency branch 7 times, most recently from 3e092ec to 04354bc Compare February 24, 2025 17:31
Signed-off-by: Jonathan A. Sternberg <jonathan.sternberg@docker.com>
@jsternberg jsternberg force-pushed the buildkit-remove-docker-api-dependency branch from 04354bc to 2270db8 Compare February 25, 2025 02:52
@thaJeztah thaJeztah closed this Jul 27, 2025
@jsternberg jsternberg deleted the buildkit-remove-docker-api-dependency branch July 28, 2025 14:53
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