Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

bump(github.com/fsouza/go-dockerclient) c68e3a80d465c178e452c843ccd9928570893203 #4467

Closed
wants to merge 1 commit into from

Conversation

csrwng
Copy link
Contributor

@csrwng csrwng commented Sep 2, 2015

Includes fix for BZ 1255502

@csrwng
Copy link
Contributor Author

csrwng commented Sep 2, 2015

@bparees ptal

@mfojtik
Copy link
Member

mfojtik commented Sep 2, 2015

@csrwng is this with sync of what we have in source-to-image repo?

@csrwng
Copy link
Contributor Author

csrwng commented Sep 2, 2015

@mfojtik, no... will create a pull there as well. Thx for reminding me

@liggitt
Copy link
Contributor

liggitt commented Sep 2, 2015

might need to wait on this... @ncdc has exec changes in flight that might need to go in concurrent with this

@liggitt
Copy link
Contributor

liggitt commented Sep 2, 2015

also, did we ever resolve the version compatibility issues? will this version work against older dockers?

@csrwng
Copy link
Contributor Author

csrwng commented Sep 2, 2015

yes, it checks the version of server API
fsouza/go-dockerclient#365

@liggitt
Copy link
Contributor

liggitt commented Sep 2, 2015

we need to stay in sync with upstream on the go-dockerclient used... so this needs to be paired with a PR there, but you may want to check with @ncdc first. this version will break oc exec until his changes are ready

@csrwng
Copy link
Contributor Author

csrwng commented Sep 2, 2015

cool, @ncdc I'll leave it up to you to do the proper updates after your changes go in

@csrwng csrwng closed this Sep 2, 2015
@csrwng csrwng deleted the update_dockerclient branch January 27, 2016 15:25
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.

None yet

3 participants