fix for IPAM leak issue

Author: g1rana

mgmtfn/dockplugin/ipamDriver.go

@@ -288,6 +288,19 @@ func releaseAddress(w http.ResponseWriter, r *http.Request) {
 
 	log.Infof("Received ReleaseAddressRequest: %+v", areq)
 
+	//Build an release request to be sent to master
+	releaseReq := master.AddressReleaseRequest{
+		NetworkID:   areq.PoolID,
+		IPv4Address: areq.Address,
+	}
+	var releaseResp master.AddressReleaseResponse
+	err := cluster.MasterPostReq("/plugin/releaseAddress",
+		&releaseReq, &releaseRsp)
+	if err != nil {
+		httpError(w, "master failed to release request:%v address",
+			releaseReq, err)
+		return
+	}
 	// response
 	relResp := api.ReleaseAddressResponse{}
 

mgmtfn/dockplugin/netDriver.go

@@ -189,7 +189,7 @@ func createEndpoint(hostname string) func(http.ResponseWriter, *http.Request) {
 			return
 		}
 
-		log.Infof("CreateEndpointRequest: %+v. Interface: %+v", cereq, cereq.Interface)
+		//log.Infof("CreateEndpointRequest: %+v. Interface: %+v", cereq, cereq.Interface)
 
 		tenantName, netName, serviceName, err := GetDockerNetworkName(cereq.NetworkID)
 		if err != nil {
@@ -213,6 +213,8 @@ func createEndpoint(hostname string) func(http.ResponseWriter, *http.Request) {
 			},
 		}
 
+		log.Infof("CreateEndpointRequest: %+v. Interface: %+v", mreq, cereq.Interface)
+
 		var mresp master.CreateEndpointResponse
 		err = cluster.MasterPostReq("/plugin/createEndpoint", &mreq, &mresp)
 		if err != nil {

netmaster/master/api.go

@@ -49,6 +49,9 @@ type AddressReleaseRequest struct {
 	NetworkID   string // Unique identifier for the network
 	IPv4Address string // Allocated address
 }
+type AddressReleaseResponse struct {
+	Status string
+}
 
 // CreateEndpointRequest has the endpoint create request from netplugin
 type CreateEndpointRequest struct {

netplugin/cluster/cluster.go

@@ -123,7 +123,8 @@ func masterReq(path string, req interface{}, resp interface{}, isDel bool) error
 				time.Sleep(time.Second)
 				continue
 			} else if err != nil {
-				log.Errorf("Error making %s request: Err: %v", reqType, err)
+				log.Errorf("Error making %s request: Err: %v"+
+					"with resp:%+v", reqType, err, resp)
 				return err
 			}