Script 'mail_helper' called by obssrc
Hello community,

here is the log from the commit of package crmsh for openSUSE:Factory checked 
in at 2022-12-30 11:08:50
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/crmsh (Old)
 and      /work/SRC/openSUSE:Factory/.crmsh.new.1563 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "crmsh"

Fri Dec 30 11:08:50 2022 rev:274 rq:1045770 version:4.4.1+20221228.326c28fd

Changes:
--------
--- /work/SRC/openSUSE:Factory/crmsh/crmsh.changes      2022-12-27 
11:55:12.703364570 +0100
+++ /work/SRC/openSUSE:Factory/.crmsh.new.1563/crmsh.changes    2022-12-30 
11:09:05.937305235 +0100
@@ -1,0 +2,15 @@
+Wed Dec 28 09:46:54 UTC 2022 - xli...@suse.com
+
+- Update to version 4.4.1+20221228.326c28fd:
+  * Dev: report: 'crm report' to collect journal.log in microseconds
+  * Dev: doc: Add deprecated note for 'crm configure erase'
+  * Dev: ui_configure: Deprecate configure erase sub-command
+
+-------------------------------------------------------------------
+Wed Dec 28 04:00:43 UTC 2022 - xli...@suse.com
+
+- Update to version 4.4.1+20221228.7c16362c:
+  * Dev: unittest: Adjust unit test for previous changes
+  * Dev: ui_cluster: Improve cluster start/stop INFO
+
+-------------------------------------------------------------------

Old:
----
  crmsh-4.4.1+20221227.bd7d326e.tar.bz2

New:
----
  crmsh-4.4.1+20221228.326c28fd.tar.bz2

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

Other differences:
------------------
++++++ crmsh.spec ++++++
--- /var/tmp/diff_new_pack.sO4leb/_old  2022-12-30 11:09:06.445308271 +0100
+++ /var/tmp/diff_new_pack.sO4leb/_new  2022-12-30 11:09:06.449308296 +0100
@@ -36,7 +36,7 @@
 Summary:        High Availability cluster command-line interface
 License:        GPL-2.0-or-later
 Group:          %{pkg_group}
-Version:        4.4.1+20221227.bd7d326e
+Version:        4.4.1+20221228.326c28fd
 Release:        0
 URL:            http://crmsh.github.io
 Source0:        %{name}-%{version}.tar.bz2

++++++ _servicedata ++++++
--- /var/tmp/diff_new_pack.sO4leb/_old  2022-12-30 11:09:06.497308582 +0100
+++ /var/tmp/diff_new_pack.sO4leb/_new  2022-12-30 11:09:06.497308582 +0100
@@ -9,7 +9,7 @@
 </service>
 <service name="tar_scm">
   <param name="url">https://github.com/ClusterLabs/crmsh.git</param>
-  <param 
name="changesrevision">bd7d326e7b0c684c3f93a95851474fcb263a5689</param>
+  <param 
name="changesrevision">326c28fd1b60f5e1a0c71006e799989869431a75</param>
 </service>
 </servicedata>
 (No newline at EOF)

++++++ crmsh-4.4.1+20221227.bd7d326e.tar.bz2 -> 
crmsh-4.4.1+20221228.326c28fd.tar.bz2 ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/crmsh-4.4.1+20221227.bd7d326e/crmsh/report/utillib.py 
new/crmsh-4.4.1+20221228.326c28fd/crmsh/report/utillib.py
--- old/crmsh-4.4.1+20221227.bd7d326e/crmsh/report/utillib.py   2022-12-27 
09:04:26.000000000 +0100
+++ new/crmsh-4.4.1+20221228.326c28fd/crmsh/report/utillib.py   2022-12-28 
10:33:36.000000000 +0100
@@ -342,7 +342,7 @@
 
     logger.debug("journalctl from: '%d' until: '%d' from_time: '%s' to_time: 
'%s' > %s",
               from_t, to_t, from_time, to_time, outf)
