Hello community,

here is the log from the commit of package autoyast2 for openSUSE:Factory 
checked in at 2018-02-24 16:36:27
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/autoyast2 (Old)
 and      /work/SRC/openSUSE:Factory/.autoyast2.new (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "autoyast2"

Sat Feb 24 16:36:27 2018 rev:235 rq:579521 version:4.0.32

Changes:
--------
--- /work/SRC/openSUSE:Factory/autoyast2/autoyast2.changes      2018-02-13 
10:23:50.601113455 +0100
+++ /work/SRC/openSUSE:Factory/.autoyast2.new/autoyast2.changes 2018-02-24 
16:36:28.779711490 +0100
@@ -1,0 +2,7 @@
+Fri Feb 23 13:15:50 UTC 2018 - igonzalezs...@suse.com
+
+- Remove calls to the old yast2-storage layer (bsc#1071978)
+- Fix AutoYaST UI to to show partitions properly
+- 4.0.32
+
+-------------------------------------------------------------------

Old:
----
  autoyast2-4.0.31.tar.bz2

New:
----
  autoyast2-4.0.32.tar.bz2

++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Other differences:
------------------
++++++ autoyast2.spec ++++++
--- /var/tmp/diff_new_pack.0eLkYW/_old  2018-02-24 16:36:29.495685721 +0100
+++ /var/tmp/diff_new_pack.0eLkYW/_new  2018-02-24 16:36:29.499685577 +0100
@@ -22,7 +22,7 @@
 %endif
 
 Name:           autoyast2
-Version:        4.0.31
+Version:        4.0.32
 Release:        0
 
 BuildRoot:      %{_tmppath}/%{name}-%{version}-build

++++++ autoyast2-4.0.31.tar.bz2 -> autoyast2-4.0.32.tar.bz2 ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/autoyast2-4.0.31/package/autoyast2.changes 
new/autoyast2-4.0.32/package/autoyast2.changes
--- old/autoyast2-4.0.31/package/autoyast2.changes      2018-02-12 
13:26:06.000000000 +0100
+++ new/autoyast2-4.0.32/package/autoyast2.changes      2018-02-23 
14:47:14.000000000 +0100
@@ -1,4 +1,11 @@
 -------------------------------------------------------------------
+Fri Feb 23 13:15:50 UTC 2018 - igonzalezs...@suse.com
+
+- Remove calls to the old yast2-storage layer (bsc#1071978)
+- Fix AutoYaST UI to to show partitions properly
+- 4.0.32
+
+-------------------------------------------------------------------
 Mon Feb 12 11:29:24 UTC 2018 - jreidin...@suse.com
 
 - fate#319119
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/autoyast2-4.0.31/package/autoyast2.spec 
new/autoyast2-4.0.32/package/autoyast2.spec
--- old/autoyast2-4.0.31/package/autoyast2.spec 2018-02-12 13:26:06.000000000 
+0100
+++ new/autoyast2-4.0.32/package/autoyast2.spec 2018-02-23 14:47:14.000000000 
+0100
@@ -22,7 +22,7 @@
 %endif
 
 Name:           autoyast2
-Version:        4.0.31
+Version:        4.0.32
 Release:        0
 
 BuildRoot:      %{_tmppath}/%{name}-%{version}-build
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/autoyast2-4.0.31/src/include/autoinstall/conftree.rb 
new/autoyast2-4.0.32/src/include/autoinstall/conftree.rb
--- old/autoyast2-4.0.31/src/include/autoinstall/conftree.rb    2018-02-12 
13:26:06.000000000 +0100
+++ new/autoyast2-4.0.32/src/include/autoinstall/conftree.rb    2018-02-23 
14:47:14.000000000 +0100
@@ -571,10 +571,6 @@
               # The settings will be written in a running system.
               # So we are switching to "normal" mode. (bnc#909223)
               Mode.SetMode("normal")
-              # As we are now in "normal" mode and accessing to the target
-              # system we have to set StorageDevices flag disks_valid to true.
-              # So InitLibstorage can scan valid disks. (bnc#1046738)
-              StorageDevices.InitDone
 
               Call.Function(module_auto, ["Write"])
               Mode.SetMode(oldMode)
@@ -756,11 +752,6 @@
             Mode.SetMode("autoinstallation")
             Stage.Set("continue")
 
-            # As we are now in "autoinstallation" mode and accessing to the 
target
-            # system we have to set StorageDevices flag disks_valid to true.
-            # So InitLibstorage can scan valid disks. (bnc#1047809)
-            StorageDevices.InitDone
-
             WFM.CallFunction("inst_autopost", [])
             AutoinstSoftware.addPostPackages(
               Ops.get_list(Profile.current, ["software", "post-packages"], [])
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/autoyast2-4.0.31/src/modules/AutoinstPartPlan.rb 
new/autoyast2-4.0.32/src/modules/AutoinstPartPlan.rb
--- old/autoyast2-4.0.31/src/modules/AutoinstPartPlan.rb        2018-02-12 
13:26:06.000000000 +0100
+++ new/autoyast2-4.0.32/src/modules/AutoinstPartPlan.rb        2018-02-23 
14:47:14.000000000 +0100
@@ -393,7 +393,8 @@
     # @return  [Boolean] success
     def Import(settings)
       log.info("entering Import with #{settings.inspect}")
-      @AutoPartPlan = settings
+      # index settings
+      @AutoPartPlan = settings.map.with_index { |d, i| d.merge("_id" => i) }
       true
     end
 
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/autoyast2-4.0.31/test/AutoinstPartPlan_test.rb 
new/autoyast2-4.0.32/test/AutoinstPartPlan_test.rb
--- old/autoyast2-4.0.31/test/AutoinstPartPlan_test.rb  2018-02-12 
13:26:06.000000000 +0100
+++ new/autoyast2-4.0.32/test/AutoinstPartPlan_test.rb  2018-02-23 
14:47:14.000000000 +0100
@@ -93,4 +93,48 @@
       expect(snapshots).to be_empty
     end
   end
+
+  describe "#getDrive" do
+    let(:sda) { { "device" => "/dev/sda" } }
+    let(:sdb) { { "device" => "/dev/sdb" } }
+    let(:settings) { [sda, sdb] }
+
+    before { subject.Import(settings) }
+
+    it "returns the drive in the given position" do
+      drive = subject.getDrive(1)
+      expect(drive["device"]).to eq("/dev/sdb")
+    end
+
+    context "when the drive does not exist" do
+      it "returns an empty hash" do
+        expect(subject.getDrive(2)).to eq({})
+      end
+    end
+  end
+
+  describe "#getPartition" do
+    let(:sda) do
+      { "device" => "/dev/sda", "partitions" => [{ "mount" => "/" }, { "mount" 
=> "/home" }] }
+    end
+    let(:settings) { [sda] }
+
+    before { subject.Import(settings) }
+
+    it "returns the drive in the given position" do
+      expect(subject.getPartition(0, 1)).to eq("mount" => "/home")
+    end
+
+    context "when the drive does not exist" do
+      it "returns an empty hash" do
+        expect(subject.getPartition(1, 0)).to eq({})
+      end
+    end
+
+    context "when the partition does not exist" do
+      it "returns an empty hash" do
+        expect(subject.getPartition(0, 2)).to eq({})
+      end
+    end
+  end
 end


Reply via email to