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