do not return err

snow/uptime/manager.go

@@ -4,7 +4,6 @@
 package uptime
 
 import (
-	"errors"
 	"time"
 
 	"github.com/ava-labs/avalanchego/database"
@@ -14,11 +13,6 @@ import (
 
 var _ Manager = (*manager)(nil)
 
-var (
-	errAlreadyStartedTracking = errors.New("already started tracking")
-	errNotStartedTracking     = errors.New("not started tracking")
-)
-
 type Manager interface {
 	Tracker
 	Calculator
@@ -62,7 +56,7 @@ func NewManager(state State, clk *mockable.Clock) Manager {
 
 func (m *manager) StartTracking(nodeIDs []ids.NodeID) error {
 	if m.startedTracking {
-		return errAlreadyStartedTracking
+		return nil
 	}
 	now := m.clock.UnixTime()
 	for _, nodeID := range nodeIDs {
@@ -87,7 +81,7 @@ func (m *manager) StartTracking(nodeIDs []ids.NodeID) error {
 
 func (m *manager) StopTracking(nodeIDs []ids.NodeID) error {
 	if !m.startedTracking {
-		return errNotStartedTracking
+		return nil
 	}
 	defer func() {
 		m.startedTracking = false