[lvm-devel] master - activation: switch to warning

Zdenek Kabelac zkabelac at fedoraproject.org
Thu Jun 23 13:01:26 UTC 2016


Gitweb:        http://git.fedorahosted.org/git/?p=lvm2.git;a=commitdiff;h=af36f5ad369ebf4c1d768145a827436ccef22f63
Commit:        af36f5ad369ebf4c1d768145a827436ccef22f63
Parent:        751163a74375cc3bfc9f338f1317f14a23798812
Author:        Zdenek Kabelac <zkabelac at redhat.com>
AuthorDate:    Tue Jun 21 15:16:03 2016 +0200
Committer:     Zdenek Kabelac <zkabelac at redhat.com>
CommitterDate: Thu Jun 23 14:53:59 2016 +0200

activation: switch to warning

Since this function does not have a way to return error use correct
reporting level as warning.
---
 WHATS_NEW               |    1 +
 lib/activate/activate.c |    4 ++--
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/WHATS_NEW b/WHATS_NEW
index ce77b6c..6788e6a 100644
--- a/WHATS_NEW
+++ b/WHATS_NEW
@@ -1,5 +1,6 @@
 Version 2.02.158 - 
 =================================
+  Switch to log_warn() for failing activation status query.
   Refactor lvconvert argument handling code.
   Add --logonly option to report only cmd log for a command, not other reports.
   Add log/command_log_selection to configure default selection used on cmd log.
diff --git a/lib/activate/activate.c b/lib/activate/activate.c
index 089af53..8892a2a 100644
--- a/lib/activate/activate.c
+++ b/lib/activate/activate.c
@@ -1454,9 +1454,9 @@ static int _lv_is_active(const struct logical_volume *lv,
 	 * Old users of this function will never be affected by this,
 	 * since they are only concerned about active vs. not active.
 	 * New users of this function who specifically ask for 'exclusive'
-	 * will be given an error message.
+	 * will be given a warning message.
 	 */
-	log_error("Unable to determine exclusivity of %s.", display_lvname(lv));
+	log_warn("WARNING: Unable to determine exclusivity of %s.", display_lvname(lv));
 
 	e = 0;
 




More information about the lvm-devel mailing list