Fix bad changes in corosync/pacemaker modules and contribute changes upstream

Bug #1453773 reported by Bogdan Dobrelya
16
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Won't Fix
Medium
Dmitry Ilyin
Nominated for Newton by Ivan Berezovskiy
6.1.x
Won't Fix
Medium
Fuel Library (Deprecated)
7.0.x
Won't Fix
Medium
Dmitry Ilyin

Bug Description

We have a patch https://review.openstack.org/#/c/180669/ changing the forked puppetlabs-corosync module in wrong way (the unicast_addressed with multi-ring support was removed and replaced by corosync_nodes w/o multi-rings)
This must be fixed and contributed in upstream ( see https://tickets.puppetlabs.com/browse/MODULES-1697 ) ASAP in the 7.0 dev.

The other related module, fuel-pacemaker, currently has broken rspecs and there is a patch fixing them https://review.openstack.org/#/c/150555/. It should be rebased and accepted ASAP in the 7.0 dev. Related changes to the pacemaker provider, which is the part of fuel-pacemaker module, should be as well contributed to puppet-openstack_extras - which is the current place the provider resides upstream.

tags: added: corosync non-release pacemaker
Changed in fuel:
milestone: none → 7.0
importance: Undecided → High
assignee: nobody → Fuel Library Team (fuel-library)
status: New → Confirmed
tags: added: tech-debt
Dmitry Pyzhov (dpyzhov)
Changed in fuel:
milestone: 7.0 → 8.0
no longer affects: fuel/8.0.x
Dmitry Pyzhov (dpyzhov)
tags: added: area-library
Changed in fuel:
milestone: 8.0 → 9.0
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/224857
Committed: https://git.openstack.org/cgit/openstack/fuel-library/commit/?id=2f2239025ac0d0893470a309b9e03c884fb0da89
Submitter: Jenkins
Branch: master

commit 2f2239025ac0d0893470a309b9e03c884fb0da89
Author: Vladimir Kuklin <email address hidden>
Date: Mon Feb 29 16:53:55 2016 +0300

    Refactor to pcmk_ resources

    Use pcmk_* resources from the new pacemaker module
    instead of the old cs_* resources from the upstream
    corosync modules and custom cs_rsc_* resources from the
    duct tape patched corosync module.

    The new pacemaker module is rewritten from the scratch
    and provides improved performance, transparency and
    debuggability. It uses it's own XML engine and depends
    neither on "pcs" nor on "crm" applications.

    "corosync" modules will still be used to install the
    corosync cluster, and the "pacemaker" modules will
    be used to work with the cluster's configuration.

    Refactor pacemaker services to use pacemaker::service
    wrapper. It controlls primitive creation and run in the
    uniform way and creates ocf_handler helpers for
    manual primitive management.

    Add run_failed_log feature to fuel_noop_tests to
    replay the failed tests after they were fixed.

    Change-Id: Ifd5813c657031cdd7b86368cd971742483a5b8a1
    Related-Bug: 1414583
    Related-Bug: 1453773
    Related-Bug: 1482248

Changed in fuel:
status: Confirmed → Fix Committed
Revision history for this message
Bogdan Dobrelya (bogdando) wrote :

Upstream is the openstack/puppet-openstack_extras, the bug remains valid

Changed in fuel:
status: Fix Committed → Confirmed
Revision history for this message
Ivan Berezovskiy (iberezovskiy) wrote :

It's too late to fix medium bugs. Moving to 10.0

Changed in fuel:
status: Confirmed → Won't Fix
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.