diff options
| author | Lv Lv <github@lvlv.email> | 2016-12-19 20:24:53 +0800 |
|---|---|---|
| committer | Lv Lv <github@lvlv.email> | 2016-12-19 20:24:53 +0800 |
| commit | 0bb7a0e73ef8b2eceb94d59677b8800938a96d89 (patch) | |
| tree | 9636cfd90bf49af4bacf19b3ea00f1d38b0b2faf | |
| parent | 7c3e8af9802c27d3f148cf1876a40c3a96c0e9df (diff) | |
add discussion about pod isolation
| -rw-r--r-- | contributors/design-proposals/propagation.md | 24 |
1 files changed, 24 insertions, 0 deletions
diff --git a/contributors/design-proposals/propagation.md b/contributors/design-proposals/propagation.md index b2aeb168..6e038d6b 100644 --- a/contributors/design-proposals/propagation.md +++ b/contributors/design-proposals/propagation.md @@ -150,6 +150,30 @@ non-privileged', an environment check and an WARNING log will be emitted about whether propagation mode is supported. +## Extra Concerns + +@lucab and @euank has some extra concerns about pod isolation when propagation +modes are changed, listed below: + +1. how to clean such pod resources (as mounts are now crossing pod boundaries, +thus they can be kept busy indefinitely by processes outside of the pod) + +1. side-effects on restarts (possibly piling up layers of full-propagation mounts) + +1. how does this interacts with other mount features (nested volumeMounts may or +may not propagate back to the host, depending of ordering of mount operations) + +1. limitations this imposes on runtimes (RO-remounting may now affects the host, +is it on purpose or a dangerous side-effect?) + +1. interaction with pod semantics (docker doesn't have a pod concept, but pod- +aware runtimes may perform additional moves/remounts while preparing a pod) + +These concerns are valid and we decide to limit the propagation mode to HostPath +volume only, in HostPath, we expect any runtime should NOT perform any additional +actions (such as clean up). This behavior is also consistent with current HostPath +logic: kube does not take care of the content in HostPath either. + <!-- BEGIN MUNGE: GENERATED_ANALYTICS --> []() <!-- END MUNGE: GENERATED_ANALYTICS --> |
