Fencing configuration generator fails deployment if action parameter is used

Bug #1753728 reported by Chris Jones
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
tripleo
Fix Released
High
Chris Jones

Bug Description

If the "action" parameter is used for "openstack overcloud generate fencing" with recent versions of Pacemaker, the following error occurs during deployment:

overcloud.AllNodesDeploySteps.ControllerDeployment_Step5.2:
  resource_type: OS::Heat::StructuredDeployment
  physical_resource_id: dde00cd2-8e51-4a4d-9c79-8c9f1e5a95a9
  status: CREATE_FAILED
  status_reason: |
    Error: resources[2]: Deployment to server failed: deploy_status_code : Deployment exited with non-zero status code: 2
  deploy_stdout: |
    ...
            " with Stdlib::Compat::Hash. There is further documentation for validate_legacy function in the README. at [\"/etc/puppet/modules/ssh/manifests/server.pp\", 12]:[\"/var/lib/tripleo-config/puppet_step_config.pp\", 41]",
            "Error: pcs -f /var/lib/pacemaker/cib/puppet-cib-backup20180303-300443-1fhpyxd create failed: Error: stonith option 'action' is deprecated and should not be used, use pcmk_off_action, pcmk_reboot_action instead, use --force to override",
            "Error: /Stage[main]/Tripleo::Fencing/Pacemaker::Stonith::Fence_ipmilan[52:54:00:c3:de:fd]/Pcmk_stonith[stonith-fence_ipmilan-525400c3defd]/ensure: change from absent to present failed: pcs -f /var/lib/pacemaker/cib/puppet-cib-backup20180303-300443-1fhpyxd create failed: Error: stonith option 'action' is deprecated and should not be used, use pcmk_off_action, pcmk_reboot_action instead, use --force to override"
        ]
    }

tl;dr it was a mistake (of mine) to ever add this parameter (it doesn't do what I thought it did, and it makes the cluster unstable), and it should be immediately deprecated and no-op'ed.

Chris Jones (cmsj)
Changed in tripleo:
status: New → Triaged
importance: Undecided → High
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tripleo-common (master)

Fix proposed to branch: master
Review: https://review.openstack.org/550028

Changed in tripleo:
assignee: nobody → Chris Jones (cmsj)
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-tripleoclient (master)

Fix proposed to branch: master
Review: https://review.openstack.org/550029

Changed in tripleo:
milestone: none → rocky-1
Changed in tripleo:
assignee: Chris Jones (cmsj) → Alex Schultz (alex-schultz)
Changed in tripleo:
assignee: Alex Schultz (alex-schultz) → Chris Jones (cmsj)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tripleo-common (master)

Reviewed: https://review.openstack.org/550028
Committed: https://git.openstack.org/cgit/openstack/tripleo-common/commit/?id=69b4962d94ca0d3fb1ba3c1033a550c1b95060fd
Submitter: Zuul
Branch: master

commit 69b4962d94ca0d3fb1ba3c1033a550c1b95060fd
Author: Chris Jones <email address hidden>
Date: Tue Mar 6 11:55:33 2018 +0000

    Remove fence_action parameter from GenerateFencingParametersAction

    We no longer accept a fence_action parameter - it was a mistake to add
    it in the first place (it overrides all actions used with Pacemaker, so
    even simple status requests can reboot/poweroff a node) and has now been
    officially deprecated by Pacemaker, which causes deployments to fail.

    Change-Id: I45e08964b69552eb5b7954278cfe72a5a59e2f8d
    Closes-Bug: #1753728

Changed in tripleo:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-tripleoclient (master)

Reviewed: https://review.openstack.org/550029
Committed: https://git.openstack.org/cgit/openstack/python-tripleoclient/commit/?id=7eaee1b2aa71ef9c6b834746d79b6d521dfdf096
Submitter: Zuul
Branch: master

commit 7eaee1b2aa71ef9c6b834746d79b6d521dfdf096
Author: Chris Jones <email address hidden>
Date: Tue Mar 6 11:57:23 2018 +0000

    Deprecate the action parameter for fencing generation.

    We can no longer pass the action parameter to Pacemaker due to
    deprecation in recent releases, nor should we ever have offered the
    parameter in the first place.

    Change-Id: I6a9de20114df7f6779fa0759528ec25c343786d6
    Depends-On: https://review.openstack.org/550028
    Closes-Bug: #1753728

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to tripleo-common (stable/queens)

Fix proposed to branch: stable/queens
Review: https://review.openstack.org/556210

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-tripleoclient (stable/queens)

Fix proposed to branch: stable/queens
Review: https://review.openstack.org/556211

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to tripleo-common (stable/queens)

Reviewed: https://review.openstack.org/556210
Committed: https://git.openstack.org/cgit/openstack/tripleo-common/commit/?id=b49c192707556c99dcbb0e5250f13a5f090576fc
Submitter: Zuul
Branch: stable/queens

commit b49c192707556c99dcbb0e5250f13a5f090576fc
Author: Chris Jones <email address hidden>
Date: Tue Mar 6 11:55:33 2018 +0000

    Remove fence_action parameter from GenerateFencingParametersAction

    We no longer accept a fence_action parameter - it was a mistake to add
    it in the first place (it overrides all actions used with Pacemaker, so
    even simple status requests can reboot/poweroff a node) and has now been
    officially deprecated by Pacemaker, which causes deployments to fail.

    Change-Id: I45e08964b69552eb5b7954278cfe72a5a59e2f8d
    Closes-Bug: #1753728
    (cherry picked from commit 69b4962d94ca0d3fb1ba3c1033a550c1b95060fd)

tags: added: in-stable-queens
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-tripleoclient (stable/queens)

Reviewed: https://review.openstack.org/556211
Committed: https://git.openstack.org/cgit/openstack/python-tripleoclient/commit/?id=be98d7f2a12c4818aed6554b4ab3fc96cfbdb26a
Submitter: Zuul
Branch: stable/queens

commit be98d7f2a12c4818aed6554b4ab3fc96cfbdb26a
Author: Chris Jones <email address hidden>
Date: Tue Mar 6 11:57:23 2018 +0000

    Deprecate the action parameter for fencing generation.

    We can no longer pass the action parameter to Pacemaker due to
    deprecation in recent releases, nor should we ever have offered the
    parameter in the first place.

    Change-Id: I6a9de20114df7f6779fa0759528ec25c343786d6
    Depends-On: https://review.openstack.org/550028
    Closes-Bug: #1753728
    (cherry picked from commit 7eaee1b2aa71ef9c6b834746d79b6d521dfdf096)

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/python-tripleoclient 9.2.0

This issue was fixed in the openstack/python-tripleoclient 9.2.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tripleo-common 8.6.0

This issue was fixed in the openstack/tripleo-common 8.6.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/python-tripleoclient 10.0.0

This issue was fixed in the openstack/python-tripleoclient 10.0.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/tripleo-common 9.0.0

This issue was fixed in the openstack/tripleo-common 9.0.0 release.

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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