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

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

 



> @@ -1313,12 +1322,19 @@ class DeviceTree(object):
>                                           sysfsPath=sysfs_path,
>                                           exists=True,
>                                           parents=[device])
> -            try:
> -                md_array.setup()
> -            except (DeviceError, MDRaidError) as e:
> -                log.info("setup of md array %s failed: %s"
> -                            % (md_array.name, e))
> +#            try:
> +#                md_array.setup()
> +#            except (DeviceError, MDRaidError) as e:
> +#                log.info("setup of md array %s failed: %s"
> +#                            % (md_array.name, e))
>              self._addDevice(md_array)
> +            try: 
> +                 # FIXME: this should be mdraid.mdadd with parameter 
> +                 # no_degraded parameter depends on which member is added
> +                 # it was tested only for isw raid 1 array with 2 disks
> +                 mdraid.mdadd_no_degraded(device.path)
> +            except MDRaidError as e:
> +                  log.warning("failed to add member to md array %s" % e)

This also needs to be moved into MDRaidArrayDevice -- that's what the
Device classes are for.


>  
>      def handleUdevDMRaidMemberFormat(self, info, device):
>          log_method_call(self, name=device.name, type=device.format.type) 
> 
> _______________________________________________
> Anaconda-devel-list mailing list
> Anaconda-devel-list@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/anaconda-devel-list

_______________________________________________
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