OpenStack HA Cluster Charm

WARNING: This command 'get-property' is deprecated, please use 'get_property'

Bug #2008704 reported by Felipe Reyes
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack HA Cluster Charm
Fix Committed
Undecided
Felipe Reyes

Bug Description

hacluster uses the command "crm configure get-property " to obtain a property of the cluster, although "get-property" has been deprecated in favor of "get_property", and since crmsh 4.2.1 a warning is printed to stdout [0] breaking the parsing:

# crm configure get-property maintenance-mode 2>/dev/null
WARNING: This command 'get-property' is deprecated, please use 'get_property'
INFO: "get-property" is accepted as "get_property"
true
# crm configure get-property maintenance-mode >/dev/null |wc -l
0

[0] https://github.com/ClusterLabs/crmsh/commit/86282af8e5389138ddd204ed5bd9358460219688

Felipe Reyes (freyes)
Changed in charm-hacluster:
assignee: nobody → Felipe Reyes (freyes)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to charm-hacluster (master)

Fix proposed to branch: master
Review: https://review.opendev.org/c/openstack/charm-hacluster/+/875496

Changed in charm-hacluster:
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to charm-hacluster (master)

Reviewed: https://review.opendev.org/c/openstack/charm-hacluster/+/875496
Committed: https://opendev.org/openstack/charm-hacluster/commit/8446b38347c666ecb2bddaadfd104ee0d4b5c92a
Submitter: "Zuul (22348)"
Branch: master

commit 8446b38347c666ecb2bddaadfd104ee0d4b5c92a
Author: Felipe Reyes <email address hidden>
Date: Mon Feb 27 10:59:00 2023 -0300

    Use get_property instead of get-property

    hacluster uses the command "crm configure get-property <CMD>" to obtain
    a property of the cluster, although "get-property" has been deprecated
    in favor of "get_property", since crmsh-4.2.1 a warning is printed to
    stdout[0] breaking the parsing.

        # crm configure get-property maintenance-mode 2>/dev/null
        WARNING: This command 'get-property' is deprecated, please use 'get_property'
        INFO: "get-property" is accepted as "get_property"
        true

    [0] https://github.com/ClusterLabs/crmsh/commit/86282af8e5389138ddd204ed5bd9358460219688

    Change-Id: Id0ee9ab1873d14dcd1c960001cdeb8318f599ef5
    Closes-Bug: #2008704

Changed in charm-hacluster:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to charm-hacluster (stable/jammy)

Fix proposed to branch: stable/jammy
Review: https://review.opendev.org/c/openstack/charm-hacluster/+/875768

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to charm-hacluster (stable/jammy)

Reviewed: https://review.opendev.org/c/openstack/charm-hacluster/+/875768
Committed: https://opendev.org/openstack/charm-hacluster/commit/6909080a77aab7a13ab5e9695e5665b25f2df4e9
Submitter: "Zuul (22348)"
Branch: stable/jammy

commit 6909080a77aab7a13ab5e9695e5665b25f2df4e9
Author: Felipe Reyes <email address hidden>
Date: Mon Feb 27 10:59:00 2023 -0300

    Use get_property instead of get-property

    hacluster uses the command "crm configure get-property <CMD>" to obtain
    a property of the cluster, although "get-property" has been deprecated
    in favor of "get_property", since crmsh-4.2.1 a warning is printed to
    stdout[0] breaking the parsing.

        # crm configure get-property maintenance-mode 2>/dev/null
        WARNING: This command 'get-property' is deprecated, please use 'get_property'
        INFO: "get-property" is accepted as "get_property"
        true

    [0] https://github.com/ClusterLabs/crmsh/commit/86282af8e5389138ddd204ed5bd9358460219688

    Change-Id: Id0ee9ab1873d14dcd1c960001cdeb8318f599ef5
    Closes-Bug: #2008704
    (cherry picked from commit 8446b38347c666ecb2bddaadfd104ee0d4b5c92a)

tags: added: in-stable-jammy
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.