-    cmd = 'journalctl -o short-iso --since "%s" --until "%s" --no-pager | tail 
-n +2' % \
+    cmd = 'journalctl -o short-iso-precise --since "%s" --until "%s" 
--no-pager | tail -n +2' % \
           (from_time, to_time)
     crmutils.str2file(get_command_info(cmd)[1], outf)
 
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/crmsh-4.4.1+20221227.bd7d326e/crmsh/ui_cluster.py 
new/crmsh-4.4.1+20221228.326c28fd/crmsh/ui_cluster.py
--- old/crmsh-4.4.1+20221227.bd7d326e/crmsh/ui_cluster.py       2022-12-27 
09:04:26.000000000 +0100
+++ new/crmsh-4.4.1+20221228.326c28fd/crmsh/ui_cluster.py       2022-12-28 
10:33:36.000000000 +0100
@@ -90,7 +90,7 @@
     @command.skill_level('administrator')
     def do_start(self, context, *args):
         '''
-        Starts the cluster services on all nodes or specific node(s)
+        Starts the cluster stack on all nodes or specific node(s)
         '''
         service_check_list = ["pacemaker.service"]
         start_qdevice = False
@@ -101,7 +101,7 @@
         node_list = parse_option_for_nodes(context, *args)
         for node in node_list[:]:
             if all([utils.service_is_active(srv, remote_addr=node) for srv in 
service_check_list]):
-                logger.info("Cluster services already started on 
{}".format(node))
+                logger.info("The cluster stack already started on 
{}".format(node))
                 node_list.remove(node)
         if not node_list:
             return
@@ -112,25 +112,25 @@
         if start_qdevice:
             qdevice.QDevice.check_qdevice_vote()
         for node in node_list:
-            logger.info("Cluster services started on {}".format(node))
+            logger.info("The cluster stack started on {}".format(node))
 
     @command.skill_level('administrator')
     def do_stop(self, context, *args):
         '''
-        Stops the cluster services on all nodes or specific node(s)
+        Stops the cluster stack on all nodes or specific node(s)
         '''
         node_list = parse_option_for_nodes(context, *args)
         for node in node_list[:]:
             if not utils.service_is_active("corosync.service", 
remote_addr=node):
                 if utils.service_is_active("sbd.service", remote_addr=node):
                     utils.stop_service("corosync", remote_addr=node)
-                    logger.info("Cluster services stopped on {}".format(node))
+                    logger.info("The cluster stack stopped on {}".format(node))
                 else:
-                    logger.info("Cluster services already stopped on 
{}".format(node))
+                    logger.info("The cluster stack already stopped on 
{}".format(node))
                 node_list.remove(node)
             elif not utils.service_is_active("pacemaker.service", 
remote_addr=node):
                 utils.stop_service("corosync", remote_addr=node)
-                logger.info("Cluster services stopped on {}".format(node))
+                logger.info("The cluster stack stopped on {}".format(node))
                 node_list.remove(node)
         if not node_list:
             return
@@ -158,12 +158,12 @@
         node_list = utils.stop_service("corosync", node_list=node_list)
 
         for node in node_list:
-            logger.info("Cluster services stopped on {}".format(node))
+            logger.info("The cluster stack stopped on {}".format(node))
 
     @command.skill_level('administrator')
     def do_restart(self, context, *args):
         '''
-        Restarts the cluster services on all nodes or specific node(s)
+        Restarts the cluster stack on all nodes or specific node(s)
         '''
         parse_option_for_nodes(context, *args)
         self.do_stop(context, *args)
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/crmsh-4.4.1+20221227.bd7d326e/crmsh/ui_configure.py 
new/crmsh-4.4.1+20221228.326c28fd/crmsh/ui_configure.py
--- old/crmsh-4.4.1+20221227.bd7d326e/crmsh/ui_configure.py     2022-12-27 
09:04:26.000000000 +0100
+++ new/crmsh-4.4.1+20221228.326c28fd/crmsh/ui_configure.py     2022-12-28 
10:33:36.000000000 +0100
@@ -866,6 +866,9 @@
     @command.completers(compl.choice(['nodes']))
     def do_erase(self, context, nodes=None):
         "usage: erase [nodes]"
+        if not options.regression_tests:
+            logger.warning("`crm configure erase` is deprecated. The 
replacement could be `crm cluster remove [node]`")
+            return True
         cib_factory.ensure_cib_updated()
         if nodes is None:
             return cib_factory.erase()
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/crmsh-4.4.1+20221227.bd7d326e/doc/crm.8.adoc 
new/crmsh-4.4.1+20221228.326c28fd/doc/crm.8.adoc
--- old/crmsh-4.4.1+20221227.bd7d326e/doc/crm.8.adoc    2022-12-27 
09:04:26.000000000 +0100
+++ new/crmsh-4.4.1+20221228.326c28fd/doc/crm.8.adoc    2022-12-28 
10:33:36.000000000 +0100
@@ -2904,6 +2904,12 @@
 [[cmdhelp_configure_erase,erase the CIB]]
 ==== `erase`
 
+.Deprecation note
+****************************
+`crm configure erase` is deprecated.
+The replacement could be `crm cluster remove [node]`
+****************************
+
 The `erase` clears all configuration. Apart from nodes. To remove
 nodes, you have to specify an additional keyword `nodes`.
 
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/crmsh-4.4.1+20221227.bd7d326e/test/unittests/test_ui_cluster.py 
new/crmsh-4.4.1+20221228.326c28fd/test/unittests/test_ui_cluster.py
--- old/crmsh-4.4.1+20221227.bd7d326e/test/unittests/test_ui_cluster.py 
2022-12-27 09:04:26.000000000 +0100
+++ new/crmsh-4.4.1+20221228.326c28fd/test/unittests/test_ui_cluster.py 
2022-12-28 10:33:36.000000000 +0100
@@ -52,8 +52,8 @@
             mock.call("pacemaker.service", remote_addr="node2")
             ])
         mock_info.assert_has_calls([
-            mock.call("Cluster services already started on node1"),
-            mock.call("Cluster services already started on node2")
+            mock.call("The cluster stack already started on node1"),
+            mock.call("The cluster stack already started on node2")
             ])
 
     @mock.patch('crmsh.qdevice.QDevice.check_qdevice_vote')
@@ -78,7 +78,7 @@
             ])
         mock_start.assert_called_once_with("corosync-qdevice", 
node_list=["node1"])
         mock_qdevice_configured.assert_called_once_with()
-        mock_info.assert_called_once_with("Cluster services started on node1")
+        mock_info.assert_called_once_with("The cluster stack started on node1")
 
     @mock.patch('logging.Logger.info')
     @mock.patch('crmsh.utils.service_is_active')
@@ -92,7 +92,7 @@
             mock.call("corosync.service", remote_addr="node1"),
             mock.call("sbd.service", remote_addr="node1")
             ])
-        mock_info.assert_called_once_with("Cluster services already stopped on 
node1")
+        mock_info.assert_called_once_with("The cluster stack already stopped 
on node1")
 
     @mock.patch('logging.Logger.debug')
     @mock.patch('logging.Logger.info')
@@ -126,6 +126,6 @@
             mock.call("corosync-qdevice.service", node_list=["node1"]),
             mock.call("corosync", node_list=["node1"])
             ])
-        mock_info.assert_called_once_with("Cluster services stopped on node1")
+        mock_info.assert_called_once_with("The cluster stack stopped on node1")
         mock_debug.assert_called_once_with("Quorum is lost; Set 
enable_quorum_fencing=0 and enable_quorum_lockspace=0 for dlm")
         mock_check.assert_called_once_with(mock_get_dc, wait_timeout=25)

Reply via email to