- disallow using unpartitioned DASDs for LVM (fate#317934)

yast · Jun 30, 2015 · 5b82a3d · 5b82a3d
1 parent d4f5c15
commit 5b82a3d
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 4 deletions.
diff --git a/package/yast2-storage.changes b/package/yast2-storage.changes
@@ -1,3 +1,9 @@
+-------------------------------------------------------------------
+Tue Jun 30 11:49:34 CEST 2015 - aschnell@suse.de
+
+- disallow using unpartitioned DASDs for LVM (fate#317934)
+- 3.1.60
+
 -------------------------------------------------------------------
 Fri Jun 26 10:18:28 CEST 2015 - aschnell@suse.de
 

diff --git a/package/yast2-storage.spec b/package/yast2-storage.spec
@@ -17,7 +17,7 @@
 
 
 Name:           yast2-storage
-Version:        3.1.59
+Version:        3.1.60
 Release:        0
 
 BuildRoot:      %{_tmppath}/%{name}-%{version}-build

diff --git a/src/include/partitioning/lvm_pv_lib.rb b/src/include/partitioning/lvm_pv_lib.rb
@@ -128,9 +128,11 @@ def get_possible_pvs(targetMap)
           )
         end
         if Builtins.size(Ops.get_list(devmap, "partitions", [])) == 0 &&
-            Storage.IsPartType(Ops.get_symbol(devmap, "type", :CT_UNKNOWN)) &&
-            (!Storage.IsUsedBy(devmap) ||
-              Ops.get_symbol(devmap, "used_by_type", :UB_NONE) == :UB_LVM)
+           Storage.IsPartType(Ops.get_symbol(devmap, "type", :CT_UNKNOWN)) &&
+           # unpartitioned DASDs cannot be used for LVM (fate #317934)
+           !String.StartsWith(dev, "/dev/dasd") &&
+           (!Storage.IsUsedBy(devmap) ||
+            Ops.get_symbol(devmap, "used_by_type", :UB_NONE) == :UB_LVM)
           p = {
             "device"  => dev,
             "maindev" => dev,