From d7a5c549d5a67c24fea892e6eaf23889546bcb77 Mon Sep 17 00:00:00 2001
From: olgashtivelman <33713489+olgashtivelman@users.noreply.github.com>
Date: Mon, 9 Jul 2018 15:12:56 +0300
Subject: [PATCH] =?UTF-8?q?UB-1098:=20change=20logging=20message=20to=20wa?=
 =?UTF-8?q?rning=20instead=20of=20info=20for=20idempo=E2=80=A6=20(#222)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

* UB-1098: change logging message to warning instead of info for idempotent issues*
---
 .../block_device_mounter_utils/block_device_mounter_utils.go    | 2 +-
 remote/mounter/block_device_utils/mpath.go                      | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/remote/mounter/block_device_mounter_utils/block_device_mounter_utils.go b/remote/mounter/block_device_mounter_utils/block_device_mounter_utils.go
index 92623422..81db91e0 100644
--- a/remote/mounter/block_device_mounter_utils/block_device_mounter_utils.go
+++ b/remote/mounter/block_device_mounter_utils/block_device_mounter_utils.go
@@ -82,7 +82,7 @@ func (b *blockDeviceMounterUtils) MountDeviceFlow(devicePath string, fsType stri
 	if isMounted {
 		for _, mountpointi := range mountpointRefs {
 			if mountpointi == mountPoint {
-				b.logger.Info(WarningMessageIdempotentDeviceAlreadyMounted, logs.Args{{"Device", devicePath}, {"mountpoint", mountPoint}})
+				b.logger.Warning(WarningMessageIdempotentDeviceAlreadyMounted, logs.Args{{"Device", devicePath}, {"mountpoint", mountPoint}})
 				return nil // Indicate idempotent issue
 			}
 		}
diff --git a/remote/mounter/block_device_utils/mpath.go b/remote/mounter/block_device_utils/mpath.go
index 69fe230f..8e14bf1d 100644
--- a/remote/mounter/block_device_utils/mpath.go
+++ b/remote/mounter/block_device_utils/mpath.go
@@ -75,7 +75,7 @@ func (b *blockDeviceUtils) Discover(volumeWwn string, deepDiscovery bool) (strin
 			b.logger.Debug(fmt.Sprintf("mpath device was NOT found for WWN [%s] even after sg_inq on all mpath devices.", volumeWwn))
 			return "", b.logger.ErrorRet(&volumeNotFoundError{volumeWwn}, "failed")
 		} else {
-			b.logger.Info(fmt.Sprintf("Warning: device [%s] found for WWN [%s] after running sg_inq on all mpath devices although it was not found in multipath -ll. (Note: Could indicate multipathing issue).", dev, volumeWwn))
+			b.logger.Warning(fmt.Sprintf("device [%s] found for WWN [%s] after running sg_inq on all mpath devices although it was not found in multipath -ll. (Note: Could indicate multipathing issue).", dev, volumeWwn))
 			mpath = b.mpathDevFullPath(dev)
 		}
 	} else {