Script 'mail_helper' called by obssrc
Hello community,

here is the log from the commit of package yast2-storage-ng for 
openSUSE:Factory checked in at 2024-04-04 22:24:07
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/yast2-storage-ng (Old)
 and      /work/SRC/openSUSE:Factory/.yast2-storage-ng.new.1905 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "yast2-storage-ng"

Thu Apr  4 22:24:07 2024 rev:156 rq:1164333 version:5.0.11

Changes:
--------
--- /work/SRC/openSUSE:Factory/yast2-storage-ng/yast2-storage-ng.changes        
2024-03-26 19:25:13.072424921 +0100
+++ 
/work/SRC/openSUSE:Factory/.yast2-storage-ng.new.1905/yast2-storage-ng.changes  
    2024-04-04 22:24:21.875127054 +0200
@@ -1,0 +2,7 @@
+Wed Apr  3 09:52:32 UTC 2024 - Ancor Gonzalez Sosa <an...@suse.com>
+
+- GuidedProposal: fixed a problem related to the :bigger_resize
+  strategy (Agama) detected at gh#openSUSE/agama#1106.
+- 5.0.11
+
+-------------------------------------------------------------------

Old:
----
  yast2-storage-ng-5.0.10.tar.bz2

New:
----
  yast2-storage-ng-5.0.11.tar.bz2

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

Other differences:
------------------
++++++ yast2-storage-ng.spec ++++++
--- /var/tmp/diff_new_pack.dv9puI/_old  2024-04-04 22:24:22.475149145 +0200
+++ /var/tmp/diff_new_pack.dv9puI/_new  2024-04-04 22:24:22.479149292 +0200
@@ -17,7 +17,7 @@
 
 
 Name:           yast2-storage-ng
-Version:        5.0.10
+Version:        5.0.11
 Release:        0
 Summary:        YaST2 - Storage Configuration
 License:        GPL-2.0-only OR GPL-3.0-only

++++++ yast2-storage-ng-5.0.10.tar.bz2 -> yast2-storage-ng-5.0.11.tar.bz2 ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-storage-ng-5.0.10/package/yast2-storage-ng.changes 
new/yast2-storage-ng-5.0.11/package/yast2-storage-ng.changes
--- old/yast2-storage-ng-5.0.10/package/yast2-storage-ng.changes        
2024-03-22 13:14:07.000000000 +0100
+++ new/yast2-storage-ng-5.0.11/package/yast2-storage-ng.changes        
2024-04-03 13:04:24.000000000 +0200
@@ -1,4 +1,11 @@
 -------------------------------------------------------------------
+Wed Apr  3 09:52:32 UTC 2024 - Ancor Gonzalez Sosa <an...@suse.com>
+
+- GuidedProposal: fixed a problem related to the :bigger_resize
+  strategy (Agama) detected at gh#openSUSE/agama#1106.
+- 5.0.11
+
+-------------------------------------------------------------------
 Thu Mar 21 12:28:56 UTC 2024 - Ancor Gonzalez Sosa <an...@suse.com>
 
 - GuidedProposal: internal settings to control the configuration
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-storage-ng-5.0.10/package/yast2-storage-ng.spec 
new/yast2-storage-ng-5.0.11/package/yast2-storage-ng.spec
--- old/yast2-storage-ng-5.0.10/package/yast2-storage-ng.spec   2024-03-22 
13:14:07.000000000 +0100
+++ new/yast2-storage-ng-5.0.11/package/yast2-storage-ng.spec   2024-04-03 
13:04:24.000000000 +0200
@@ -16,7 +16,7 @@
 #
 
 Name:           yast2-storage-ng
-Version:        5.0.10
+Version:        5.0.11
 Release:        0
 Summary:        YaST2 - Storage Configuration
 License:        GPL-2.0-only OR GPL-3.0-only
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-storage-ng-5.0.10/src/lib/y2storage/proposal/partitions_distribution_calculator.rb
 
new/yast2-storage-ng-5.0.11/src/lib/y2storage/proposal/partitions_distribution_calculator.rb
--- 
old/yast2-storage-ng-5.0.10/src/lib/y2storage/proposal/partitions_distribution_calculator.rb
        2024-03-22 13:14:07.000000000 +0100
+++ 
new/yast2-storage-ng-5.0.11/src/lib/y2storage/proposal/partitions_distribution_calculator.rb
        2024-04-03 13:04:24.000000000 +0200
