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

Removes RunPod in the Runtime interface #7657

Merged
merged 1 commit into from May 1, 2015

Conversation

Projects
None yet
4 participants
@yujuhong
Copy link
Member

yujuhong commented May 1, 2015

SyncPod subsumes RunPod, so the latter is no longer needed.
After this change, DockerManager implements the Runtime interface.

Removes RunPod in the Runtime interface
SyncPod subsumes RunPod, so the latter is no longer needed.
After this change, DockerManager implements the Runtime interface.

@googlebot googlebot added the cla: yes label May 1, 2015

@yujuhong

This comment has been minimized.

Copy link
Member Author

yujuhong commented May 1, 2015

@vmarmol vmarmol self-assigned this May 1, 2015

@vmarmol

This comment has been minimized.

Copy link
Contributor

vmarmol commented May 1, 2015

LGTM!

@yifan-gu

This comment has been minimized.

Copy link
Member

yifan-gu commented May 1, 2015

LGTM

vmarmol added a commit that referenced this pull request May 1, 2015

Merge pull request #7657 from yujuhong/docker_runtime
Removes RunPod in the Runtime interface

@vmarmol vmarmol merged commit c5a8638 into kubernetes:master May 1, 2015

2 of 3 checks passed

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
Shippable Shippable builds completed
Details
cla/google All necessary CLAs are signed

@yujuhong yujuhong deleted the yujuhong:docker_runtime branch May 8, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.