Skip to content

Commit

Permalink
cilium-docker: removed duplicate DelLinkByName
Browse files Browse the repository at this point in the history
Since docker calls leaveEndpoint first and then deleteEndpoint we don't
need to call DelLinkByName twice.

Signed-off-by: André Martins <andre@cilium.io>
  • Loading branch information
aanm authored and tgraf committed Mar 22, 2017
1 parent c501d6a commit 81b80a4
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions plugins/cilium-docker/driver/driver.go
Original file line number Diff line number Diff line change
Expand Up @@ -456,8 +456,5 @@ func (driver *driver) leaveEndpoint(w http.ResponseWriter, r *http.Request) {
log.Warningf("Leaving the endpoint failed: %s", err)
}

if err := plugins.DelLinkByName(plugins.Endpoint2IfName(l.EndpointID)); err != nil {
log.Warningf("Error while deleting link: %s", err)
}
emptyResponse(w)
}

0 comments on commit 81b80a4

Please sign in to comment.