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 /lib/commands | |
| 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 'lib/commands')
| -rw-r--r-- | lib/commands/toolcontext.h | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/lib/commands/toolcontext.h b/lib/commands/toolcontext.h index 6e4530c8a..488752c8f 100644 --- a/lib/commands/toolcontext.h +++ b/lib/commands/toolcontext.h @@ -148,6 +148,7 @@ struct cmd_context { unsigned unknown_system_id:1; unsigned include_historical_lvs:1; /* also process/report/display historical LVs */ unsigned record_historical_lvs:1; /* record historical LVs */ + unsigned include_exported_vgs:1; unsigned include_foreign_vgs:1; /* report/display cmds can reveal foreign VGs */ unsigned include_shared_vgs:1; /* report/display cmds can reveal lockd VGs */ unsigned include_active_foreign_vgs:1; /* cmd should process foreign VGs with active LVs */ |