cybozu-go / coil

CNI plugin for Kubernetes designed for scalability and extensibility
Apache License 2.0
165 stars 20 forks source link

Fix the bug that Coil accidentally deletes a live peer #227

Closed ysksuzuki closed 2 years ago

ysksuzuki commented 2 years ago

This PR fixes the problem that Coil accidentally deletes a live peer in the case as follows.

  1. A job pod is scheduled on a node, runs, and is terminated eventually.
  2. Another pod is scheduled on the same node, assigned the same IP the job pod was using and runs.
  3. The job pod is deleted.
  4. Coil deletes another pod's peer that is still alive.
terassyi commented 2 years ago

LGTM!