FAQ Search Today's Posts Mark Forums Read
» Video Reviews

» Linux Archive

Linux-archive is a website aiming to archive linux email lists and to make them easily accessible for linux users/developers.


» Sponsor

» Partners

» Sponsor

Go Back   Linux Archive > Ubuntu > Kubuntu Development

 
 
LinkBack Thread Tools
 
Old 03-04-2010, 01:15 AM
David Lehman
 
Default Change udev_trigger block calls to use change action instead of add

On Wed, 2010-03-03 at 16:44 -0800, Brian C. Lane wrote:
> This fixes bug #569373, where exiting the filter screen would leave the
> /dev/dm-7 and /dev/dm-7 devices missing. It appears that device-mapper
> is removing the devices when udevadm trigger --action=add
> /class/block/dm-0 is run.

Looks good to me.

> ---
> iw/filter_gui.py | 4 ++--
> kickstart.py | 2 +-
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/iw/filter_gui.py b/iw/filter_gui.py
> index 0594712..3d553c7 100644
> --- a/iw/filter_gui.py
> +++ b/iw/filter_gui.py
> @@ -416,7 +416,7 @@ class FilterWindow(InstallWindow):
> if not addDrive(self.anaconda):
> return
>
> - udev_trigger(subsystem="block")
> + udev_trigger(subsystem="block", action="change")
> new_disks = filter(udev_device_is_disk, udev_get_block_devices())
> (new_singlepaths, new_mpaths, new_partitions) = identifyMultipaths(new_disks)
> (new_raids, new_nonraids) = self.split_list(lambda d: isRAID(d) and not isCCISS(d),
> @@ -538,7 +538,7 @@ class FilterWindow(InstallWindow):
> gobject.TYPE_STRING)
> self.store.set_sort_column_id(MODEL_COL, gtk.SORT_ASCENDING)
>
> - udev_trigger(subsystem="block")
> + udev_trigger(subsystem="block", action="change")
> # So that drives onlined by these show up in the filter UI
> storage.iscsi.iscsi().startup(anaconda.intf)
> storage.fcoe.fcoe().startup(anaconda.intf)
> diff --git a/kickstart.py b/kickstart.py
> index 7a00921..36908d0 100644
> --- a/kickstart.py
> +++ b/kickstart.py
> @@ -1208,7 +1208,7 @@ def parseKickstart(anaconda, file):
> ksparser = AnacondaKSParser(handler)
>
> # We need this so all the /dev/disk/* stuff is set up before parsing.
> - udev_trigger(subsystem="block")
> + udev_trigger(subsystem="block", action="change")
> # So that drives onlined by these can be used in the ks file
> storage.iscsi.iscsi().startup()
> storage.fcoe.fcoe().startup()


_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@redhat.com
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
 
Old 03-04-2010, 01:33 PM
Chris Lumens
 
Default Change udev_trigger block calls to use change action instead of add

> This fixes bug #569373, where exiting the filter screen would leave the
> /dev/dm-7 and /dev/dm-7 devices missing. It appears that device-mapper
> is removing the devices when udevadm trigger --action=add
> /class/block/dm-0 is run.

The patch looks fine.

If you put the bug number in the first line of the commit message, we'll
also get it in the anaconda.spec file when we rebuild the package. The
%changelog is auto-generated from git and only uses the first line of
each commit.

- Chris

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

Thread Tools




All times are GMT. The time now is 01:55 PM.

VBulletin, Copyright ©2000 - 2014, Jelsoft Enterprises Ltd.
Content Relevant URLs by vBSEO ©2007, Crawlability, Inc.
Copyright 2007 - 2008, www.linux-archive.org