Skip to content

Fix IPAM Leak  #633

@neaggarwMS

Description

@neaggarwMS

Details are found in ICM# https://icm.ad.msft.net/imp/v3/incidents/details/194727242/home

Fix:
Ensure even in failure cases, we release the pool when the IP is not in use.

Solution:
To accomplish that, we will be adding the container ID to the IPAM state file. Thereby if CNI ADD fails, it will still be able to delete any dangling state in the initial ADD call during the subsequent DELETE.

Also, add more calls to release pool/address in the CNI delete call. Such as if Endpoint is not found, network is not found etc.

Metadata

Metadata

Assignees

Labels

staleStale due to inactivity.

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions