search

kubernetes-retired / rktlet

[EOL] The rkt implementation of the Kubernetes Container Runtime Interface
Apache License 2.0
137 stars 43 forks source link

docs: Initial design document #4

Closed euank closed 8 years ago

euank commented 8 years ago

The proposal in #3 was for discussion.

This attempts to distil that discussion into specific decisions and supersedes that PR.

This PR is fairly light on details, and I'd be happy to include more details or entertain more discussion on some parts, though the main motivation is to get the ball rolling.

cc @kubernetes-incubator/maintainers-rktlet @vishh

yifan-gu commented 8 years ago

LGTM

yifan-gu commented 8 years ago

Fix #2

euank commented 8 years ago

@vishh's comments about the difficulties of a separate repo are well-said and worth noting, but based on offline discussion I think they're non-blocking.

Since we can also revisit this choice in the future with more evidence, I think the best course of action is to continue as a separate repository for now.

Based on @yifan-gu's LGTM and discussion in #2, I don't think there's any reason not to merge this and handle any further changes/refinement/details/the "design picture TODO" in future PRs.