cleaning up TODOs

Author: rjdenney

cns/restserver/ipusage.go

@@ -0,0 +1,58 @@
+package restserver
+
+import (
+	"github.com/Azure/azure-container-networking/cns/logger"
+	"github.com/Azure/azure-container-networking/cns/types"
+)
+
+type ipState struct {
+	// allocatedIPs are all the IPs given to CNS by DNC.
+	allocatedIPs int64
+	// assignedIPs are the IPs CNS gives to Pods.
+	assignedIPs int64
+	// availableIPs are the IPs in state "Available".
+	availableIPs int64
+	// programmingIPs are the IPs in state "PendingProgramming".
+	programmingIPs int64
+	// releasingIPs are the IPs in state "PendingReleasr".
+	releasingIPs int64
+}
+
+func (service *HTTPRestService) buildIPState() *ipState {
+	service.Lock()
+	defer service.Unlock()
+
+	state := ipState{
+		allocatedIPs:   0,
+		assignedIPs:    0,
+		availableIPs:   0,
+		programmingIPs: 0,
+		releasingIPs:   0,
+	}
+
+	//nolint:gocritic // This has to iterate over the IP Config state to get the counts.
+	for _, ipConfig := range service.PodIPConfigState {
+		state.allocatedIPs++
+		if ipConfig.GetState() == types.Assigned {
+			state.assignedIPs++
+		}
+		if ipConfig.GetState() == types.Available {
+			state.availableIPs++
+		}
+		if ipConfig.GetState() == types.PendingProgramming {
+			state.programmingIPs++
+		}
+		if ipConfig.GetState() == types.PendingRelease {
+			state.releasingIPs++
+		}
+	}
+
+	logger.Printf("[IP Usage] Allocated IPs: %d, Assigned IPs: %d, Available IPs: %d, PendingProgramming IPs: %d, PendingRelease IPs: %d",
+		state.allocatedIPs,
+		state.assignedIPs,
+		state.availableIPs,
+		state.programmingIPs,
+		state.releasingIPs,
+	)
+	return &state
+}

cns/restserver/restserver.go

@@ -54,7 +54,7 @@ type imdsClient interface {
 	GetVMUniqueID(ctx context.Context) (string, error)
 }
 
-// HTTPRestService represents http listener for CNS - Container Networking Service.// TODO: Add a new value for IPFamily
+// 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