addressing pr comments

api/health/service.go

@@ -29,12 +29,12 @@ type Service interface {
 }
 
 func NewService(checkFreq time.Duration, log logging.Logger, registry prometheus.Registerer) (Service, error) {
-	healthL, err := healthlib.NewService(checkFreq, log, registry)
+	service, err := healthlib.NewService(checkFreq, log, registry)
 	if err != nil {
 		return nil, err
 	}
 	return &apiServer{
-		Service: healthL,
+		Service: service,
 		log:     log,
 	}, nil
 }

health/metrics.go

@@ -7,8 +7,6 @@ import (
 	"github.com/ava-labs/avalanchego/utils/logging"
 	"github.com/ava-labs/avalanchego/utils/wrappers"
 	"github.com/prometheus/client_golang/prometheus"
-
-	health "github.com/AppsFlyer/go-sundheit"
 )
 
 // metrics reports commonly used health check metrics.
@@ -39,11 +37,11 @@ func newMetrics(log logging.Logger, namespace string, registerer prometheus.Regi
 }
 
 // healthy handles the metrics for the healthy cases
-func (m *metrics) healthy(health.Result) {
+func (m *metrics) healthy() {
 	m.failingChecks.Set(0)
 }
 
 // unHealthy handles the metrics for the unhealthy cases
-func (m *metrics) unHealthy(health.Result) {
+func (m *metrics) unHealthy() {
 	m.failingChecks.Inc()
 }

health/service.go

@@ -111,19 +111,19 @@ func (c *checkListener) OnCheckCompleted(name string, result health.Result) {
 	if !exists || isHealthy == previouslyHealthy {
 		if isHealthy {
 			c.log.Debug("%q returned healthy with: %s", name, string(resultJSON))
-			c.metrics.healthy(result)
+			c.metrics.healthy()
 		} else {
 			c.log.Debug("%q returned unhealthy with: %s", name, string(resultJSON))
-			c.metrics.unHealthy(result)
+			c.metrics.unHealthy()
 		}
 		return
 	}
 
 	if isHealthy {
 		c.log.Info("%q became healthy with: %s", name, string(resultJSON))
-		c.metrics.healthy(result)
+		c.metrics.healthy()
 	} else {
 		c.log.Warn("%q became unhealthy with: %s", name, string(resultJSON))
-		c.metrics.unHealthy(result)
+		c.metrics.unHealthy()
 	}
 }

network/peer.go

@@ -487,7 +487,7 @@ func (p *peer) GetPeerList() {
 	p.net.log.AssertNoError(err)
 	p.Send(msg)
 	p.net.getPeerlist.numReceived.Inc()
-	p.net.getPeerlist.sentBytes.Add(float64(len(msg.Bytes())))
+	p.net.getPeerlist.receivedBytes.Add(float64(len(msg.Bytes())))
 }
 
 // assumes the stateLock is not held