diff options
author | David Teigland <teigland@redhat.com> | 2019-06-21 13:37:11 -0500 |
---|---|---|
committer | David Teigland <teigland@redhat.com> | 2019-06-25 15:39:08 -0500 |
commit | b4402bd821723067c2becaf8e2a0452d4896cfd7 (patch) | |
tree | 1627d22c3654a09f9adb4aea3b07b295cbb34c53 /tools/pvresize.c | |
parent | d16142f90fdcf2aef42a51ecabd0c4ff11733d7c (diff) | |
download | lvm2-b4402bd821723067c2becaf8e2a0452d4896cfd7.tar.gz |
exported vg handling
The exported VG checking/enforcement was scattered and
inconsistent. This centralizes it and makes it consistent,
following the existing approach for foreign and shared
VGs/PVs, which are very similar to exported VGs/PVs.
The access policy that now applies to foreign/shared/exported
VGs/PVs, is that if a foreign/shared/exported VG/PV is named
on the command line (i.e. explicitly requested by the user),
and the command is not permitted to operate on it because it
is foreign/shared/exported, then an access error is reported
and the command exits with an error. But, if the command is
processing all VGs/PVs, and happens to come across a
foreign/shared/exported VG/PV (that is not explicitly named on
the command line), then the command silently skips it and does
not produce an error.
A command using tags or --select handles inaccessible VGs/PVs
the same way as a command processing all VGs/PVs, and will
not report/return errors if these inaccessible VGs/PVs exist.
The new policy fixes the exit codes on a somewhat random set of
commands that previously exited with an error if they were
looking at all VGs/PVs and an exported VG existed on the system.
There should be no change to which commands are allowed/disallowed
on exported VGs/PVs.
Certain LV commands (lvs/lvdisplay/lvscan) would previously not
display LVs from an exported VG (for unknown reasons). This has
not changed. The lvm fullreport command would previously report
info about an exported VG but not about the LVs in it. This
has changed to include all info from the exported VG.
Diffstat (limited to 'tools/pvresize.c')
-rw-r--r-- | tools/pvresize.c | 7 |
1 files changed, 1 insertions, 6 deletions
diff --git a/tools/pvresize.c b/tools/pvresize.c index c7e750d3d..2c7f543a0 100644 --- a/tools/pvresize.c +++ b/tools/pvresize.c @@ -36,11 +36,6 @@ static int _pvresize_single(struct cmd_context *cmd, } params->total++; - if (vg && vg_is_exported(vg)) { - log_error("Volume group %s is exported", vg->name); - return ECMD_FAILED; - } - /* * Needed to change a property on an orphan PV. * i.e. the global lock is only needed for orphans. @@ -93,7 +88,7 @@ int pvresize(struct cmd_context *cmd, int argc, char **argv) handle->custom_handle = ¶ms; - ret = process_each_pv(cmd, argc, argv, NULL, 0, READ_FOR_UPDATE | READ_ALLOW_EXPORTED, handle, _pvresize_single); + ret = process_each_pv(cmd, argc, argv, NULL, 0, READ_FOR_UPDATE, handle, _pvresize_single); log_print_unless_silent("%d physical volume(s) resized or updated / %d physical volume(s) " "not resized", params.done, params.total - params.done); |