If uid_attribute is explicitly set to an empty string, multipath should
log the uid at the default log level, since using the fallback code is
the expected behavior.

Signed-off-by: Benjamin Marzinski <[email protected]>
---
 libmultipath/discovery.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/libmultipath/discovery.c b/libmultipath/discovery.c
index f650534f..435f2639 100644
--- a/libmultipath/discovery.c
+++ b/libmultipath/discovery.c
@@ -2091,7 +2091,8 @@ get_uid (struct path * pp, int path_state, struct 
udev_device *udev,
                }
                if ((!udev_available || (len <= 0 && allow_fallback))
                    && has_uid_fallback(pp)) {
-                       used_fallback = 1;
+                       if (udev_available || !(udev && pp->uid_attribute))
+                               used_fallback = 1;
                        len = uid_fallback(pp, path_state, &origin);
                }
        }
-- 
2.17.2

--
dm-devel mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/dm-devel

Reply via email to