Bug 2112268 - Support ISO 8601 duration specifications for pacemaker "time" properties
Summary: Support ISO 8601 duration specifications for pacemaker "time" properties
Keywords:
Status: POST
Alias: None
Product: Red Hat Enterprise Linux 9
Classification: Red Hat
Component: pcs
Version: 9.0
Hardware: All
OS: All
low
low
Target Milestone: rc
: 9.4
Assignee: Miroslav Lisik
QA Contact: cluster-qe
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2022-07-29 08:15 UTC by Tomas Jelinek
Modified: 2023-08-15 07:12 UTC (History)
9 users (show)

Fixed In Version:
Doc Type: Enhancement
Doc Text:
Feature: Support ISO 8601 duration specifications for pacemaker "time" properties. Reason: Pacemaker supports ISO 8601 duration specification since Pacemaker 2.0.4 / RHEL 8.3. Result: Pcs allows to specify values for pacemaker "time" properties according to ISO 8601 duration specification.
Clone Of: 1807636
Environment:
Last Closed:
Type: Bug
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Github ClusterLabs pcs pull 712 0 None Merged Support ISO 8601 duration specifications for pacemaker "time" properties 2023-07-21 11:13:27 UTC
Red Hat Issue Tracker RHELPLAN-129618 0 None None None 2022-07-29 08:31:05 UTC

Description Tomas Jelinek 2022-07-29 08:15:32 UTC
+++ This bug was initially created as a clone of Bug #1807636 +++

Description of problem: Expected starting in Pacemaker 2.0.4 / RHEL 8.3, Pacemaker cluster properties marked as having the "time" type (cluster-recheck-interval, etc.) will accept ISO 8601 duration specifications in addition to the current possibilities (bare number of seconds or number plus units).


Steps to Reproduce:
1. pcs property set cluster-recheck-interval=P600S

Actual results:
Error: invalid value of property: 'cluster-recheck-interval=P600S', (use --force to override)

Expected results: pcs accepts a valid ISO 8601 interval

Additional notes: This was just an easy enhancement and not something driven by user demand, so it is very low priority.

--- Additional comment from Ken Gaillot on 2020-02-26 20:45:23 CET ---

FYI, pacemaker currently accepts a number, optionally with whitespace before or after, plus optional units:
  ms or msec
  us or usec
  s or sec (or no unit specified)
  m or min
  h or hr

Anything else (including no number, unrecognized units, or anything after the units) is rejected.

Comment 3 Tomas Jelinek 2023-07-21 11:12:12 UTC
Upstream patch from community: https://github.com/ClusterLabs/pcs/commit/d22b63fd3ef9f7f4ce9a80044fa811f8b3bacf99

Comment 5 Miroslav Lisik 2023-07-24 07:40:05 UTC
Upstream commit: https://github.com/ClusterLabs/pcs/commit/d22b63fd3ef9f7f4ce9a80044fa811f8b3bacf99

Updated command:
pcs property set

All pacemaker time properties except stonith-watchdog-timeout supports values
defined in ISO 8601 duration format:
https://en.wikipedia.org/wiki/ISO_8601#Durations

Environment: A runnig cluster

Test:
# pcs property set cluster-delay=P60S
# pcs property config cluster-delay
Cluster Properties: cib-bootstrap-options
  cluster-delay=P60S


Note You need to log in before you can comment on or make changes to this bug.