Rabbit OCF: we should idempotently block / unblock

Bug #1375824 reported by Bogdan Dobrelya
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
Medium
Bogdan Dobrelya
5.1.x
Fix Committed
Medium
Bogdan Dobrelya
6.0.x
Fix Committed
Medium
Bogdan Dobrelya

Bug Description

Related bug: https://bugs.launchpad.net/fuel/+bug/1373569

We should idempotent add/remove blocking rules to iptables. If we have it already for some reason, we should skip the action, otherwise under some conditions we'll have more than one rule left.

Changed in fuel:
status: New → Triaged
importance: Undecided → Medium
milestone: none → 6.0
assignee: nobody → Fuel Library Team (fuel-library)
tags: added: ha pacemaker rabbitmq
Changed in fuel:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (master)

Reviewed: https://review.openstack.org/124121
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=f390f336683250ebd2cf41110edab33f4ed5ef80
Submitter: Jenkins
Branch: master

commit f390f336683250ebd2cf41110edab33f4ed5ef80
Author: Bogdan Dobrelya <email address hidden>
Date: Thu Sep 25 20:55:43 2014 +0300

    Fix blocking reject rule for rabbit ocf

    * Make RMQ unblock call safe (remove all discovered RMQ
      blocking rules, if there are many of them).
    * Use unblock safe call prior to the blocking one.
    * Make block call idempotent and add 5 retries for iptables.
    * Add info log messages about block/unblock actions. Notify
      if RMQ blocking rule cannot be added for some strange reason.

    Partial-bug: #1373569
    Closes-bug: #1375824

    Change-Id: I46c6bf3c83ada4273eaa05530e80886ebac7e75f
    Signed-off-by: Bogdan Dobrelya <email address hidden>

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

Fix proposed to branch: stable/5.1
Review: https://review.openstack.org/128305

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to fuel-library (stable/5.1)

Reviewed: https://review.openstack.org/128305
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=9be38a613486eb115050d8d165ce212d4a185f35
Submitter: Jenkins
Branch: stable/5.1

commit 9be38a613486eb115050d8d165ce212d4a185f35
Author: Bogdan Dobrelya <email address hidden>
Date: Thu Sep 25 20:55:43 2014 +0300

    Fix blocking reject rule for rabbit ocf

    * Make RMQ unblock call safe (remove all discovered RMQ
      blocking rules, if there are many of them).
    * Use unblock safe call prior to the blocking one.
    * Make block call idempotent and add 5 retries for iptables.
    * Add info log messages about block/unblock actions. Notify
      if RMQ blocking rule cannot be added for some strange reason.

    Partial-bug: #1373569
    Closes-bug: #1375824

    Change-Id: I46c6bf3c83ada4273eaa05530e80886ebac7e75f
    Signed-off-by: Bogdan Dobrelya <email address hidden>

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.