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

Update to latest docker client #6698

Merged
merged 1 commit into from
Apr 13, 2015
Merged

Conversation

pweil-
Copy link
Contributor

@pweil- pweil- commented Apr 10, 2015

@googlebot
Copy link

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project, in which case you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


  • If you've already signed a CLA, it's possible we don't have your GitHub username or you're using a different email address. Check your existing CLA data and verify that your email is set on your git commits.
  • If you signed the CLA as a corporation, please let us know the company's name.

@dchen1107 dchen1107 self-assigned this Apr 10, 2015
@dchen1107
Copy link
Member

LGTM. Can you run e2e test before merge?

@pweil-
Copy link
Contributor Author

pweil- commented Apr 13, 2015

Will do. Thanks!

@pweil-
Copy link
Contributor Author

pweil- commented Apr 13, 2015

@dchen1107 - I have a clean gce run of most of the e2e tests. For some reason the MasterCerts spec hangs for me. I can run the same gcloud ls command in another window and see it return with all the files the test should be looking for.

[pweil@localhost kubernetes]$ go run hack/e2e.go -v -test -test_args="-t Docker Containers -t Pods -t PD -t Networking -t Cadvisor -t emptyDir -t Events -t ReplicationController -t Services"

<snip>

Ran 24 of 39 Specs in 548.645 seconds
SUCCESS! -- 24 Passed | 0 Failed | 0 Pending | 15 Skipped I0413 17:38:49.167678   21765 driver.go:114] All tests pass

@dchen1107
Copy link
Member

Thanks!

dchen1107 added a commit that referenced this pull request Apr 13, 2015
@dchen1107 dchen1107 merged commit 7464575 into kubernetes:master Apr 13, 2015
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