Goroutine creation can be simplified using WaitGroup.Go (modernize)

collectors/collectorManager.go

@@ -123,9 +123,7 @@ func (cm *collectorManager) Start() {
 	tick := make(chan time.Time)
 	cm.ticker.AddChannel(tick)
 
-	cm.wg.Add(1)
+	cm.wg.Go(func() {
-	go func() {
-		defer cm.wg.Done()
 		// Collector manager is done
 		done := func() {
 			// close all metric collectors
@@ -180,7 +178,7 @@ func (cm *collectorManager) Start() {
 				}
 			}
 		}
-	}()
+	})
 
 	// Collector manager is started
 	cclog.ComponentDebug("CollectorManager", "STARTED")

internal/metricRouter/metricCache.go

@@ -102,9 +102,7 @@ func (c *metricCache) Start() {
 		return oldPeriod
 	}
 
-	c.wg.Add(1)
+	c.wg.Go(func() {
-	go func() {
-		defer c.wg.Done()
 		for {
 			select {
 			case <-c.done:
@@ -124,7 +122,7 @@ func (c *metricCache) Start() {
 				}
 			}
 		}
-	}()
+	})
 	cclog.ComponentDebug("MetricCache", "START")
 }
 

internal/metricRouter/metricRouter.go

@@ -291,9 +291,7 @@ func (r *metricRouter) Start() {
 		r.cache.Start()
 	}
 
-	r.wg.Add(1)
+	r.wg.Go(func() {
-	go func() {
-		defer r.wg.Done()
 
 		for {
 			select {
@@ -324,7 +322,7 @@ func (r *metricRouter) Start() {
 				}
 			}
 		}
-	}()
+	})
 	cclog.ComponentDebug("MetricRouter", "STARTED")
 }