Skip to content

vendor: Remove go-dockerclient dependency#122

Merged
aeneasr merged 6 commits into
v3from
fix-deps
Apr 11, 2018
Merged

vendor: Remove go-dockerclient dependency#122
aeneasr merged 6 commits into
v3from
fix-deps

Conversation

@aeneasr

@aeneasr aeneasr commented Apr 11, 2018

Copy link
Copy Markdown
Member

No description provided.

@aeneasr

aeneasr commented Apr 11, 2018

Copy link
Copy Markdown
Member Author

ping @dennwc

@aeneasr aeneasr merged commit 2e92e77 into v3 Apr 11, 2018
@aeneasr aeneasr deleted the fix-deps branch April 11, 2018 18:02
@johanbrandhorst

Copy link
Copy Markdown

I'm confused by this. Did you simply copy the vendored package into your own directory? What is the point of that?

@aeneasr

aeneasr commented Apr 12, 2018

Copy link
Copy Markdown
Member Author

go-dockerclient had a dependency on docker-swarm which pulled a ton of external dependencies. Instead of forking go-dockerclient I vendored the docker dependency here and removed the swarm capabilities which we don't need anyways. Still has some dependencies to smaller repos but we can strip them as time passes

@aeneasr

aeneasr commented Apr 12, 2018

Copy link
Copy Markdown
Member Author

Another issue addressed by this is the ridiculous instability of dep and the docker repository in general. I got failing builds everywhere because dep was unable to copy some unrelated go files (.pem test fixtures). Vendoring it in here will resolve this, but we might have to update this if docker releases breaking changes.

@johanbrandhorst

Copy link
Copy Markdown

I see. Have you considered just forking go-dockerclient instead and using that repo as a proper vendor directory?

@aeneasr

aeneasr commented Apr 12, 2018

Copy link
Copy Markdown
Member Author

Yes, I did. I don't want to give the impression though that it's officially maintained and an alternative to go-dockerclient because it's not.

@aeneasr

aeneasr commented Apr 12, 2018

Copy link
Copy Markdown
Member Author

I would also prefer to just use http.DefaultClient to talk to Docker but I don't have the time to dig in and it also seems like there's more to it (like handling their streams, properly reading auth data, ...)

xstevens pushed a commit to xstevens/dockertest that referenced this pull request Dec 2, 2019
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