Script 'mail_helper' called by obssrc
Hello community,

here is the log from the commit of package libstorage-ng for openSUSE:Factory 
checked in at 2023-06-17 22:20:03
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/libstorage-ng (Old)
 and      /work/SRC/openSUSE:Factory/.libstorage-ng.new.15902 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "libstorage-ng"

Sat Jun 17 22:20:03 2023 rev:228 rq:1093332 version:4.5.120

Changes:
--------
--- /work/SRC/openSUSE:Factory/libstorage-ng/libstorage-ng.changes      
2023-06-12 15:26:04.822920288 +0200
+++ /work/SRC/openSUSE:Factory/.libstorage-ng.new.15902/libstorage-ng.changes   
2023-06-17 22:20:12.411499592 +0200
@@ -1,0 +2,16 @@
+Thu Jun 15 14:36:02 UTC 2023 - aschn...@suse.com
+
+- merge gh#openSUSE/libstorage-ng#936
+- use correct error callback when nvme-cli is missing during probing
+- 4.5.120
+
+--------------------------------------------------------------------
+Wed Jun 14 14:14:16 UTC 2023 - aschn...@suse.com
+
+- merge gh#openSUSE/libstorage-ng#935
+- added functions to convert BcacheType and CacheMode to string
+- added pool name for bcaches
+- check range in is_usable_as_partitionable()
+- 4.5.119
+
+--------------------------------------------------------------------

Old:
----
  libstorage-ng-4.5.118.tar.xz

New:
----
  libstorage-ng-4.5.120.tar.xz

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

Other differences:
------------------
++++++ libstorage-ng.spec ++++++
--- /var/tmp/diff_new_pack.gB6FQf/_old  2023-06-17 22:20:13.695507309 +0200
+++ /var/tmp/diff_new_pack.gB6FQf/_new  2023-06-17 22:20:13.699507332 +0200
@@ -18,7 +18,7 @@
 
 %define libname %{name}1
 Name:           libstorage-ng
-Version:        4.5.118
+Version:        4.5.120
 Release:        0
 Summary:        Library for storage management
 License:        GPL-2.0-only

++++++ libstorage-ng-4.5.118.tar.xz -> libstorage-ng-4.5.120.tar.xz ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/libstorage-ng-4.5.118/LIBVERSION 
new/libstorage-ng-4.5.120/LIBVERSION
--- old/libstorage-ng-4.5.118/LIBVERSION        2023-06-12 11:37:48.000000000 
+0200
+++ new/libstorage-ng-4.5.120/LIBVERSION        2023-06-15 16:36:02.000000000 
+0200
@@ -1 +1 @@
-1.88.0
+1.89.0
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/libstorage-ng-4.5.118/VERSION 
new/libstorage-ng-4.5.120/VERSION
--- old/libstorage-ng-4.5.118/VERSION   2023-06-12 11:37:48.000000000 +0200
+++ new/libstorage-ng-4.5.120/VERSION   2023-06-15 16:36:02.000000000 +0200
@@ -1 +1 @@
-4.5.118
+4.5.120
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/libstorage-ng-4.5.118/storage/Devices/Bcache.cc 
new/libstorage-ng-4.5.120/storage/Devices/Bcache.cc
--- old/libstorage-ng-4.5.118/storage/Devices/Bcache.cc 2023-06-12 
11:37:48.000000000 +0200
+++ new/libstorage-ng-4.5.120/storage/Devices/Bcache.cc 2023-06-15 
16:36:02.000000000 +0200
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) [2016-2019] SUSE LLC
+ * Copyright (c) [2016-2023] SUSE LLC
  *
  * All Rights Reserved.
  *
@@ -30,6 +30,20 @@
     using namespace std;
 
 
