cybozu-go / coil

CNI plugin for Kubernetes designed for scalability and extensibility
Apache License 2.0
158 stars 18 forks source link

Remove sleep from CNI DEL implementation #215

Closed ysksuzuki closed 2 years ago

ysksuzuki commented 2 years ago

Currently, Coil sleeps 30s in its CNI DEL implementation for the graceful termination for pods. However, we recently discovered that kubelet calls the StopSandbox API of the container runtime after it stops container processes. It means container processes aren't running when the container runtime executes the CNI DEL. Therefore, this commit removes sleep from the CNI DEL implementation because it's no longer necessary, as described above.

Signed-off-by: Yusuke Suzuki yusuke-suzuki@cybozu.co.jp