On Tue, 2011-03-01 at 18:53 +0100, Ales Kozumplik wrote:
> Related: rhbz#572862
> ---
> iutil.py | 10 ++++++++++
> kickstart.py | 4 ++++
> 2 files changed, 14 insertions(+), 0 deletions(-)
Ack.
>
> diff --git a/iutil.py b/iutil.py
> index 8b3e6b9..c482b18 100644
> --- a/iutil.py
> +++ b/iutil.py
> @@ -444,6 +444,16 @@ def cpuFeatureFlags():
>
> return []
>
> +def valid_dm_name(name):
> + """ Tests whether name is a good name for a dm device
> +
> + Sometimes the user could call the device 'devp3' and then how is
> + bootloaderInfo.getDiskPart() to tell it's not a partition?
> + """
> + if re.search("p\d{1,2}$", name):
> + return False
> + return True
> +
> def writeRpmPlatform(root="/"):
> import rhpl.arch
>
> diff --git a/kickstart.py b/kickstart.py
> index 1f414c5..46c6c1f 100644
> --- a/kickstart.py
> +++ b/kickstart.py
> @@ -393,6 +393,10 @@ class AnacondaKSHandlers(KickstartHandlers):
> newname = ""
> it = True
> for path in mpath.paths:
> + if not iutil.valid_dm_name(path.name):
> + msg = "Invalid name for a multipath device '%s' "\
> + "(resembles a partition?)." % path.name
> + raise KickstartValueError, formatErrorMsg(self.lineno, msg)
> dev = path.device
> log.debug("Searching for mpath having '%s' as a member, the scsi id or wwpn:lunid" % (dev,))
> log.debug("mpath '%s' has members %s" % (mp.name, list(mp.members)))
_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