+    string
+    get_bcache_type_name(BcacheType bcache_type)
+    {
+       return toString(bcache_type);
+    }
+
+
+    string
+    get_cache_mode_name(CacheMode cache_mode)
+    {
+       return toString(cache_mode);
+    }
+
+
     Bcache*
     Bcache::create(Devicegraph* devicegraph, const string& name)
     {
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/libstorage-ng-4.5.118/storage/Devices/Bcache.h 
new/libstorage-ng-4.5.120/storage/Devices/Bcache.h
--- old/libstorage-ng-4.5.118/storage/Devices/Bcache.h  2023-06-12 
11:37:48.000000000 +0200
+++ new/libstorage-ng-4.5.120/storage/Devices/Bcache.h  2023-06-15 
16:36:02.000000000 +0200
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) [2016-2020] SUSE LLC
+ * Copyright (c) [2016-2023] SUSE LLC
  *
  * All Rights Reserved.
  *
@@ -45,6 +45,14 @@
 
 
     /**
+     * Convert the BcacheType bcache_type to a string.
+     *
+     * @see BcacheType
+     */
+    std::string get_bcache_type_name(BcacheType bcache_type);
+
+
+    /**
      * The Cache mode attribute.
      */
     enum class CacheMode {
@@ -53,6 +61,14 @@
 
 
     /**
+     * Convert the CacheMode cache_mode to a string.
+     *
+     * @see CacheMode
+     */
+    std::string get_cache_mode_name(CacheMode cache_mode);
+
+
+    /**
      * A <a 
href="https://www.kernel.org/doc/Documentation/bcache.txt";>bcache</a> device.
      *
      * Bcache technology supports two kinds of bcache devices. Bcache devices 
with a backing
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/libstorage-ng-4.5.118/storage/Devices/BcacheCsetImpl.h 
new/libstorage-ng-4.5.120/storage/Devices/BcacheCsetImpl.h
--- old/libstorage-ng-4.5.118/storage/Devices/BcacheCsetImpl.h  2023-06-12 
11:37:48.000000000 +0200
+++ new/libstorage-ng-4.5.120/storage/Devices/BcacheCsetImpl.h  2023-06-15 
16:36:02.000000000 +0200
@@ -45,8 +45,7 @@
     {
     public:
 
-       Impl()
-           : Device::Impl(), uuid() {}
+       Impl() : Device::Impl() {}
 
        Impl(const xmlNode* node);
 
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/libstorage-ng-4.5.118/storage/Devices/BcacheImpl.cc 
new/libstorage-ng-4.5.120/storage/Devices/BcacheImpl.cc
--- old/libstorage-ng-4.5.118/storage/Devices/BcacheImpl.cc     2023-06-12 
11:37:48.000000000 +0200
+++ new/libstorage-ng-4.5.120/storage/Devices/BcacheImpl.cc     2023-06-15 
16:36:02.000000000 +0200
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) [2016-2020] SUSE LLC
+ * Copyright (c) [2016-2023] SUSE LLC
  *
  * All Rights Reserved.
  *
@@ -70,6 +70,7 @@
        "writethrough", "writeback", "writearound", "none"
     });
 
+
     Bcache::Impl::Impl(const string& name, BcacheType type)
        : Partitionable::Impl(name), cache_mode(CacheMode::NONE),
          sequential_cutoff(0), type(type), writeback_percent(0)
@@ -199,6 +200,16 @@
     }
 
 
+    string
+    Bcache::Impl::pool_name() const
+    {
+       if (!is_usable_as_partitionable())
+           return "";
+
+       return "bcaches";
+    }
+
+
     static bool
     is_backed(Prober& prober, const string& short_name)
     {
@@ -258,7 +269,7 @@
 
 
     // mapping between human string of libstorage-ng and bcache sysfs sizes
-    static const map<std::string, unsigned long long> size_mapping = {
+    static const map<string, unsigned long long> size_mapping = {
        { "k", KiB },
        { "M", MiB },
        { "G", GiB }
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/libstorage-ng-4.5.118/storage/Devices/BcacheImpl.h 
new/libstorage-ng-4.5.120/storage/Devices/BcacheImpl.h
--- old/libstorage-ng-4.5.118/storage/Devices/BcacheImpl.h      2023-06-12 
11:37:48.000000000 +0200
+++ new/libstorage-ng-4.5.120/storage/Devices/BcacheImpl.h      2023-06-15 
16:36:02.000000000 +0200
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) [2016-2019] SUSE LLC
+ * Copyright (c) [2016-2023] SUSE LLC
  *
  * All Rights Reserved.
  *
@@ -96,7 +96,7 @@
 
        BcacheType get_type() const { return type; }
 
-       virtual string pool_name() const override { return ""; }
+       virtual string pool_name() const override;
 
        CacheMode get_cache_mode() const { return cache_mode; }
        void set_cache_mode(CacheMode mode) { cache_mode = mode; }
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/libstorage-ng-4.5.118/storage/Devices/DasdImpl.cc 
new/libstorage-ng-4.5.120/storage/Devices/DasdImpl.cc
--- old/libstorage-ng-4.5.118/storage/Devices/DasdImpl.cc       2023-06-12 
11:37:48.000000000 +0200
+++ new/libstorage-ng-4.5.120/storage/Devices/DasdImpl.cc       2023-06-15 
16:36:02.000000000 +0200
@@ -126,6 +126,9 @@
     bool
     Dasd::Impl::is_usable_as_partitionable() const
     {
+       if (!Partitionable::Impl::is_usable_as_partitionable())
+           return false;
+
        if (type == DasdType::ECKD && (format == DasdFormat::CDL || format == 
DasdFormat::LDL))
            return true;
 
@@ -139,6 +142,9 @@
     string
     Dasd::Impl::pool_name() const
     {
+       if (!is_usable_as_partitionable())
+           return "";
+
        if (get_type() == DasdType::ECKD)
        {
            if (get_format() == DasdFormat::CDL)
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/libstorage-ng-4.5.118/storage/Devices/DiskImpl.cc 
new/libstorage-ng-4.5.120/storage/Devices/DiskImpl.cc
--- old/libstorage-ng-4.5.118/storage/Devices/DiskImpl.cc       2023-06-12 
11:37:48.000000000 +0200
+++ new/libstorage-ng-4.5.120/storage/Devices/DiskImpl.cc       2023-06-15 
16:36:02.000000000 +0200
@@ -124,6 +124,9 @@
     bool
     Disk::Impl::is_usable_as_partitionable() const
     {
+       if (!Partitionable::Impl::is_usable_as_partitionable())
+           return false;
+
        return zone_model != ZoneModel::HOST_MANAGED;
     }
 
@@ -213,10 +216,18 @@
 
        if (is_nvme())
        {
-           system_info.getCmdNvmeList();       // so far just for logging
-           const CmdNvmeListSubsys& cmd_nvme_list_subsys = 
system_info.getCmdNvmeListSubsys();
+           try
+           {
+               system_info.getCmdNvmeList();   // so far just for logging
+               const CmdNvmeListSubsys& cmd_nvme_list_subsys = 
system_info.getCmdNvmeListSubsys();
 
-           transport = cmd_nvme_list_subsys.get_transport(get_name(), 
system_info);
+               transport = cmd_nvme_list_subsys.get_transport(get_name(), 
system_info);
+           }
+           catch (const Exception& exception)
+           {
+               // TRANSLATORS: error message
+               prober.handle(exception, sformat(_("Probing disk %s failed"), 
get_name()), UF_NVME);
+           }
        }
        else if (is_pmem() || is_brd())
        {
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/libstorage-ng-4.5.118/storage/Devices/PartitionableImpl.h 
new/libstorage-ng-4.5.120/storage/Devices/PartitionableImpl.h
--- old/libstorage-ng-4.5.118/storage/Devices/PartitionableImpl.h       
2023-06-12 11:37:48.000000000 +0200
+++ new/libstorage-ng-4.5.120/storage/Devices/PartitionableImpl.h       
2023-06-15 16:36:02.000000000 +0200
@@ -49,7 +49,7 @@
        unsigned int get_range() const { return range; }
        void set_range(unsigned int range) { Impl::range = range; }
 
-       virtual bool is_usable_as_partitionable() const { return true; }
+       virtual bool is_usable_as_partitionable() const { return range > 1; }
 
        PtType get_default_partition_table_type() const;
 
@@ -74,6 +74,9 @@
 
        virtual string partition_name(int number) const;
 
+       /**
+        * Returns empty string iff no pool name applies.
+        */
        virtual string pool_name() const = 0;
 
        virtual bool equal(const Device::Impl& rhs) const override;

Reply via email to