fixing comments

Author: rjdenney

cns/middlewares/k8sSwiftV2.go

@@ -100,7 +100,6 @@ func (k *K8sSWIFTv2Middleware) IPConfigsRequestHandlerWrapper(defaultHandler, fa
 	}
 }
 
-// For our purposes we would skip over this logic or need to replace it with something to check the delegated NIC
 // validateIPConfigsRequest validates if pod is multitenant by checking the pod labels, used in SWIFT V2 AKS scenario.
 // nolint
 func (k *K8sSWIFTv2Middleware) validateIPConfigsRequest(ctx context.Context, req *cns.IPConfigsRequest) (podInfo cns.PodInfo, respCode types.ResponseCode, message string) {

cns/restserver/ipam.go

@@ -998,12 +998,14 @@ func (service *HTTPRestService) AssignAvailableIPConfigs(podInfo cns.PodInfo) ([
 		return nil, ErrNoNCs
 	}
 
+	// Gets the IPFamilies from all NCs and stores them in a map. This will be ued to determine the amount of IPs to return
 	for ncID := range service.state.ContainerStatus {
 		for ipFamily := range service.state.ContainerStatus[ncID].CreateNetworkContainerRequest.IPFamilies {
 			ipFamilies[ipFamily] = struct{}{}
 		}
 	}
 
+	// Makes sure we have at least one IPFamily across all NCs
 	numOfIPFamilies := len(ipFamilies)
 	if numOfIPFamilies == 0 {
 		return nil, ErrNoIPFamilies

cns/restserver/restserver.go

@@ -55,7 +55,6 @@ type imdsClient interface {
 }
 
 // HTTPRestService represents http listener for CNS - Container Networking Service.
-// TODO: If we add a new type of Middleware that will be reflected in the IPConfigsHandlerMiddleware value
 type HTTPRestService struct {
 	*cns.Service
 	dockerClient             *dockerclient.Client