diff options
author | Zdenek Kabelac <zkabelac@redhat.com> | 2011-10-11 08:57:13 +0000 |
---|---|---|
committer | Zdenek Kabelac <zkabelac@redhat.com> | 2011-10-11 08:57:13 +0000 |
commit | 4007ac814f019106d8ae47bb5fc2095537ea313c (patch) | |
tree | 63b269e9874aafeecfb0576e37034018bbcc3342 /lib/activate | |
parent | 409bf6e6d8b6cb87a858a80cb6dac82544e16e6c (diff) | |
download | lvm2-4007ac814f019106d8ae47bb5fc2095537ea313c.tar.gz lvm2-4007ac814f019106d8ae47bb5fc2095537ea313c.tar.xz lvm2-4007ac814f019106d8ae47bb5fc2095537ea313c.zip |
Change message severity
Using log_warn to report missing symlinks as warning, since the command
itself returns as successful, we should not produce log_error().
log_warn is better fit here.
Diffstat (limited to 'lib/activate')
-rw-r--r-- | lib/activate/dev_manager.c | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/lib/activate/dev_manager.c b/lib/activate/dev_manager.c index 9aa53b6c..87a2b61b 100644 --- a/lib/activate/dev_manager.c +++ b/lib/activate/dev_manager.c @@ -1886,7 +1886,7 @@ static int _tree_action(struct dev_manager *dm, struct logical_volume *lv, if (!dm_tree_deactivate_children(root, dlid, ID_LEN + sizeof(UUID_PREFIX) - 1)) goto_out; if (!_remove_lv_symlinks(dm, root)) - log_error("Failed to remove all device symlinks associated with %s.", lv->name); + log_warn("Failed to remove all device symlinks associated with %s.", lv->name); break; case SUSPEND: dm_tree_skip_lockfs(root); @@ -1913,10 +1913,8 @@ static int _tree_action(struct dev_manager *dm, struct logical_volume *lv, if (action == ACTIVATE) { if (!dm_tree_activate_children(root, dlid, ID_LEN + sizeof(UUID_PREFIX) - 1)) goto_out; - if (!_create_lv_symlinks(dm, root)) { - log_error("Failed to create symlinks for %s.", lv->name); - goto out; - } + if (!_create_lv_symlinks(dm, root)) + log_warn("Failed to create symlinks for %s.", lv->name); } break; |