[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [rhel6-branch] mpath: filter member partitions wiser in lvm.



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)

Ales


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]