aws / amazon-vpc-cni-plugins

VPC CNI plugins for Amazon ECS and EKS.
Apache License 2.0
68 stars 45 forks source link

Add missing logging in branch and tunnel plugins #70

Closed samjkon closed 2 years ago

samjkon commented 2 years ago

A specific error returned while deleting resources inside a network namespace was not logged. Added the log to improve visibility.

Description of changes: The branch and tunnel plugins executes a function inside a network namespace as part of their DEL workflow. Sometimes we noticed there are chances that execution of that function inside the network namespace fail due to unknown reasons. A certain error in the workflow was noticed to be not logged. This change ensure that error is logged so that we will have better visibility.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.