Script 'mail_helper' called by obssrc
Hello community,

here is the log from the commit of package crmsh for openSUSE:Factory checked 
in at 2024-01-23 22:57:23
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Comparing /work/SRC/openSUSE:Factory/crmsh (Old)
 and      /work/SRC/openSUSE:Factory/.crmsh.new.16006 (New)
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++

Package is "crmsh"

Tue Jan 23 22:57:23 2024 rev:321 rq:1141003 version:4.6.0+20240123.680089b0

Changes:
--------
--- /work/SRC/openSUSE:Factory/crmsh/crmsh.changes      2024-01-09 
20:50:48.090724770 +0100
+++ /work/SRC/openSUSE:Factory/.crmsh.new.16006/crmsh.changes   2024-01-23 
22:57:36.307620015 +0100
@@ -1,0 +2,16 @@
+Tue Jan 23 08:50:35 UTC 2024 - xli...@suse.com
+
+- Update to version 4.6.0+20240123.680089b0:
+  * Dev: unittest: Adjust unit test for previous changes
+  * Dev: testcases: Adjust original regression test cases
+  * Dev: behave: Adjust functional test for previous change
+  * Fix: parse: Automatically append 's' as default time unit for timeout and 
interval (#1304)
+  * Dev: ui_configure: Update the operation id after the monitor interval 
changed
+
+-------------------------------------------------------------------
+Tue Jan 09 09:02:11 UTC 2024 - xli...@suse.com
+
+- Update to version 4.6.0+20240109.1eb53276:
+  * Dev: ChangeLog: update ChangeLog for release 4.6.0
+
+-------------------------------------------------------------------

Old:
----
  crmsh-4.6.0+20240109.288ef6c4.tar.bz2

New:
----
  crmsh-4.6.0+20240123.680089b0.tar.bz2

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

Other differences:
------------------
++++++ crmsh.spec ++++++
--- /var/tmp/diff_new_pack.yUiwCL/_old  2024-01-23 22:57:36.871640637 +0100
+++ /var/tmp/diff_new_pack.yUiwCL/_new  2024-01-23 22:57:36.871640637 +0100
@@ -36,7 +36,7 @@
 Summary:        High Availability cluster command-line interface
 License:        GPL-2.0-or-later
 Group:          %{pkg_group}
-Version:        4.6.0+20240109.288ef6c4
+Version:        4.6.0+20240123.680089b0
 Release:        0
 URL:            http://crmsh.github.io
 Source0:        %{name}-%{version}.tar.bz2

++++++ _servicedata ++++++
--- /var/tmp/diff_new_pack.yUiwCL/_old  2024-01-23 22:57:36.907641954 +0100
+++ /var/tmp/diff_new_pack.yUiwCL/_new  2024-01-23 22:57:36.907641954 +0100
@@ -9,7 +9,7 @@
 </service>
 <service name="tar_scm">
   <param name="url">https://github.com/ClusterLabs/crmsh.git</param>
-  <param 
name="changesrevision">5f1fb4db03cceb407e529c1c369fb0472cc06bb6</param>
+  <param 
name="changesrevision">680089b02364187fb8a0ac27819d07ac5dc4e273</param>
 </service>
 </servicedata>
 (No newline at EOF)

++++++ crmsh-4.6.0+20240109.288ef6c4.tar.bz2 -> 
crmsh-4.6.0+20240123.680089b0.tar.bz2 ++++++
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/crmsh-4.6.0+20240109.288ef6c4/ChangeLog 
new/crmsh-4.6.0+20240123.680089b0/ChangeLog
--- old/crmsh-4.6.0+20240109.288ef6c4/ChangeLog 2024-01-09 04:56:33.000000000 
+0100
+++ new/crmsh-4.6.0+20240123.680089b0/ChangeLog 2024-01-23 09:26:56.000000000 
+0100
@@ -1,3 +1,9 @@
+* Tue Jan 9 2024 Xin Liang <xli...@suse.com>
+- Release 4.6.0
+- Fix: report: Unable to gather log files that are in the syslog format 
(bsc#1218491)
+- Dev: ui_corosync: Add a completer for corosync.set to enumerate all current 
paths
+- Dev: bootstrap: Assign hosts with _context.node_list_in_cluster in 
join_ssh_merge (bsc#1218331)
+
 * Fri Dec 22 2023 Xin Liang <xli...@suse.com>
 - Release 4.6.0 rc2
 - Dev: ui_cluster: Move --use-ssh-agent to optional arguments
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/crmsh-4.6.0+20240109.288ef6c4/crmsh/parse.py 
new/crmsh-4.6.0+20240123.680089b0/crmsh/parse.py
--- old/crmsh-4.6.0+20240109.288ef6c4/crmsh/parse.py    2024-01-09 
04:56:33.000000000 +0100
+++ new/crmsh-4.6.0+20240123.680089b0/crmsh/parse.py    2024-01-23 
09:26:56.000000000 +0100
@@ -12,7 +12,7 @@
 from . import schema
 from .utils import keyword_cmp, verify_boolean, lines2cli
 from .utils import get_boolean, olist, canonical_boolean
-from .utils import handle_role_for_ocf_1_1, compatible_role
+from .utils import handle_role_for_ocf_1_1, compatible_role, 
add_time_unit_if_needed
 from . import xmlutil
 from . import log
 
@@ -700,9 +700,9 @@
                         constants.DEFAULT_INTERVAL_IN_ACTION
                 adv_timeout = extract_advised_value(action_advised_attr_dict, 
action, 'timeout', op_node.get('role'))
                 if op_node.get('interval') is None:
-                    op_node.set('interval', adv_interval)
+                    op_node.set('interval', 
add_time_unit_if_needed(adv_interval))
                 if op_node.get('timeout') is None and adv_timeout:
-                    op_node.set('timeout', adv_timeout)
+                    op_node.set('timeout', 
add_time_unit_if_needed(adv_timeout))
                 configured_action_list.append(action)
 
         for action in action_advised_attr_dict:
@@ -717,9 +717,15 @@
                     for k, v in v_dict.items():
                         # set normal attributes
                         if k in constants.ADVISED_KEY_LIST:
+                            if k in ('interval', 'timeout'):
+                                v = add_time_unit_if_needed(v)
                             op_node.set(k, handle_role_for_ocf_1_1(v))
                     operations_node.append(op_node)
             else:
+                for k, v in value.items():
+                    if k in ('interval', 'timeout'):
+                        v = add_time_unit_if_needed(v)
+                        value.update({k: v})
                 op_node = xmlutil.new('op', name=action, **value)
                 operations_node.append(op_node)
 
@@ -774,10 +780,14 @@
         valid_attrs = validator.op_attributes()
         inst_attrs = None
         for nvp in all_attrs:
-            if nvp.get('name') in valid_attrs:
+            name = nvp.get('name')
+            if name in valid_attrs:
                 if inst_attrs is not None:
-                    self.err("Attribute order error: {} must appear before any 
instance attribute".format(nvp.get('name')))
-                node.set(nvp.get('name'), nvp.get('value'))
+                    self.err(f"Attribute order error: {name} must appear 
before any instance attribute")
+                value = nvp.get('value')
+                if name in ('interval', 'timeout'):
+                    value = add_time_unit_if_needed(value)
+                node.set(name, value)
             else:
                 if inst_attrs is None:
                     inst_attrs = xmlutil.child(node, 'instance_attributes')
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/crmsh-4.6.0+20240109.288ef6c4/crmsh/ui_configure.py 
new/crmsh-4.6.0+20240123.680089b0/crmsh/ui_configure.py
--- old/crmsh-4.6.0+20240109.288ef6c4/crmsh/ui_configure.py     2024-01-09 
04:56:33.000000000 +0100
+++ new/crmsh-4.6.0+20240123.680089b0/crmsh/ui_configure.py     2024-01-23 
09:26:56.000000000 +0100
@@ -648,14 +648,23 @@
                 context.fatal_error("Operation \"{}\" not found for resource 
{}".format(op_type, obj_id))
             if len(op_res) > 1:
                 context.fatal_error("Should specify interval of 
{}".format(op_type))
+            if name in ('interval', 'timeout'):
+                value = utils.add_time_unit_if_needed(value)
+            if name == 'interval':
+                op_res[0].set('id', f'{obj_id}-{op_type}-{value}')
             op_res[0].set(name, value)
 
         # Use case for: set id.op_type.interval.name value
         if len(other_path_list) == 3:
             op_type, iv, name = other_path_list
-            op_res = 
rsc.node.xpath(".//operations/op[@id='{}-{}-{}']".format(obj_id, op_type, iv))
+            iv = iv[:-1] if utils.time_value_with_unit(iv) else iv
+            # Search for IDs both with and without the associated time unit
+            op_res = 
rsc.node.xpath(f".//operations/op[@id='{obj_id}-{op_type}-{iv}' or 
@id='{obj_id}-{op_type}-{iv}s']")
             if not op_res:
-                context.fatal_error("Operation \"{}\" interval \"{}\" not 
found for resource {}".format(op_type, iv, obj_id))
+                context.fatal_error(f"Operation \"{op_type}\" interval 
\"{iv}s\" not found for resource {obj_id}")
+            if name == 'interval':
+                value = utils.add_time_unit_if_needed(value)
+                op_res[0].set('id', f'{obj_id}-{op_type}-{value}')
             op_res[0].set(name, value)
 
         rsc.set_updated()
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/crmsh-4.6.0+20240109.288ef6c4/crmsh/utils.py 
new/crmsh-4.6.0+20240123.680089b0/crmsh/utils.py
--- old/crmsh-4.6.0+20240109.288ef6c4/crmsh/utils.py    2024-01-09 
04:56:33.000000000 +0100
+++ new/crmsh-4.6.0+20240123.680089b0/crmsh/utils.py    2024-01-23 
09:26:56.000000000 +0100
@@ -3147,4 +3147,18 @@
         return ""
 
     return data
+
+
+def add_time_unit_if_needed(time_value):
+    """
+    Add time unit if needed
+    """
+    return "{}s".format(time_value) if not time_value_with_unit(time_value) 
else time_value
+
+
+def time_value_with_unit(time_value):
+    """
+    Check if the time value contains unit
+    """
+    return re.search(r'^\d+[a-z]+$', time_value) is not None
 # vim:ts=4:sw=4:et:
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/crmsh-4.6.0+20240109.288ef6c4/test/features/resource_set.feature 
new/crmsh-4.6.0+20240123.680089b0/test/features/resource_set.feature
--- old/crmsh-4.6.0+20240109.288ef6c4/test/features/resource_set.feature        
2024-01-09 04:56:33.000000000 +0100
+++ new/crmsh-4.6.0+20240123.680089b0/test/features/resource_set.feature        
2024-01-23 09:26:56.000000000 +0100
@@ -29,7 +29,7 @@
     When    Try "crm configure set d.name value"
     Then    Except "ERROR: configure.set: Attribute not found: d.name"
     When    Try "crm configure set d.monitor.100.timeout 10"
-    Then    Except "ERROR: configure.set: Operation "monitor" interval "100" 
not found for resource d"
+    Then    Except "ERROR: configure.set: Operation "monitor" interval "100s" 
not found for resource d"
     When    Try "crm configure set s.monitor.interval 20"
     Then    Except "ERROR: configure.set: Should specify interval of monitor"
 
@@ -43,6 +43,25 @@
     When    Run "crm configure set s.monitor.5s.interval 20s" on "hanode1"
     And     Run "crm configure show s" on "hanode1"
     Then    Expected "interval=20s" in stdout
+    When    Run "crm configure set s.monitor.20s.interval 60s" on "hanode1"
+    And     Run "crm configure show s" on "hanode1"
+    Then    Expected "interval=60s" in stdout
+    When    Run "crm configure set s.monitor.60.interval 50" on "hanode1"
+    And     Run "crm configure show s" on "hanode1"
+    Then    Expected "interval=50s" in stdout
+
+    When    Run "crm configure primitive d2 Dummy op monitor interval=10 
timeout=20 op start timeout=66" on "hanode1"
+    And     Run "crm configure show d2" on "hanode1"
+    Then    Expected "monitor interval=10s timeout=20s" in stdout
+    When    Run "crm configure show d2" on "hanode1"
+    Then    Expected "start timeout=66s" in stdout
+    When    Run "crm configure set d2.monitor.interval 33" on "hanode1"
+    And     Run "crm configure show d2" on "hanode1"
+    Then    Expected "interval=33s" in stdout
+    When    Run "crm configure set d2.monitor.33s.interval 50" on "hanode1"
+    And     Run "crm configure show d2" on "hanode1"
+    Then    Expected "interval=50s" in stdout
+
     When    Run "crm configure set op-options.timeout 101" on "hanode1"
     And     Run "crm configure show op-options" on "hanode1"
     Then    Expected "timeout=101" in stdout
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' old/crmsh-4.6.0+20240109.288ef6c4/test/testcases/acl.exp 
new/crmsh-4.6.0+20240123.680089b0/test/testcases/acl.exp
--- old/crmsh-4.6.0+20240109.288ef6c4/test/testcases/acl.exp    2024-01-09 
04:56:33.000000000 +0100
+++ new/crmsh-4.6.0+20240123.680089b0/test/testcases/acl.exp    2024-01-23 
09:26:56.000000000 +0100
@@ -40,7 +40,7 @@
        meta target-role=Started requires=nothing \
        op start timeout=60s interval=0s \
        op monitor interval=60m timeout=60s \
-       op stop timeout=15 interval=0s
+       op stop timeout=15s interval=0s
 property cib-bootstrap-options: \
        enable-acl=true
 role basic-read \
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/crmsh-4.6.0+20240109.288ef6c4/test/testcases/bugs.exp 
new/crmsh-4.6.0+20240123.680089b0/test/testcases/bugs.exp
--- old/crmsh-4.6.0+20240109.288ef6c4/test/testcases/bugs.exp   2024-01-09 
04:56:33.000000000 +0100
+++ new/crmsh-4.6.0+20240123.680089b0/test/testcases/bugs.exp   2024-01-23 
09:26:56.000000000 +0100
@@ -31,9 +31,9 @@
 primitive st stonith:null \
        params hostlist=node1 \
        meta description="some description here" requires=nothing \
-       op monitor interval=60m timeout=20 \
-        op start timeout=20 interval=0s \
-        op stop timeout=15 interval=0s
+       op monitor interval=60m timeout=20s \
+        op start timeout=20s interval=0s \
+        op stop timeout=15s interval=0s
 primitive p4 Dummy \
         op monitor timeout=20s interval=10s \
         op start timeout=20s interval=0s \
@@ -69,9 +69,9 @@
 primitive st stonith:null \
        params hostlist=node1 \
        meta description="some description here" requires=nothing \
-       op monitor interval=60m timeout=20 \
-        op start timeout=20 interval=0s \
-        op stop timeout=15 interval=0s
+       op monitor interval=60m timeout=20s \
+        op start timeout=20s interval=0s \
+        op stop timeout=15s interval=0s
 primitive p4 Dummy \
         op monitor timeout=20s interval=10s \
         op start timeout=20s interval=0s \
@@ -170,9 +170,9 @@
 primitive st stonith:null \
        params hostlist=node1 \
        meta description="some description here" requires=nothing \
-       op monitor interval=60m timeout=20 \
-        op start timeout=20 interval=0s \
-        op stop timeout=15 interval=0s
+       op monitor interval=60m timeout=20s \
+        op start timeout=20s interval=0s \
+        op stop timeout=15s interval=0s
 property SAPHanaSR: \
        hana_ha1_site_lss_WDF1=4
 .INP: commit
@@ -184,9 +184,9 @@
 primitive st stonith:null \
        params hostlist=node1 \
        meta description="some description here" requires=nothing \
-       op monitor interval=60m timeout=20 \
-        op start timeout=20 interval=0s \
-        op stop timeout=15 interval=0s
+       op monitor interval=60m timeout=20s \
+        op start timeout=20s interval=0s \
+        op stop timeout=15s interval=0s
 property SAPHanaSR: \
        hana_ha1_site_lss_WDF1=4
 property SAPHanaSR_2: \
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/crmsh-4.6.0+20240109.288ef6c4/test/testcases/bundle.exp 
new/crmsh-4.6.0+20240123.680089b0/test/testcases/bundle.exp
--- old/crmsh-4.6.0+20240109.288ef6c4/test/testcases/bundle.exp 2024-01-09 
04:56:33.000000000 +0100
+++ new/crmsh-4.6.0+20240123.680089b0/test/testcases/bundle.exp 2024-01-23 
09:26:56.000000000 +0100
@@ -28,7 +28,7 @@
        utilization cpu=4
 primitive dummy Dummy \
        meta target-role=Stopped \
-       op monitor interval=10 timeout=20s \
+       op monitor interval=10s timeout=20s \
        op start timeout=20s interval=0s \
        op stop timeout=20s interval=0s
 primitive st stonith:ssh \
@@ -36,12 +36,12 @@
        meta target-role=Started requires=nothing \
        op start timeout=60s interval=0s \
        op monitor interval=60m timeout=60s \
-       op stop timeout=15 interval=0s
+       op stop timeout=15s interval=0s
 primitive st2 stonith:ssh \
        params hostlist="node1 node2" \
-       op monitor timeout=20 interval=3600 \
-       op start timeout=20 interval=0s \
-       op stop timeout=15 interval=0s
+       op monitor timeout=20s interval=3600s \
+       op start timeout=20s interval=0s \
+       op stop timeout=15s interval=0s
 bundle bundle-test1 \
        docker image=test \
        network ip-range-start=10.10.10.123 port-mapping id=port1 port=80 \
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/crmsh-4.6.0+20240109.288ef6c4/test/testcases/commit.exp 
new/crmsh-4.6.0+20240123.680089b0/test/testcases/commit.exp
--- old/crmsh-4.6.0+20240109.288ef6c4/test/testcases/commit.exp 2024-01-09 
04:56:33.000000000 +0100
+++ new/crmsh-4.6.0+20240123.680089b0/test/testcases/commit.exp 2024-01-23 
09:26:56.000000000 +0100
@@ -75,9 +75,9 @@
 primitive st stonith:null \
        params hostlist=node1 \
        meta yoyo-meta="yoyo 2" requires=nothing \
-       op monitor interval=60m timeout=20 \
-       op start timeout=20 interval=0s \
-       op stop timeout=15 interval=0s
+       op monitor interval=60m timeout=20s \
+       op start timeout=20s interval=0s \
+       op stop timeout=15s interval=0s
 group g1 d1 p2
 group g2 d3
 colocation cl1 inf: g1 p3
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/crmsh-4.6.0+20240109.288ef6c4/test/testcases/confbasic-xml.exp 
new/crmsh-4.6.0+20240123.680089b0/test/testcases/confbasic-xml.exp
--- old/crmsh-4.6.0+20240109.288ef6c4/test/testcases/confbasic-xml.exp  
2024-01-09 04:56:33.000000000 +0100
+++ new/crmsh-4.6.0+20240123.680089b0/test/testcases/confbasic-xml.exp  
2024-01-23 09:26:56.000000000 +0100
@@ -33,7 +33,7 @@
         <operations>
           <op name="start" timeout="60s" interval="0s" id="st-start-0s"/>
           <op name="monitor" interval="60m" timeout="60s" id="st-monitor-60m"/>
-          <op name="stop" timeout="15" interval="0s" id="st-stop-0s"/>
+          <op name="stop" timeout="15s" interval="0s" id="st-stop-0s"/>
         </operations>
       </primitive>
       <primitive id="st2" class="stonith" type="ssh">
@@ -41,9 +41,9 @@
           <nvpair name="hostlist" value="node1 node2" 
id="st2-instance_attributes-hostlist"/>
         </instance_attributes>
         <operations>
-          <op name="monitor" timeout="20" interval="3600" 
id="st2-monitor-3600"/>
-          <op name="start" timeout="20" interval="0s" id="st2-start-0s"/>
-          <op name="stop" timeout="15" interval="0s" id="st2-stop-0s"/>
+          <op name="monitor" timeout="20s" interval="3600s" 
id="st2-monitor-3600s"/>
+          <op name="start" timeout="20s" interval="0s" id="st2-start-0s"/>
+          <op name="stop" timeout="15s" interval="0s" id="st2-stop-0s"/>
         </operations>
       </primitive>
       <group id="g1">
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/crmsh-4.6.0+20240109.288ef6c4/test/testcases/confbasic.exp 
new/crmsh-4.6.0+20240123.680089b0/test/testcases/confbasic.exp
--- old/crmsh-4.6.0+20240109.288ef6c4/test/testcases/confbasic.exp      
2024-01-09 04:56:33.000000000 +0100
+++ new/crmsh-4.6.0+20240123.680089b0/test/testcases/confbasic.exp      
2024-01-23 09:26:56.000000000 +0100
@@ -102,7 +102,7 @@
 primitive d7 Dummy \
        params rule #uname eq node1 fake=1 \
        params rule #uname eq node2 fake=2 \
-       op start interval=0 timeout=60s \
+       op start interval=0s timeout=60s \
        op_params 2: rule #uname eq node1 op_param=dummy \
        op_params 1: op_param=smart \
        op_meta 2: rule #ra-version version:gt 1.0 start-delay=120m \
@@ -134,12 +134,12 @@
        meta target-role=Started \
        op start timeout=60s interval=0s \
        op monitor interval=60m timeout=60s \
-       op stop timeout=15 interval=0s
+       op stop timeout=15s interval=0s
 primitive st2 stonith:ssh \
        params hostlist="node1 node2" \
-       op monitor timeout=20 interval=3600 \
-       op start timeout=20 interval=0s \
-       op stop timeout=15 interval=0s
+       op monitor timeout=20s interval=3600s \
+       op start timeout=20s interval=0s \
+       op stop timeout=15s interval=0s
 group g1 d1 d2
 ms m d4
 ms m5 s5
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/crmsh-4.6.0+20240109.288ef6c4/test/testcases/delete.exp 
new/crmsh-4.6.0+20240123.680089b0/test/testcases/delete.exp
--- old/crmsh-4.6.0+20240109.288ef6c4/test/testcases/delete.exp 2024-01-09 
04:56:33.000000000 +0100
+++ new/crmsh-4.6.0+20240123.680089b0/test/testcases/delete.exp 2024-01-23 
09:26:56.000000000 +0100
@@ -27,7 +27,7 @@
        meta target-role=Started \
        op start timeout=60s interval=0s \
        op monitor interval=60m timeout=60s \
-        op stop timeout=15 interval=0s
+        op stop timeout=15s interval=0s
 location d1-pref d1 100: node1
 .INP: _test
 .INP: rename d1 p1
@@ -47,7 +47,7 @@
        meta target-role=Started \
        op start timeout=60s interval=0s \
        op monitor interval=60m timeout=60s \
-        op stop timeout=15 interval=0s
+        op stop timeout=15s interval=0s
 location d1-pref p1 100: node1
 .INP: # delete primitive
 .INP: delete d2
@@ -63,7 +63,7 @@
        meta target-role=Started \
        op start timeout=60s interval=0s \
        op monitor interval=60m timeout=60s \
-       op stop timeout=15 interval=0s
+       op stop timeout=15s interval=0s
 location d1-pref p1 100: node1
 .INP: # delete primitive with constraint
 .INP: delete p1
@@ -76,7 +76,7 @@
        meta target-role=Started \
        op start timeout=60s interval=0s \
        op monitor interval=60m timeout=60s \
-       op stop timeout=15 interval=0s
+       op stop timeout=15s interval=0s
 .INP: primitive d1 ocf:pacemaker:Dummy
 .INP: location d1-pref d1 100: node1
 .INP: _test
@@ -97,7 +97,7 @@
        meta target-role=Started \
        op start timeout=60s interval=0s \
        op monitor interval=60m timeout=60s \
-       op stop timeout=15 interval=0s
+       op stop timeout=15s interval=0s
 group g1 d1
 location d1-pref g1 100: node1
 .INP: _test
@@ -114,7 +114,7 @@
        meta target-role=Started \
        op start timeout=60s interval=0s \
        op monitor interval=60m timeout=60s \
-       op stop timeout=15 interval=0s
+       op stop timeout=15s interval=0s
 location d1-pref d1 100: node1
 .INP: verify
 .INP: # delete a group which is in a clone
@@ -141,7 +141,7 @@
        meta target-role=Started \
        op start timeout=60s interval=0s \
        op monitor interval=60m timeout=60s \
-       op stop timeout=15 interval=0s
+       op stop timeout=15s interval=0s
 location d1-pref d2 100: node1
 .INP: _test
 .INP: group g1 d2 d1
@@ -168,7 +168,7 @@
        meta target-role=Started \
        op start timeout=60s interval=0s \
        op monitor interval=60m timeout=60s \
-       op stop timeout=15 interval=0s
+       op stop timeout=15s interval=0s
 location d1-pref d2 100: node1
 .INP: _test
 .INP: group g1 d2 d1
@@ -187,7 +187,7 @@
        meta target-role=Started \
        op start timeout=60s interval=0s \
        op monitor interval=60m timeout=60s \
-       op stop timeout=15 interval=0s
+       op stop timeout=15s interval=0s
 .INP: _test
 .INP: # verify
 .INP: verify
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/crmsh-4.6.0+20240109.288ef6c4/test/testcases/edit.exp 
new/crmsh-4.6.0+20240123.680089b0/test/testcases/edit.exp
--- old/crmsh-4.6.0+20240109.288ef6c4/test/testcases/edit.exp   2024-01-09 
04:56:33.000000000 +0100
+++ new/crmsh-4.6.0+20240123.680089b0/test/testcases/edit.exp   2024-01-23 
09:26:56.000000000 +0100
@@ -25,9 +25,9 @@
 primitive st stonith:null \
        params hostlist=node1 \
        meta description="some description here" requires=nothing \
-       op monitor interval=60m timeout=20 \
-       op start timeout=20 interval=0s \
-       op stop timeout=15 interval=0s
+       op monitor interval=60m timeout=20s \
+       op start timeout=20s interval=0s \
+       op stop timeout=15s interval=0s
 group g1 p1 p2
 op_defaults op-options: \
        timeout=2m
@@ -46,9 +46,9 @@
 primitive st stonith:null \
        params hostlist=node1 \
        meta description="some description here" requires=nothing \
-       op monitor interval=60m timeout=20 \
-       op start timeout=20 interval=0s \
-       op stop timeout=15 interval=0s
+       op monitor interval=60m timeout=20s \
+       op start timeout=20s interval=0s \
+       op stop timeout=15s interval=0s
 group g1 p1 p3
 op_defaults op-options: \
        timeout=2m
@@ -176,9 +176,9 @@
 primitive st stonith:null \
        params hostlist=node1 \
        meta description="some description here" requires=nothing \
-       op monitor interval=60m timeout=20 \
-       op start timeout=20 interval=0s \
-       op stop timeout=15 interval=0s
+       op monitor interval=60m timeout=20s \
+       op start timeout=20s interval=0s \
+       op stop timeout=15s interval=0s
 group g1 p1 p2 d3
 group g2 d1 d2
 clone c1 g1
@@ -305,9 +305,9 @@
 primitive st stonith:null \
        params hostlist=node1 \
        meta description="some description here" requires=nothing \
-       op monitor interval=60m timeout=20 \
-       op start timeout=20 interval=0s \
-       op stop timeout=15 interval=0s
+       op monitor interval=60m timeout=20s \
+       op start timeout=20s interval=0s \
+       op stop timeout=15s interval=0s
 group as a0 a1 a9 a2 a3 a4 a5 a6 a7 a8 aErr
 group g1 p1 p2 d3
 group g2 d1 d2
@@ -412,9 +412,9 @@
 primitive st stonith:null \
        params hostlist=node1 \
        meta description="some description here" requires=nothing \
-       op monitor interval=60m timeout=20 \
-       op start timeout=20 interval=0s \
-       op stop timeout=15 interval=0s
+       op monitor interval=60m timeout=20s \
+       op start timeout=20s interval=0s \
+       op stop timeout=15s interval=0s
 group as a0 a1 a9 a2 a3 a4 a5 a6 a7 a8 aErr
 group g1 p1 p2 d3
 group g2 d1 d2
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/crmsh-4.6.0+20240109.288ef6c4/test/testcases/file.exp 
new/crmsh-4.6.0+20240123.680089b0/test/testcases/file.exp
--- old/crmsh-4.6.0+20240109.288ef6c4/test/testcases/file.exp   2024-01-09 
04:56:33.000000000 +0100
+++ new/crmsh-4.6.0+20240123.680089b0/test/testcases/file.exp   2024-01-23 
09:26:56.000000000 +0100
@@ -20,9 +20,9 @@
         op stop timeout=20s interval=0s
 primitive st stonith:null \
        params hostlist=node1 \
-        op monitor timeout=20 interval=3600 \
-       op start timeout=20 interval=0s \
-       op stop timeout=15 interval=0s
+        op monitor timeout=20s interval=3600s \
+       op start timeout=20s interval=0s \
+       op stop timeout=15s interval=0s
 clone c1 p1 \
        meta interleave=true
 clone m1 p2 \
@@ -65,9 +65,9 @@
         op stop timeout=20s interval=0s
 primitive st stonith:null \
        params hostlist=node1 \
-        op monitor timeout=20 interval=3600 \
-       op start timeout=20 interval=0s \
-       op stop timeout=15 interval=0s
+        op monitor timeout=20s interval=3600s \
+       op start timeout=20s interval=0s \
+       op stop timeout=15s interval=0s
 property cib-bootstrap-options: \
        cluster-recheck-interval=10m
 rsc_defaults build-resource-defaults: \
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/crmsh-4.6.0+20240109.288ef6c4/test/testcases/newfeatures.exp 
new/crmsh-4.6.0+20240123.680089b0/test/testcases/newfeatures.exp
--- old/crmsh-4.6.0+20240109.288ef6c4/test/testcases/newfeatures.exp    
2024-01-09 04:56:33.000000000 +0100
+++ new/crmsh-4.6.0+20240123.680089b0/test/testcases/newfeatures.exp    
2024-01-23 09:26:56.000000000 +0100
@@ -55,7 +55,7 @@
        meta target-role=Started requires=nothing \
        op start timeout=60s interval=0s \
        op monitor interval=60m timeout=60s \
-       op stop timeout=15 interval=0s
+       op stop timeout=15s interval=0s
 tag ones l1 p1
 tag tag1 p0 p1 p2
 tag tag2 p0 p1 p2
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/crmsh-4.6.0+20240109.288ef6c4/test/testcases/rset-xml.exp 
new/crmsh-4.6.0+20240123.680089b0/test/testcases/rset-xml.exp
--- old/crmsh-4.6.0+20240109.288ef6c4/test/testcases/rset-xml.exp       
2024-01-09 04:56:33.000000000 +0100
+++ new/crmsh-4.6.0+20240123.680089b0/test/testcases/rset-xml.exp       
2024-01-23 09:26:56.000000000 +0100
@@ -12,8 +12,8 @@
         </instance_attributes>
         <operations>
           <op name="start" timeout="60s" interval="0s" id="st-start-0s"/>
-          <op name="monitor" timeout="20" interval="3600" 
id="st-monitor-3600"/>
-          <op name="stop" timeout="15" interval="0s" id="st-stop-0s"/>
+          <op name="monitor" timeout="20s" interval="3600s" 
id="st-monitor-3600s"/>
+          <op name="stop" timeout="15s" interval="0s" id="st-stop-0s"/>
         </operations>
       </primitive>
       <primitive id="d1" class="ocf" provider="pacemaker" type="Dummy">
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/crmsh-4.6.0+20240109.288ef6c4/test/testcases/rset.exp 
new/crmsh-4.6.0+20240123.680089b0/test/testcases/rset.exp
--- old/crmsh-4.6.0+20240109.288ef6c4/test/testcases/rset.exp   2024-01-09 
04:56:33.000000000 +0100
+++ new/crmsh-4.6.0+20240123.680089b0/test/testcases/rset.exp   2024-01-23 
09:26:56.000000000 +0100
@@ -57,8 +57,8 @@
 primitive st stonith:ssh \
        params hostlist=node1 \
        op start timeout=60s interval=0s \
-       op monitor timeout=20 interval=3600 \
-       op stop timeout=15 interval=0s
+       op monitor timeout=20s interval=3600s \
+       op stop timeout=15s interval=0s
 colocation c1 inf: ( d1 d3 )
 colocation c2 inf: d3 d1
 colocation c3 inf: d3 d1
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/crmsh-4.6.0+20240109.288ef6c4/test/unittests/test_bugs.py 
new/crmsh-4.6.0+20240123.680089b0/test/unittests/test_bugs.py
--- old/crmsh-4.6.0+20240109.288ef6c4/test/unittests/test_bugs.py       
2024-01-09 04:56:33.000000000 +0100
+++ new/crmsh-4.6.0+20240123.680089b0/test/unittests/test_bugs.py       
2024-01-23 09:26:56.000000000 +0100
@@ -322,7 +322,7 @@
           </rule>
         </instance_attributes>
         <operations>
-          <op id="rsc1-monitor-10" interval="10" name="monitor"/>
+          <op id="rsc1-monitor-10s" interval="10s" name="monitor"/>
         </operations>
       </primitive>'''
     data = etree.fromstring(xml)
@@ -330,7 +330,7 @@
     assert obj is not None
     data = obj.repr_cli(format_mode=-1)
     print("OUTPUT:", data)
-    exp = 'primitive rsc1 ocf:pacemaker:Dummy params rule 0: #cluster-name eq 
clusterA state="/var/run/Dummy-rsc1-clusterA" params rule 0: #cluster-name eq 
clusterB state="/var/run/Dummy-rsc1-clusterB" op monitor interval=10'
+    exp = 'primitive rsc1 ocf:pacemaker:Dummy params rule 0: #cluster-name eq 
clusterA state="/var/run/Dummy-rsc1-clusterA" params rule 0: #cluster-name eq 
clusterB state="/var/run/Dummy-rsc1-clusterB" op monitor interval=10s'
     assert data == exp
     assert obj.cli_use_validate()
 
@@ -349,7 +349,7 @@
 def test_op_role():
     xml = '''<primitive class="ocf" id="rsc2" provider="pacemaker" 
type="Dummy">
         <operations>
-          <op id="rsc2-monitor-10" interval="10" name="monitor" 
role="Stopped"/>
+          <op id="rsc2-monitor-10s" interval="10s" name="monitor" 
role="Stopped"/>
         </operations>
       </primitive>'''
     data = etree.fromstring(xml)
@@ -357,7 +357,7 @@
     assert obj is not None
     data = obj.repr_cli(format_mode=-1)
     print("OUTPUT:", data)
-    exp = 'primitive rsc2 ocf:pacemaker:Dummy op monitor interval=10 
role=Stopped'
+    exp = 'primitive rsc2 ocf:pacemaker:Dummy op monitor interval=10s 
role=Stopped'
     assert data == exp
     assert obj.cli_use_validate()
 
diff -urN '--exclude=CVS' '--exclude=.cvsignore' '--exclude=.svn' 
'--exclude=.svnignore' 
old/crmsh-4.6.0+20240109.288ef6c4/test/unittests/test_cliformat.py 
new/crmsh-4.6.0+20240123.680089b0/test/unittests/test_cliformat.py
--- old/crmsh-4.6.0+20240109.288ef6c4/test/unittests/test_cliformat.py  
2024-01-09 04:56:33.000000000 +0100
+++ new/crmsh-4.6.0+20240123.680089b0/test/unittests/test_cliformat.py  
2024-01-23 09:26:56.000000000 +0100
@@ -125,9 +125,9 @@
 def test_ordering():
     xml = """<primitive id="dummy" class="ocf" provider="pacemaker" 
type="Dummy"> \
   <operations> \
-    <op name="start" timeout="60" interval="0" id="dummy-start-0"/> \
-    <op name="stop" timeout="60" interval="0" id="dummy-stop-0"/> \
-    <op name="monitor" interval="60" timeout="30" id="dummy-monitor-60"/> \
+    <op name="start" timeout="60s" interval="0s" id="dummy-start-0s"/> \
+    <op name="stop" timeout="60s" interval="0s" id="dummy-stop-0s"/> \
+    <op name="monitor" interval="60s" timeout="30s" id="dummy-monitor-60s"/> \
   </operations> \
   <meta_attributes id="dummy-meta_attributes"> \
     <nvpair id="dummy-meta_attributes-target-role" name="target-role"
@@ -139,7 +139,7 @@
     assert_is_not_none(obj)
     data = obj.repr_cli(format_mode=-1)
     print(data)
-    exp = 'primitive dummy ocf:pacemaker:Dummy op start timeout=60 interval=0 
op stop timeout=60 interval=0 op monitor interval=60 timeout=30 meta 
target-role=Stopped'
+    exp = 'primitive dummy ocf:pacemaker:Dummy op start timeout=60s 
interval=0s op stop timeout=60s interval=0s op monitor interval=60s timeout=30s 
meta target-role=Stopped'
     assert exp == data
     assert obj.cli_use_validate()
 
@@ -151,9 +151,9 @@
 value="Stopped"/> \
   </meta_attributes> \
   <operations> \
-    <op name="start" timeout="60" interval="0" id="dummy2-start-0"/> \
-    <op name="stop" timeout="60" interval="0" id="dummy2-stop-0"/> \
-    <op name="monitor" interval="60" timeout="30" id="dummy2-monitor-60"/> \
+    <op name="start" timeout="60s" interval="0s" id="dummy2-start-0s"/> \
+    <op name="stop" timeout="60s" interval="0s" id="dummy2-stop-0s"/> \
+    <op name="monitor" interval="60s" timeout="30s" id="dummy2-monitor-60s"/> \
   </operations> \
 </primitive>"""
     data = etree.fromstring(xml)
@@ -162,8 +162,8 @@
     data = obj.repr_cli(format_mode=-1)
     print(data)
     exp = 'primitive dummy2 ocf:pacemaker:Dummy meta target-role=Stopped ' \
-          'op start timeout=60 interval=0 op stop timeout=60 interval=0 ' \
-          'op monitor interval=60 timeout=30'
+          'op start timeout=60s interval=0s op stop timeout=60s interval=0s ' \
+          'op monitor interval=60s timeout=30s'
     assert exp == data
     assert obj.cli_use_validate()
 
@@ -235,7 +235,7 @@
 
 def test_operation_rules():
     roundtrip('primitive test Dummy ' +
-              'op start interval=0 '
+              'op start interval=0s '
               'op_params 2: rule #uname eq node1 fake=fake ' +
               'op_params 1: fake=real ' +
               'op_meta 2: rule #ra-version version:gt 1.0 timeout=120s ' +

Reply via email to