From d3a4dcffef6dac0e78c83e8e3f1d7410c49adf86 Mon Sep 17 00:00:00 2001
From: Dan Laine <daniel.laine@avalabs.org>
Date: Tue, 30 Jun 2020 16:51:41 -0400
Subject: [PATCH] add comment; handle error on health check registration

---
 node/node.go | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/node/node.go b/node/node.go
index f05edc4..1d41e27 100644
--- a/node/node.go
+++ b/node/node.go
@@ -507,7 +507,9 @@ func (n *Node) initHealthAPI() error {
 	}
 	n.Log.Info("initializing Health API")
 	service := health.NewService(n.Log)
-	service.RegisterHeartbeat("network.validators.heartbeat", n.Net, 5*time.Minute)
+	if err := service.RegisterHeartbeat("network.validators.heartbeat", n.Net, 5*time.Minute); err != nil {
+		return fmt.Errorf("couldn't register heartbeat health check: %w", err)
+	}
 	isBootstrappedFunc := func() (interface{}, error) {
 		pChainID, err := n.chainManager.Lookup("P")
 		if err != nil {
@@ -529,6 +531,7 @@ func (n *Node) initHealthAPI() error {
 		}
 		return nil, nil
 	}
+	// Passes if the P, X and C chains are finished bootstrapping
 	if err := service.RegisterMonotonicCheckFunc("defaultChainsBootstrapped", isBootstrappedFunc); err != nil {
 		return err
 	}