fix: addressing the comments

Author: behzad-mir

network/network_windows.go

@@ -45,6 +45,9 @@ const (
 	defaultIPv6NextHop = "fe80::1234:5678:9abc"
 )
 
+var errHcnNetworkCreate = errors.New("Failed to create hcn network")
+var errHcnNetworkGet = errors.New("Failed to get hcn network")
+
 // Windows implementation of route.
 type route interface{}
 
@@ -365,18 +368,19 @@ func (nm *networkManager) newNetworkImplHnsV2(nwInfo *EndpointInfo, extIf *exter
 			} else {
 				if strings.Contains(err.Error(), "already exists") {
 					// fetch the network name again since the parallel CNI Add call has created the HNS network
+					logger.Error("Failed to create hcn network.due to error:", zap.String("hnsResponse", hcnNetwork.Name), zap.String("hnsResponse", err.Error()))
 					hnsResponse, err = Hnsv2.GetNetworkByName(hcnNetwork.Name)
 					if err != nil {
-						return nil, fmt.Errorf("Failed to get hcn network: %s due to error: %v", hcnNetwork.Name, err)
+						return nil, fmt.Errorf("%s: %s due to error %s", errHcnNetworkGet, hcnNetwork.Name, err.Error())
 					}
-					logger.Info("Successfully fetched hcn network with response", zap.Any("hnsResponse", hnsResponse))
+					logger.Info("Successfully fetched hcn network with response", zap.Any("hnsResponse", hnsResponse.Id))
 				} else {
-					return nil, fmt.Errorf("Failed to create hcn network: %s due to error: %v", hcnNetwork.Name, err)
+					return nil, fmt.Errorf("%s: %s due to error %s", errHcnNetworkCreate, hcnNetwork.Name, err.Error())
 				}
 			}
 		} else {
 			// we can't validate if the network already exists, don't continue
-			return nil, fmt.Errorf("Failed to create hcn network: %s, failed to query for existing network with error: %v", hcnNetwork.Name, err)
+			return nil, fmt.Errorf("%s: %s due to error %s", errHcnNetworkCreate, hcnNetwork.Name, err.Error())
 		}
 	} else {
 		if hcnNetwork.Type == hcn.Transparent {