On 01/20/2010 04:50 PM, Peter Jones wrote: > On 01/20/2010 04:19 PM, Peter Jones wrote: >> Use device-mapper-multipath's "multipath" tool to find and set up >> multipath devices. >> --- >> storage/devicelibs/mpath.py | 91 +++++++++++++++++++++++++++++-------------- >> storage/devices.py | 78 ++++++++++++++++++++++++++++++------- >> 2 files changed, 125 insertions(+), 44 deletions(-) >> >> diff --git a/storage/devicelibs/mpath.py b/storage/devicelibs/mpath.py >> index 4d1b262..0f3efc2 100644 >> --- a/storage/devicelibs/mpath.py >> +++ b/storage/devicelibs/mpath.py >> @@ -1,10 +1,13 @@ >> from ..udev import * >> +import iutil >> >> def parseMultipathOutput(output): >> # this function parses output from "multipath -d", so we can use its >> # logic for our topology. It returns a structure like: >> # [ {'mpathb':['sdb','sdc']}, ... ] >> mpaths = {} >> + if output is None: >> + return mpaths > > This, of course, is still wrong. (In danger of having a whole conversation with myself...) No it's not wrong. This is parseMultipathOutput(), which returns a dictionary, not identifyMultipaths(). Dumbass. -- Peter Computers have already beaten communists at chess. The next thing you know, they'll be beating humans. -- Dale _______________________________________________ Anaconda-devel-list mailing list Anaconda-devel-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/anaconda-devel-list