mirror of
git://sourceware.org/git/lvm2.git
synced 2024-12-22 17:35:59 +03:00
process_each_pv: remove unnecessary workaround
The problem addressed by this workaround no longer seems to exist, so remove it. PVs with no mdas no longer appear in both their actual VG and in the orphan VG.
This commit is contained in:
parent
63d59254d9
commit
be1b1f3d89
@ -3033,7 +3033,6 @@ static int _process_pvs_in_vg(struct cmd_context *cmd,
|
||||
const char *pv_name;
|
||||
int selected;
|
||||
int process_pv;
|
||||
int dev_found;
|
||||
int ret_max = ECMD_PROCESSED;
|
||||
int ret = 0;
|
||||
|
||||
@ -3087,21 +3086,7 @@ static int _process_pvs_in_vg(struct cmd_context *cmd,
|
||||
else
|
||||
log_very_verbose("Processing PV %s in VG %s.", pv_name, vg->name);
|
||||
|
||||
dev_found = _device_list_remove(all_devices, pv->dev);
|
||||
|
||||
/*
|
||||
* FIXME PVs with no mdas may turn up in an orphan VG when
|
||||
* not using lvmetad as well as their correct VG. They
|
||||
* will be missing from all_devices the second time
|
||||
* around but must not be processed twice or trigger a message.
|
||||
*
|
||||
* Missing PVs will also need processing even though they are
|
||||
* not present in all_devices.
|
||||
*/
|
||||
if (!dev_found && !is_missing_pv(pv)) {
|
||||
log_verbose("Skipping PV %s in VG %s: not in device list.", pv_name, vg->name);
|
||||
continue;
|
||||
}
|
||||
_device_list_remove(all_devices, pv->dev);
|
||||
|
||||
if (!skip) {
|
||||
ret = process_single_pv(cmd, vg, pv, handle);
|
||||
|
Loading…
Reference in New Issue
Block a user