Apply suggestions from code review

Co-authored-by: Copilot <[email protected]>
Signed-off-by: Paul Miller <[email protected]>

Author: paulgmiller

cns/restserver/synchostnc.go

@@ -208,7 +208,7 @@ func (service *HTTPRestService) mustGenerateCNIConflistOnce() {
 }
 
 func (service *HTTPRestService) WaitForConfList(ctx context.Context) {
-	//sync loop never set up get out of here.
+	// Sync loop never set up, get out of here.
 	if !service.ncSyncLoop.Load() {
 		return
 	}

cns/service/main.go

@@ -1118,10 +1118,10 @@ func main() {
 		httpRemoteRestService.StartNodeSubnet(rootCtx)
 	}
 
-	//we wrote out the conflist right?
+	// Wait for NC sync to complete before marking service as ready.
 	httpRemoteRestService.WaitForConfList(rootCtx)
 	// mark the service as "ready"
-	close(readyCh) //will still block if root ctx is canceled?
+	close(readyCh) // This will not block, even if rootCtx is canceled.
 	// block until process exiting
 	<-rootCtx.Done()