Re: [PATCH 7/8] Changes for using mdadm for isw_raid_member

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

 



Hans de Goede wrote:
I still don't get this one, you wrote in response
to my previous question:

name=/dev/md/imsm0
self.path=/dev/imsm
Mdadm 3.0 allows to create files with specific name in /dev/md not in /dev

First of all I assume that both lines should have imsm0 ?
Yes, my mistake, I'm sorry. self.path is set to /dev/imsm0
and this name for container is incorrect, so I'm changing it to /dev/md/imsm0
I think I'll rewrite this patch because I can set correct name of container in devicetree.py
before calling MDRaidArrayDevice.__init__()
Also not that:
"/dev/imsm".split("/")[2]
Will not get you "imsm" but an array out
of bounds exception. What you want then is:
"/dev/imsm".split("/")[1]

Hmm, I can see something different in the script below

path="/dev/imsm0"
print "1:", path.split("/")[1]
print "2:", path.split("/")[2]
print "/dev/imsm0".split("/")[2]

It gives me:
1: dev
2: imsm0
imsm0

But if the imsm device node lives under
/dev/md, then the path property should be changed
to properly return this for the imsm representing
device.

Yes, I'll change this.


_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list

[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux