diff options
| author | David Teigland <teigland@redhat.com> | 2015-01-20 13:08:22 -0600 |
|---|---|---|
| committer | David Teigland <teigland@redhat.com> | 2015-01-20 13:08:22 -0600 |
| commit | a164d603d386c5daed06b46316ef3c001807dff9 (patch) | |
| tree | 042a69c98e7912c6d870c053d286feaad275049a | |
| parent | 404c834e14657645d70d19b825d328bff5b165b7 (diff) | |
| download | lvm2-a164d603d386c5daed06b46316ef3c001807dff9.tar.gz | |
vgimportclone: remove arg check that uses pvs
The arg check using pvs is unnecessary. If the arg is not a PV,
the command will just fail later. Using the pvs command at this
point in the command is a problem when lvmetad is running, because
the pvs command does not report duplicate PVs when using lvmetad.
(Alternatively, use_lvmetad could be disabled by adding a --config
override to this pvs command.)
| -rwxr-xr-x | scripts/vgimportclone.sh | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/scripts/vgimportclone.sh b/scripts/vgimportclone.sh index 9d319bd97..388d14c87 100755 --- a/scripts/vgimportclone.sh +++ b/scripts/vgimportclone.sh @@ -204,11 +204,6 @@ for ARG do if [ -b "$ARG" ] then - PVS_OUT=`"${LVM}" pvs ${LVM_OPTS} --noheadings -o vg_name "$ARG"` - checkvalue $? "$ARG could not be verified to be a PV without errors." - PV_VGNAME=$(echo $PVS_OUT | $GREP -v '[[:space:]]+$') - [ -z "$PV_VGNAME" ] && die 3 "$ARG is not in a VG." - ln -s "$ARG" ${TMP_LVM_SYSTEM_DIR}/vgimport${DEVNO} DISKS="${DISKS} ${TMP_LVM_SYSTEM_DIR}/vgimport${DEVNO}" DEVNO=$((${DEVNO}+1)) |