From 1a6a699725321d2b8f0f7daadefc2872e8d7550a Mon Sep 17 00:00:00 2001
From: capossele <angelocapossele@gmail.com>
Date: Thu, 18 Jun 2020 18:39:42 +0100
Subject: [PATCH] =?UTF-8?q?=F0=9F=9A=A8=20Fix=20metrics=20plugin=20linter?=
 =?UTF-8?q?=20warning?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 plugins/metrics/autopeering.go  |  4 ++++
 plugins/metrics/clients_info.go |  1 +
 plugins/metrics/db.go           |  1 +
 plugins/metrics/network.go      |  5 +++++
 plugins/metrics/parameters.go   | 18 ++++++++++++++----
 plugins/metrics/process.go      |  4 ++--
 plugins/prometheus/process.go   |  2 +-
 7 files changed, 28 insertions(+), 7 deletions(-)

diff --git a/plugins/metrics/autopeering.go b/plugins/metrics/autopeering.go
index 36795f4b..a94ee57e 100644
--- a/plugins/metrics/autopeering.go
+++ b/plugins/metrics/autopeering.go
@@ -44,12 +44,14 @@ var (
 	})
 )
 
+// NeighborDropCount returns the neighbor drop count.
 func NeighborDropCount() uint64 {
 	neighborMutex.RLock()
 	defer neighborMutex.RUnlock()
 	return neighborDropCount
 }
 
+// AvgNeighborConnectionLifeTime return the average neighbor connection lifetime.
 func AvgNeighborConnectionLifeTime() float64 {
 	neighborMutex.RLock()
 	defer neighborMutex.RUnlock()
@@ -59,12 +61,14 @@ func AvgNeighborConnectionLifeTime() float64 {
 	return float64(neighborConnectionsLifeTime.Milliseconds()) / float64(neighborDropCount)
 }
 
+// ConnectionsCount returns the neighbors connections count.
 func ConnectionsCount() uint64 {
 	distanceMutex.RLock()
 	defer distanceMutex.RUnlock()
 	return connectionsCount
 }
 
+// AutopeeringDistanceStats returns statistics of the autopeering distance function.
 func AutopeeringDistanceStats() (min, max uint64, avg float64) {
 	distanceMutex.RLock()
 	defer distanceMutex.RUnlock()
diff --git a/plugins/metrics/clients_info.go b/plugins/metrics/clients_info.go
index b7b792bf..14693cba 100644
--- a/plugins/metrics/clients_info.go
+++ b/plugins/metrics/clients_info.go
@@ -10,6 +10,7 @@ import (
 	"go.uber.org/atomic"
 )
 
+// ClientInfo holds info of a client.
 type ClientInfo struct {
 	OS          string
 	Arch        string
diff --git a/plugins/metrics/db.go b/plugins/metrics/db.go
index 78a55a04..256914f0 100644
--- a/plugins/metrics/db.go
+++ b/plugins/metrics/db.go
@@ -19,6 +19,7 @@ var (
 	})
 )
 
+// DBSize returns the size of the db.
 func DBSize() uint64 {
 	return dbSize.Load()
 }
diff --git a/plugins/metrics/network.go b/plugins/metrics/network.go
index 9ca96d22..3a741d5b 100644
--- a/plugins/metrics/network.go
+++ b/plugins/metrics/network.go
@@ -19,22 +19,27 @@ var (
 	analysisOutboundBytes atomic.Uint64
 )
 
+// FPCInboundBytes returns the total inbound FPC traffic.
 func FPCInboundBytes() uint64 {
 	return _FPCInboundBytes.Load()
 }
 
+// FPCOutboundBytes returns the total outbound FPC traffic.
 func FPCOutboundBytes() uint64 {
 	return _FPCOutboundBytes.Load()
 }
 
+// GossipInboundBytes returns the total inbound gossip traffic.
 func GossipInboundBytes() uint64 {
 	return gossipCurrentRx.Load()
 }
 
+// GossipOutboundBytes returns the total outbound gossip traffic.
 func GossipOutboundBytes() uint64 {
 	return gossipCurrentTx.Load()
 }
 
+// AnalysisOutboundBytes returns the total outbound analysis traffic.
 func AnalysisOutboundBytes() uint64 {
 	return analysisOutboundBytes.Load()
 }
diff --git a/plugins/metrics/parameters.go b/plugins/metrics/parameters.go
index b504f77e..b26d64f0 100644
--- a/plugins/metrics/parameters.go
+++ b/plugins/metrics/parameters.go
@@ -8,14 +8,24 @@ import (
 
 const (
 	// should always be 1 second
+
+	// MPSMeasurementInterval defines the MPS measurement interval
 	MPSMeasurementInterval = 1 * time.Second
+	// TPSMeasurementInterval defines the TPS measurement interval
 	TPSMeasurementInterval = 1 * time.Second
+
 	// can be adjusted as wished
+
+	// MessageTipsMeasurementInterval defines the Communication-layer tips measurement interval
 	MessageTipsMeasurementInterval = 1 * time.Second
-	ValueTipsMeasurementInterval   = 1 * time.Second
-	CPUUsageMeasurementInterval    = 1 * time.Second
-	MemUsageMeasurementInterval    = 1 * time.Second
-	SyncedMeasurementInterval      = 1 * time.Second
+	// ValueTipsMeasurementInterval defines the Value-layer tips measurement interval
+	ValueTipsMeasurementInterval = 1 * time.Second
+	// CPUUsageMeasurementInterval defines the CPU usage measurement interval
+	CPUUsageMeasurementInterval = 1 * time.Second
+	// MemUsageMeasurementInterval defines the mem usage measurement interval
+	MemUsageMeasurementInterval = 1 * time.Second
+	// SyncedMeasurementInterval defines the synced measurement interval
+	SyncedMeasurementInterval = 1 * time.Second
 )
 
 const (
diff --git a/plugins/metrics/process.go b/plugins/metrics/process.go
index 10dc4423..4d448be2 100644
--- a/plugins/metrics/process.go
+++ b/plugins/metrics/process.go
@@ -14,8 +14,8 @@ var (
 	memUsageBytes atomic.Uint64
 )
 
-// CpuUsage returns the current cpu usage
-func CpuUsage() float64 {
+// CPUUsage returns the current cpu usage
+func CPUUsage() float64 {
 	return cpuUsage.Load()
 }
 
diff --git a/plugins/prometheus/process.go b/plugins/prometheus/process.go
index c38a4338..46f5adbf 100644
--- a/plugins/prometheus/process.go
+++ b/plugins/prometheus/process.go
@@ -27,6 +27,6 @@ func registerProcessMetrics() {
 }
 
 func collectProcesskMetrics() {
-	cpuUsage.Set(float64(metrics.CpuUsage()))
+	cpuUsage.Set(float64(metrics.CPUUsage()))
 	memUsageBytes.Set(float64(metrics.MemUsage()))
 }
-- 
GitLab