From ce6706b0d5f634af4f44c40cac5c451e3e077e07 Mon Sep 17 00:00:00 2001 From: Cedric Charly Date: Sat, 3 Jun 2017 13:44:01 -0500 Subject: [PATCH] storage/memory: set default prometheus_reporting_interval When left blank in the config, the default prometheus_reporting_interval value defaulted to 0, causing a panic during peer store initialization. This change sets the default value to 1 if not provided. Fixes #319 --- storage/memory/peer_store.go | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/storage/memory/peer_store.go b/storage/memory/peer_store.go index 00f9e18..4419cd5 100644 --- a/storage/memory/peer_store.go +++ b/storage/memory/peer_store.go @@ -125,6 +125,10 @@ func New(cfg Config) (storage.PeerStore, error) { ps.wg.Add(1) go func() { defer ps.wg.Done() + if cfg.PrometheusReportingInterval <= 0 { + cfg.PrometheusReportingInterval = 1 + log.Warn("storage: PrometheusReportingInterval not specified/invalid, defaulting to 1 second") + } t := time.NewTicker(cfg.PrometheusReportingInterval) for { select {