Bug 2151577

Summary: 'pcs resource config' fails to print boolean attributes set to false [rhel-8.7.0.z]
Product: Red Hat Enterprise Linux 8 Reporter: RHEL Program Management Team <pgm-rhel-tools>
Component: pcsAssignee: Michal Pospisil <mpospisi>
Status: CLOSED ERRATA QA Contact: cluster-qe <cluster-qe>
Severity: high Docs Contact:
Priority: high    
Version: 8.7CC: cfeist, cluster-maint, cluster-qe, idevat, mjuricek, mlisik, mmazoure, mpospisi, nhostako, omular, tojeline
Target Milestone: rcKeywords: Regression, Triaged, ZStream
Target Release: ---Flags: pm-rhel: mirror+
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: pcs-0.10.14-5.el8_7.1 Doc Type: Bug Fix
Doc Text:
Cause: Values of bool and int type are handled incorrectly in `pcs resource|stonith config` command Consequence: Options with values `0` and `false` are not displayed at all as if they were not set. Fix: Correctly handle bool and int values. Result: All set options are correctly displayed.
Story Points: ---
Clone Of: 2151166 Environment:
Last Closed: 2023-01-12 09:26:39 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On: 2151166    
Bug Blocks:    

Comment 1 Michal Pospisil 2022-12-19 12:47:06 UTC
Upstream patch:
https://github.com/ClusterLabs/pcs/commit/f9de2cbf0e17f4bdd1a81f329e7fa905b0e2e066

ENVIRONMENT PREPARATION:
- any cluster

DEV TEST:
## Create a resource with disabled monitor operation
# pcs resource create dummy ocf:pacemaker:Dummy op monitor enabled=false

## Display the configuration, "enabled=0" must be present in the output
# pcs resource config dummy
Resource: dummy (class=ocf provider=pacemaker type=Dummy)
  Operations:
    migrate_from: dummy-migrate_from-interval-0s
      interval=0s
      timeout=20s
    migrate_to: dummy-migrate_to-interval-0s
      interval=0s
      timeout=20s
    monitor: dummy-monitor-interval-60s
      interval=60s
      enabled=0
    reload: dummy-reload-interval-0s
      interval=0s
      timeout=20s
    reload-agent: dummy-reload-agent-interval-0s
      interval=0s
      timeout=20s
    start: dummy-start-interval-0s
      interval=0s
      timeout=20s
    stop: dummy-stop-interval-0s
      interval=0s
      timeout=20s

Comment 2 Michal Pospisil 2022-12-21 15:29:39 UTC
DevTestResults:

[root@r08-07-a ~]# rpm -q pcs
pcs-0.10.14-5.el8_7.1.x86_64

[root@r08-07-a ~]# pcs resource create dummy1 ocf:pacemaker:Dummy op monitor enabled=false

[root@r08-07-a ~]# pcs resource config dummy1
Resource: dummy1 (class=ocf provider=pacemaker type=Dummy)
 Operations:
   migrate_from: dummy1-migrate_from-interval-0s
     interval=0s
     timeout=20s
   migrate_to: dummy1-migrate_to-interval-0s
     interval=0s
     timeout=20s
   monitor: dummy1-monitor-interval-60s
     interval=60s
     enabled=0
   reload: dummy1-reload-interval-0s
     interval=0s
     timeout=20s
   reload-agent: dummy1-reload-agent-interval-0s
     interval=0s
     timeout=20s
   start: dummy1-start-interval-0s
     interval=0s
     timeout=20s
   stop: dummy1-stop-interval-0s
     interval=0s
     timeout=20s

Comment 9 errata-xmlrpc 2023-01-12 09:26:39 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (pcs bug fix and enhancement update), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHBA-2023:0127