fix: update comments

Author: santhoshmprabhu

cns/restserver/nodesubnet.go

@@ -51,7 +51,7 @@ func (service *HTTPRestService) InitializeNodeSubnet(ctx context.Context, podInf
 		return errors.Wrap(err, "reconcile initial CNS state")
 	}
 	// statefile (if any) is reconciled. Initialize the IP fetcher. Start the IP fetcher only after the service is started,
-	// and any pending async delete operations are completed.
+	// because starting the IP fetcher will generate conflist, which should be done only once we are ready to respond to IPAM requests.
 	service.nodesubnetIPFetcher = nodesubnet.NewIPFetcher(service.nma, service, 0, 0, logger.Log)
 
 	return nil

cns/service/main.go

@@ -1046,8 +1046,9 @@ func main() {
 	}
 
 	if config.ChannelMode == cns.AzureHost {
-		// at this point, rest service is running, and any pending async deletes have been submitted to the rest
-		// service. We can now start serving new requests.
+		// at this point, rest service is running. We can now start serving new requests. So call StartNodeSubnet, which
+		// will fetch secondary IPs and generate conflist. Do not move this all before rest service start - this will cause
+		// CNI to start sending requests, and if the service doesn't start successfully, the requests will fail.
 		httpRemoteRestService.StartNodeSubnet(rootCtx)
 	}