Skip to content

Conversation

@tamilmani1989
Copy link
Member

…pace is deleted

Reason for Change:

The issue is that in transparent mode cni generates the veth pair names based on pod name and namespace. In this case, a new container with the same pod name and namespace called CNI ADD before the old container with the same pod name and namespace call CNI DEL. Due to this, when the old container issued CNI DEL, it deleted the veth host interface, in turn deleting the eth0(other veth) of the new container.

Issue Fixed:

Requirements:

Notes:

@codecov
Copy link

codecov bot commented Jan 29, 2021

Codecov Report

Merging #775 (ace525a) into master (c133472) will increase coverage by 0.07%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #775      +/-   ##
==========================================
+ Coverage   41.68%   41.76%   +0.07%     
==========================================
  Files         142      142              
  Lines       13778    13773       -5     
==========================================
+ Hits         5744     5752       +8     
+ Misses       7317     7303      -14     
- Partials      717      718       +1     

@ashvindeodhar ashvindeodhar self-requested a review February 4, 2021 05:29
@ashvindeodhar
Copy link
Member

/lgtm

@tamilmani1989 tamilmani1989 merged commit 8307081 into Azure:master Feb 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants