From 873f1fb87cdbf19fc332ee7f302e32ab98143242 Mon Sep 17 00:00:00 2001
From: Simon Pasquier <spasquie@redhat.com>
Date: Tue, 26 Feb 2019 16:41:50 +0100
Subject: [PATCH] config: update coordinator's logs

Signed-off-by: Simon Pasquier <spasquie@redhat.com>
---
 config/coordinator.go | 22 ++++++++++------------
 1 file changed, 10 insertions(+), 12 deletions(-)

diff --git a/config/coordinator.go b/config/coordinator.go
index 35c2049a..04e4d3f4 100644
--- a/config/coordinator.go
+++ b/config/coordinator.go
@@ -95,19 +95,9 @@ func (c *Coordinator) notifySubscribers() error {
 
 // loadFromFile triggers a configuration load, discarding the old configuration.
 func (c *Coordinator) loadFromFile() error {
-	level.Info(c.logger).Log(
-		"msg", "Loading configuration file",
-		"file", c.configFilePath,
-	)
-
 	conf, plainConfig, err := LoadFile(c.configFilePath)
 	if err != nil {
 		c.configSuccessMetric.Set(0)
-		level.Error(c.logger).Log(
-			"msg", "Loading configuration file failed",
-			"file", c.configFilePath,
-			"err", err,
-		)
 		return err
 	}
 
@@ -126,14 +116,22 @@ func (c *Coordinator) Reload() error {
 	c.mutex.Lock()
 	defer c.mutex.Unlock()
 
+	level.Info(c.logger).Log(
+		"msg", "Loading configuration file",
+		"file", c.configFilePath,
+	)
 	if err := c.loadFromFile(); err != nil {
-		c.logger.Log(
-			"msg", "loading configuration file failed",
+		level.Error(c.logger).Log(
+			"msg", "Loading configuration file failed",
 			"file", c.configFilePath,
 			"err", err,
 		)
 		return err
 	}
+	level.Info(c.logger).Log(
+		"msg", "Completed loading of configuration file",
+		"file", c.configFilePath,
+	)
 
 	if err := c.notifySubscribers(); err != nil {
 		c.logger.Log(