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