Re: [dm-devel] [PATCH 23/57] multipath: remove warning 'failed to get wwid'

2016-05-02 Thread Hannes Reinecke 
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
-- 
Dr. Hannes Reinecke   zSeries & Storage
h...@suse.de  +49 911 74053 688
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg
GF: J. Hawn, J. Guild, F. Imendörffer, HRB 16746 (AG Nürnberg)

--
dm-devel mailing list
dm-devel@redhat.com
https://www.redhat.com/mailman/listinfo/dm-devel

Re: [dm-devel] [PATCH 23/57] multipath: remove warning 'failed to get wwid'

2016-05-02 Thread Benjamin Marzinski 
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 
> ---
>  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

[dm-devel] [PATCH 23/57] multipath: remove warning 'failed to get wwid'

2016-04-27 Thread Hannes Reinecke 
Signed-off-by: Hannes Reinecke 
---
 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