Author: fehr Date: Wed Jan 25 12:25:00 2012 New Revision: 67285 URL: http://svn.opensuse.org/viewcvs/yast?rev=67285&view=rev Log: fix bug handling used devices of simple btrfs volumes in resize Modified: trunk/storage/package/yast2-storage.changes trunk/storage/storage/src/include/ep-btrfs-dialogs.ycp Modified: trunk/storage/package/yast2-storage.changes URL: http://svn.opensuse.org/viewcvs/yast/trunk/storage/package/yast2-storage.cha... ============================================================================== --- trunk/storage/package/yast2-storage.changes (original) +++ trunk/storage/package/yast2-storage.changes Wed Jan 25 12:25:00 2012 @@ -1,4 +1,9 @@ ------------------------------------------------------------------- +Wed Jan 25 12:20:53 CET 2012 - fehr@suse.de + +- fix bug handling used devices of simple btrfs volumes in resize + +------------------------------------------------------------------- Tue Jan 24 14:15:29 CET 2012 - aschnell@suse.de - do not query user to autostart multipath in AutoYaST (bnc#742913) Modified: trunk/storage/storage/src/include/ep-btrfs-dialogs.ycp URL: http://svn.opensuse.org/viewcvs/yast/trunk/storage/storage/src/include/ep-bt... ============================================================================== --- trunk/storage/storage/src/include/ep-btrfs-dialogs.ycp (original) +++ trunk/storage/storage/src/include/ep-btrfs-dialogs.ycp Wed Jan 25 12:25:00 2012 @@ -69,7 +69,7 @@ y2milestone( "GetPossibleVols parts:%1", devmap["partitions"]:[] ); list<map> parts = filter( map part, devmap["partitions"]:[], - ``( size(part["mount"]:"")==0 && + ``( (size(part["mount"]:"")==0 || part["used_by_type"]:`UB_NONE==`UB_BTRFS) && !contains( types_no, part["type"]:`primary ) && contains(allowed_enc_types, part["enc_type"]:`none) && (!Storage::IsUsedBy(part) || part["used_by_type"]:`UB_NONE==`UB_BTRFS) && @@ -135,9 +135,10 @@ list<symbol> fields = StorageSettings::FilterTable([ `device, `udev_path, `udev_id, `size, `encrypted, `type ]); - map<string, map> target_map = Storage::GetTargetMap(); - list<map> unused_pvs = filter(map pv, GetPossibleVols(target_map), { return !Storage::IsUsedBy(pv); }); - list<map> used_pvs = filter(map pv, GetPossibleVols(target_map), { return pv["used_by_device"]:"" == data["uuid"]:""; }); + list<map> poss_pvs = GetPossibleVols(Storage::GetTargetMap()); + y2milestone( "MiniWorkflowStepResizeVolume poss:%1", maplist( map pv, poss_pvs, ``( [ pv["device"]:"", pv["used_by_device"]:"" ]))); + list<map> unused_pvs = filter(map pv, poss_pvs, ``(pv["used_by_type"]:`UB_NONE==`UB_NONE)); + list<map> used_pvs = filter(map pv, poss_pvs, ``(pv["used_by_device"]:""==data["uuid"]:"")); term contents = `VBox(); -- To unsubscribe, e-mail: yast-commit+unsubscribe@opensuse.org For additional commands, e-mail: yast-commit+help@opensuse.org