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-06-17 19:27:08
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/yast2-storage-ng (Old)
 and      /work/SRC/openSUSE:Factory/.yast2-storage-ng.new.19518 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "yast2-storage-ng"

Mon Jun 17 19:27:08 2024 rev:160 rq:1180916 version:5.0.15

Changes:
--------
--- /work/SRC/openSUSE:Factory/yast2-storage-ng/yast2-storage-ng.changes        
2024-05-09 12:08:22.050952603 +0200
+++ 
/work/SRC/openSUSE:Factory/.yast2-storage-ng.new.19518/yast2-storage-ng.changes 
    2024-06-17 19:27:32.077951890 +0200
@@ -1,0 +2,13 @@
+Fri Jun 14 12:41:07 UTC 2024 - Ancor Gonzalez Sosa <an...@suse.com>
+
+- Fixed a problem when trying to locate a Btrfs root file-system
+  directly on top of a disk (gh#openSUSE/agama#1339).
+- 5.0.15
+
+-------------------------------------------------------------------
+Thu Jun 13 15:16:24 UTC 2024 - José Iván López González <jlo...@suse.com>
+
+- Fix device description for encrypted PV
+  (gh#yast/yast-storage-ng#1384).
+
+-------------------------------------------------------------------

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

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

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

Other differences:
------------------
++++++ yast2-storage-ng.spec ++++++
--- /var/tmp/diff_new_pack.WZrm1e/_old  2024-06-17 19:27:32.689974179 +0200
+++ /var/tmp/diff_new_pack.WZrm1e/_new  2024-06-17 19:27:32.693974325 +0200
@@ -17,7 +17,7 @@
 
 
 Name:           yast2-storage-ng
-Version:        5.0.14
+Version:        5.0.15
 Release:        0
 Summary:        YaST2 - Storage Configuration
 License:        GPL-2.0-only OR GPL-3.0-only

++++++ yast2-storage-ng-5.0.14.tar.bz2 -> yast2-storage-ng-5.0.15.tar.bz2 ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-storage-ng-5.0.14/package/yast2-storage-ng.changes 
new/yast2-storage-ng-5.0.15/package/yast2-storage-ng.changes
--- old/yast2-storage-ng-5.0.14/package/yast2-storage-ng.changes        
2024-05-08 11:10:59.000000000 +0200
+++ new/yast2-storage-ng-5.0.15/package/yast2-storage-ng.changes        
2024-06-14 14:59:33.000000000 +0200
@@ -1,4 +1,17 @@
 -------------------------------------------------------------------
+Fri Jun 14 12:41:07 UTC 2024 - Ancor Gonzalez Sosa <an...@suse.com>
+
+- Fixed a problem when trying to locate a Btrfs root file-system
+  directly on top of a disk (gh#openSUSE/agama#1339).
+- 5.0.15
+
+-------------------------------------------------------------------
+Thu Jun 13 15:16:24 UTC 2024 - José Iván López González <jlo...@suse.com>
+
+- Fix device description for encrypted PV
+  (gh#yast/yast-storage-ng#1384).
+
+-------------------------------------------------------------------
 Tue May  7 14:38:42 UTC 2024 - Ancor Gonzalez Sosa <an...@suse.com>
 
 - Proposal: Make the encryption method and the key derivation
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-storage-ng-5.0.14/package/yast2-storage-ng.spec 
new/yast2-storage-ng-5.0.15/package/yast2-storage-ng.spec
--- old/yast2-storage-ng-5.0.14/package/yast2-storage-ng.spec   2024-05-08 
11:10:59.000000000 +0200
+++ new/yast2-storage-ng-5.0.15/package/yast2-storage-ng.spec   2024-06-14 
14:59:33.000000000 +0200
@@ -16,7 +16,7 @@
 #
 
 Name:           yast2-storage-ng
-Version:        5.0.14
+Version:        5.0.15
 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.14/src/lib/y2storage/device_description.rb 
new/yast2-storage-ng-5.0.15/src/lib/y2storage/device_description.rb
--- old/yast2-storage-ng-5.0.14/src/lib/y2storage/device_description.rb 
2024-05-08 11:10:59.000000000 +0200
+++ new/yast2-storage-ng-5.0.15/src/lib/y2storage/device_description.rb 
2024-06-14 14:59:33.000000000 +0200
@@ -206,7 +206,7 @@
 
       if encrypted && include_encryption
         # TRANSLATORS: %s is the volume group name. E.g., "vg0"
-        format(_("Encrypted PV of %s"), vg.basename)
+        return format(_("Encrypted PV of %s"), vg.basename)
       end
 
       # TRANSLATORS: %s is the volume group name. E.g., "vg0"
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-storage-ng-5.0.14/src/lib/y2storage/proposal/devices_planner.rb 
new/yast2-storage-ng-5.0.15/src/lib/y2storage/proposal/devices_planner.rb
--- old/yast2-storage-ng-5.0.14/src/lib/y2storage/proposal/devices_planner.rb   
2024-05-08 11:10:59.000000000 +0200
+++ new/yast2-storage-ng-5.0.15/src/lib/y2storage/proposal/devices_planner.rb   
2024-06-14 14:59:33.000000000 +0200
@@ -350,6 +350,7 @@
       # @param volume [VolumeSpecification]
       def adjust_btrfs_sizes(planned_device, volume)
         return if volume.ignore_snapshots_sizes?
+        return unless planned_device.respond_to?(:min_size)
 
         if volume.snapshots_size > DiskSize.zero
           planned_device.min_size += volume.snapshots_size
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-storage-ng-5.0.14/test/y2storage/device_description_test.rb 
new/yast2-storage-ng-5.0.15/test/y2storage/device_description_test.rb
--- old/yast2-storage-ng-5.0.14/test/y2storage/device_description_test.rb       
2024-05-08 11:10:59.000000000 +0200
+++ new/yast2-storage-ng-5.0.15/test/y2storage/device_description_test.rb       
2024-06-14 14:59:33.000000000 +0200
@@ -190,6 +190,8 @@
             expect(description).to eq("PV of LVM")
           end
         end
+
+        include_examples "Encrypted device"
       end
 
       context "and it is an unused LVM physical volume" do
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/yast2-storage-ng-5.0.14/test/y2storage/proposal_agama_reuse_test.rb 
new/yast2-storage-ng-5.0.15/test/y2storage/proposal_agama_reuse_test.rb
--- old/yast2-storage-ng-5.0.14/test/y2storage/proposal_agama_reuse_test.rb     
2024-05-08 11:10:59.000000000 +0200
+++ new/yast2-storage-ng-5.0.15/test/y2storage/proposal_agama_reuse_test.rb     
2024-06-14 14:59:33.000000000 +0200
@@ -45,7 +45,10 @@
     # Let's define some volumes to shuffle them around among the disks
     let(:volumes) { [root_vol, srv_vol, swap_vol] }
     let(:root_vol) do
-      { "mount_point" => "/", "fs_type" => "xfs", "min_size" => "5 GiB", 
"max_size" => "30 GiB" }
+      {
+        "mount_point" => "/", "fs_type" => "btrfs", "min_size" => "5 GiB", 
"max_size" => "30 GiB",
+        "snapshots" => true, "snapshots_percentage" => 160
+      }
     end
     let(:srv_vol) do
       { "mount_point" => "/srv", "fs_type" => "xfs", "min_size" => "5 GiB", 
"max_size" => "10 GiB" }
@@ -269,6 +272,22 @@
           expect(filesystem.type.is?(:xfs)).to eq true
         end
       end
+
+      context "if the relocated volume is the root Btrfs with snapshots" do
+        before do
+          srv = settings.volumes.find { |v| v.mount_point == "/" }
+          srv.reuse_name = "/dev/sdc"
+        end
+
+        # The proposal used to throw an exception when adjusting the 
Btrfs-related sizes
+        # if the target device was a disk (so no sizes could actually be 
adjusted)
+        it "makes the proposal and assigns the mount point" do
+          proposal.propose
+          filesystem = proposal.devices.filesystems.find { |i| i.mount_path == 
"/" }
+          expect(filesystem.blk_devices.first.name).to eq "/dev/sdc"
+          expect(filesystem.type.is?(:btrfs)).to eq true
+        end
+      end
     end
 
     context "when using the existing filesystem from a disk" do

Reply via email to