diff mbox

[23/57] multipath: remove warning 'failed to get wwid'

Message ID 1461755458-29225-24-git-send-email-hare@suse.de (mailing list archive)
State Not Applicable, archived
Headers show

Commit Message

Hannes Reinecke April 27, 2016, 11:10 a.m. UTC 
Signed-off-by: Hannes Reinecke <hare@suse.com>
---
 multipath/main.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Benjamin Marzinski May 2, 2016, 3:43 p.m. UTC | #1 
On Wed, Apr 27, 2016 at 01:10:24PM +0200, Hannes Reinecke wrote:

Again. I'd prefer leaving the messages in either at a higher log level
or with less alarming text.

-Ben

> Signed-off-by: Hannes Reinecke <hare@suse.com>
> ---
>  multipath/main.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/multipath/main.c b/multipath/main.c
> index d14a913..da8fa23 100644
> --- a/multipath/main.c
> +++ b/multipath/main.c
> @@ -284,7 +284,6 @@ configure (void)
>  		int failed = get_refwwid(conf->dev, conf->dev_type, pathvec,
>  					 &refwwid);
>  		if (!refwwid) {
> -			condlog(3, "%s: failed to get wwid", conf->dev);
>  			if (failed == 2 && conf->cmd == CMD_VALID_PATH)
>  				printf("%s is not a valid multipath device path\n", conf->dev);
>  			else
> -- 
> 2.6.6

--
dm-devel mailing list
dm-devel@redhat.com
https://www.redhat.com/mailman/listinfo/dm-devel
Hannes Reinecke May 3, 2016, 5:48 a.m. UTC | #2 
On 05/02/2016 05:43 PM, Benjamin Marzinski wrote:
> On Wed, Apr 27, 2016 at 01:10:24PM +0200, Hannes Reinecke wrote:
> 
> Again. I'd prefer leaving the messages in either at a higher log level
> or with less alarming text.
> 
I'm fine with moving it to log level 4.

Cheers,

Hannes
diff mbox

Patch

diff --git a/multipath/main.c b/multipath/main.c
index d14a913..da8fa23 100644
--- a/multipath/main.c
+++ b/multipath/main.c
@@ -284,7 +284,6 @@  configure (void)
 		int failed = get_refwwid(conf->dev, conf->dev_type, pathvec,
 					 &refwwid);
 		if (!refwwid) {
-			condlog(3, "%s: failed to get wwid", conf->dev);
 			if (failed == 2 && conf->cmd == CMD_VALID_PATH)
 				printf("%s is not a valid multipath device path\n", conf->dev);
 			else