At a certain number of LUN paths you start to see them ending with "p" which the old expression would catch - at least that was my interpretation from the discussion with David Lehman.<div><br></div><div>--Rob<br><br>On Wed, Jan 21, 2015 at 2:33 AM, Vratislav Podzimek &lt;vpodzime@redhat.com&gt; wrote:<br>
<blockquote type="cite"><div class="plaintext" style="white-space: pre-wrap;">On Tue, 2015-01-20 at 14:12 -0500, Robert Marshall wrote:
<blockquote> If installing a system with over 250 LUN paths the blivet subsystem
 would crash with an error. Partnered with David Lehman and he was
 able to provide a solution.
 
 Resolves rhbz#1181336
 ---
  blivet/udev.py | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)
 
 diff --git a/blivet/udev.py b/blivet/udev.py
 index 65652bd..c23ce7e 100644
 --- a/blivet/udev.py
 +++ b/blivet/udev.py
 @@ -577,7 +577,7 @@ def device_is_biosraid_member(info):
      return False
  
  def device_get_dm_partition_disk(info):
 -    return re.sub(r'p?\d*$', '', device_get_name(info))
 +    return os.path.basename(os.path.dirname(device_get_sysfs_path(info)))
  
</blockquote>I may be missing something, but I don't see how the original statement
causes a crash. Is it that names of LUNs start with some other letter if
there are more than 250 of them?

<div>-- 
</div>Vratislav Podzimek

Anaconda Rider | Red Hat, Inc. | Brno - Czech Republic

_______________________________________________
anaconda-patches mailing list
anaconda-patches@lists.fedorahosted.org
<a href="https://lists.fedorahosted.org/mailman/listinfo/anaconda-patches">https://lists.fedorahosted.org/mailman/listinfo/anaconda-patches</a>
</div></blockquote></div>