On Tue, 24 Aug 2010, Ales Kozumplik wrote:
On 08/19/2010 08:05 PM, Ales Kozumplik wrote:
This patch essentialy reverts c5b49fe5c62a7f062fd4d19c157e481fc514f584. It
goes through all partitions of a mpath member at the moment we check it's
ignored and adds the partitions explicitly to the filter list.
Sending an updated version, the only added part is the
# slice off the "/dev/" part, lvm filter cares only about the
rest
partitions_paths = [p[5:] for p in partitions_paths]
.
I have one log from IBM available for the original patch and it seems to be
working okay (except the '/dev/' part of course):
16:58:25,413 DEBUG : ignoring sdi
(/devices/vio/3000000b/host8/target8:0:1/8:0:1:0/block/sdi)
16:58:25,413 DEBUG : adding partitions on /dev/sdi to the lvm ignore list
16:58:25,450 DEBUG : lvm filter: adding /dev/sdi1 to the reject list
16:58:25,451 DEBUG : lvm filter: adding /dev/sdi2 to the reject list
16:58:25,451 DEBUG : lvm filter: adding /dev/sdi3 to the reject list
Then when lvm is called:
16:58:30,078 INFO : Running... ['lvm', 'lvchange', '-a', 'y', '--config',
' devices {
filter=["r|/sda$|","r|/sdc$|","r|//dev/sdc1$|","r|//dev/sdc2$|","r|//dev/sdc3$|","r|/sdi$|","r|//dev/sdi1$|","r|//dev/sdi2$|","r|//dev/sdi3$|","r|/sdd$|","r|/sdf$|","r|/sdb$|","r|//dev/sdb1$|","r|//dev/sdb2$|","r|//dev/sdb3$|","r|/sdh$|","r|//dev/sdh1$|","r|//dev/sdh2$|","r|//dev/sdh3$|"]
} ', 'vg_jabulanlp4/lv_root']
(notice how the sdi* devices are there)
Ack. Good find, IBM will be pleased.
--
David Cantrell <dcantrell@xxxxxxxxxx>
Red Hat / Honolulu, HI
_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
