Peter Rajnoha 7e5e2dd4ee vgextend: do not allow PV with 0 MDAs to be added while already in a VG ...

If extending a VG and including a PV with 0 MDAs that was already
a part of a VG, the vgextend allowed that PV to be added and we
ended up *with one PV in two VGs*!

The vgextend code used the 'pv_by_path' fn that returned a PV for
a given path. However, when the PV did not have any metadata areas,
the fn just returned a PV without any reference to existing VG.
Consequently, any checks for the existing VG failed.

[0] raw/~ # pvcreate --metadatacopies 0 /dev/sda
  Physical volume "/dev/sda" successfully created

[0] raw/~ # pvcreate --metadatacopies 1 /dev/sdb
  Physical volume "/dev/sdb" successfully created

[0] raw/~ # vgcreate vg1 /dev/sda /dev/sdb
  Volume group "vg1" successfully created

[0] raw/~ # pvcreate --metadatacopies 1 /dev/sdc
  Physical volume "/dev/sdc" successfully created

[0] raw/~ # vgcreate vg2 /dev/sdc
  Volume group "vg2" successfully created

Before this patch (incorrect):
[0] raw/~ # vgextend vg2 /dev/sda
  Volume group "vg2" successfully extended

With this patch (correct):
[0] raw/~ # vgextend vg2 /dev/sda
  Physical volume '/dev/sda' is already in volume group 'vg1'
  Unable to add physical volume '/dev/sda' to volume group 'vg2'.

2013-03-19 14:57:36 +01:00

..

activate

cleanup: move comment

2013-03-13 15:13:50 +01:00

cache

lvmetad: fix to properly process embedding area

2013-02-27 10:36:49 +01:00

commands

config: use config checks and add support for creating trees from config definition (config_def_create_tree fn)

2013-03-06 10:46:35 +01:00

config

config: make DEFAULT_MAX_HISTORY unconditional

2013-03-06 12:47:23 +01:00

datastruct

Switch to return void

2012-02-08 12:52:58 +00:00

device

config: refer to config nodes using assigned IDs

2013-03-06 10:14:33 +01:00

display

config: refer to config nodes using assigned IDs

2013-03-06 10:14:33 +01:00

error

cleanup: drop unneeded included header files

2012-08-23 14:37:20 +02:00

filters

filters: power2 devs get precedence if PVIDs match

2013-03-11 20:10:49 +00:00

format1

pv_header_extension: add support for writing PV header extension (flags & Embedding Area)

2013-02-26 11:28:00 +01:00

format_pool

cleanup: use struct pvcreate_restorable_params throughout

2013-02-26 11:25:11 +01:00

format_text

config: refer to config nodes using assigned IDs

2013-03-06 10:14:33 +01:00

freeseg

cleanup: drop unneeded included header files

2012-08-23 14:37:20 +02:00

label

logging: classify log_debug messages

2013-01-07 22:30:29 +00:00

locking

config: refer to config nodes using assigned IDs

2013-03-06 10:14:33 +01:00

log

logging: add debug classes

2013-01-07 22:25:19 +00:00

metadata

vgextend: do not allow PV with 0 MDAs to be added while already in a VG

2013-03-19 14:57:36 +01:00

mirror

config: refer to config nodes using assigned IDs

2013-03-06 10:14:33 +01:00

misc

config: refer to config nodes using assigned IDs

2013-03-06 10:14:33 +01:00

mm

config: refer to config nodes using assigned IDs

2013-03-06 10:14:33 +01:00

raid

RAID: Code changes missing from previous commit (bbc6378)

2013-03-06 10:17:11 -06:00

replicator

cleanup: drop unneeded included header files

2012-08-23 14:37:20 +02:00

report

report: add reporting fields for Embedding Area start and size

2013-02-26 14:46:42 +01:00

snapshot

config: refer to config nodes using assigned IDs

2013-03-06 10:14:33 +01:00

striped

Add activation/use_linear_target enabled by default. (prajnoha)

2011-11-28 20:37:51 +00:00

thin

config: refer to config nodes using assigned IDs

2013-03-06 10:14:33 +01:00

unknown

cleanup: drop unneeded included header files

2012-08-23 14:37:20 +02:00

uuid

Revert the #include changes. Need to fix this at the #include site for now, and

2011-07-18 14:34:33 +00:00

zero

cleanup: drop unneeded included header files

2012-08-23 14:37:20 +02:00

Makefile.in

cleanup: drop unneeded included header files

2012-08-23 14:37:20 +02:00