diff options
author | Benjamin Marzinski <bmarzins@redhat.com> | 2009-09-28 12:19:00 -0500 |
---|---|---|
committer | Christophe Varoqui <christophe.varoqui@free.fr> | 2009-10-02 22:10:31 +0200 |
commit | 60fcfb4a2c6a5223637929ef967bb3e39584f21e (patch) | |
tree | a170b507576a027ae70150b1109269d1a15436fd /libmultipath/configure.c | |
parent | aa0a885e1f19359c41b63151bfcface38ccca176 (diff) | |
download | multipath-tools-60fcfb4a2c6a5223637929ef967bb3e39584f21e.tar.gz multipath-tools-60fcfb4a2c6a5223637929ef967bb3e39584f21e.tar.bz2 multipath-tools-60fcfb4a2c6a5223637929ef967bb3e39584f21e.zip |
multipath-tools: Fix dry-run output
When multipath checks whether or not the multipath device needs to be renamed,
it only does the check if dry-run isn't selected. This means that you will
instead see all your renames as creates during a dry-run. The attached patch
fixes this.
Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com>
Diffstat (limited to 'libmultipath/configure.c')
-rw-r--r-- | libmultipath/configure.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/libmultipath/configure.c b/libmultipath/configure.c index ef7a6d4..9c549d0 100644 --- a/libmultipath/configure.c +++ b/libmultipath/configure.c @@ -151,7 +151,7 @@ select_action (struct multipath * mpp, vector curmp, int force_reload) if (!cmpp) { cmpp = find_mp_by_wwid(curmp, mpp->wwid); - if (cmpp && !conf->dry_run) { + if (cmpp) { condlog(2, "%s: rename %s to %s", mpp->wwid, cmpp->alias, mpp->alias); strncpy(mpp->alias_old, cmpp->alias, WWID_SIZE); |