@@ -477,6 +477,10 @@
 
           # At least one distribution is valid
           assigned_spaces = distros.map { |i| i.spaces.find { |a| 
a.disk_space.growing? } }
+
+          # There are valid distributions that don't need to use the growing 
space
+          return DiskSize.zero if assigned_spaces.include?(nil)
+
           missing = assigned_spaces.map(&:total_missing_size).min
           return missing.ceil(align_grain)
         end
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-storage-ng-5.0.10/test/y2storage/proposal_agama_advanced_test.rb 
new/yast2-storage-ng-5.0.11/test/y2storage/proposal_agama_advanced_test.rb
--- old/yast2-storage-ng-5.0.10/test/y2storage/proposal_agama_advanced_test.rb  
1970-01-01 01:00:00.000000000 +0100
+++ new/yast2-storage-ng-5.0.11/test/y2storage/proposal_agama_advanced_test.rb  
2024-04-03 13:04:24.000000000 +0200
@@ -0,0 +1,84 @@
+#!/usr/bin/env rspec
+
+# Copyright (c) [2024] SUSE LLC
+#
+# All Rights Reserved.
+#
+# This program is free software; you can redistribute it and/or modify it
+# under the terms of version 2 of the GNU General Public License as published
+# by the Free Software Foundation.
+#
+# This program is distributed in the hope that it will be useful, but WITHOUT
+# ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
+# FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License for
+# more details.
+#
+# You should have received a copy of the GNU General Public License along
+# with this program; if not, contact SUSE LLC.
+#
+# To contact SUSE LLC about this file by physical or electronic mail, you may
+# find current contact information at www.suse.com.
+
+require_relative "spec_helper"
+require "storage"
+require "y2storage"
+require_relative "#{TEST_PATH}/support/proposal_examples"
+require_relative "#{TEST_PATH}/support/proposal_context"
+
+describe Y2Storage::MinGuidedProposal do
+  describe "#propose with settings in the Agama style" do
+    subject(:proposal) { described_class.new(settings: settings) }
+
+    include_context "proposal"
+    let(:architecture) { :x86 }
+    let(:settings_format) { :ng }
+    let(:separate_home) { true }
+    let(:control_file_content) { { "partitioning" => { "proposal" => {}, 
"volumes" => volumes } } }
+    let(:volumes) { [root_vol, swap_vol] }
+    let(:root_vol) do
+      { "mount_point" => "/", "fs_type" => "xfs", "min_size" => "30 GiB" }
+    end
+    let(:swap_vol) do
+      { "mount_point" => "swap", "fs_type" => "swap", "min_size" => "1 GiB", 
"max_size" => "2 GiB" }
+    end
+
+    let(:scenario) { "mixed_disks" }
+
+    before do
+      # Speed-up things by avoiding calls to hwinfo
+      allow_any_instance_of(Y2Storage::Disk).to 
receive(:hwinfo).and_return(Y2Storage::HWInfoDisk.new)
+
+      # Remove existing mount points, this is a new installation
+      fake_devicegraph.mount_points.each { |i| 
i.parents.first.remove_mount_point }
+
+      settings.space_settings.strategy = :bigger_resize
+      # Agama uses homogeneous weights for all volumes
+      settings.volumes.each { |v| v.weight = 100 }
+      # Activate support for separate LVM VGs
+      settings.separate_vgs = true
+    end
+
+    context "when there are resize actions for a disk that already contains 
the needed space" do
+      let(:resize_info) do
+        instance_double("Y2Storage::ResizeInfo", resize_ok?: true, reasons: 0, 
reason_texts: [],
+          min_size: Y2Storage::DiskSize.GiB(40), max_size: 
Y2Storage::DiskSize.GiB(200))
+      end
+
+      let(:lvm) { true }
+
+      before do
+        settings.candidate_devices = ["/dev/sdb"]
+        settings.root_device = "/dev/sda"
+        settings.space_settings.actions = { "/dev/sda1" => :resize, 
"/dev/sda2" => :resize }
+        allow(storage_arch).to receive(:efiboot?).and_return(true)
+      end
+
+      # This used to fail when evaluating the resize actions at sda1 and/or 
sda2, although
+      # those actions are not really needed to allocate /boot/efi (the only 
partition that
+      # needs to get created at sda).
+      it "does not crash" do
+        expect { proposal.propose }.to_not raise_error
+      end
+    end
+  end
+end

Reply via email to