event-engine works incorrectly in HA

Bug #1715848 reported by Nikolay Makhotkin
18
This bug affects 4 people
Affects Status Importance Assigned to Milestone
Mistral
Fix Released
High
Jose Castro Leon

Bug Description

If we start, say, 3 instances of event-engine, have registered some trigger on some event and trigger this event, all event-engines receive this event and trigger a workflow. Each event-engine works independently now and starts a workflow so we have 3 started workflow at this point.

Need some kind of synchronization for event-engine to start at most 1 workflow for each event.

Changed in mistral:
milestone: queens-1 → queens-3
Changed in mistral:
milestone: queens-3 → none
Changed in mistral:
assignee: nobody → Jose Castro Leon (jose-castro-leon)
Revision history for this message
Jose Castro Leon (jose-castro-leon) wrote :

A previous patch allows to have a common queue for the listeners to only launch a workflow
https://github.com/openstack/mistral/commit/e0eeca6706a873b90f895674af2b00e6dd57ae69

But the RPC calls are still synchronous, so only one event_engine modifies its listeners, leaving the other ones behind

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to mistral (master)

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

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

Reviewed: https://review.openstack.org/548044
Committed: https://git.openstack.org/cgit/openstack/mistral/commit/?id=defff0877392400e572dca772d24b2c6fe49dca1
Submitter: Zuul
Branch: master

commit defff0877392400e572dca772d24b2c6fe49dca1
Author: Jose Castro Leon <email address hidden>
Date: Mon Feb 26 16:00:27 2018 +0100

    [Event-engine] Allow event_engine to work in HA

    A previous patch allows to make multiple event_engines to listen
    to a single queue, but the RPC calls on CRUD are still synchronous

    This patch modifies the calls and broadcasts them on all the event
    engines allow them to modify each independent listeners.

    Closes-Bug: #1715848
    Change-Id: Ia37831a03993f5a1bf980d62344d25377062788d

Changed in mistral:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to mistral (stable/rocky)

Fix proposed to branch: stable/rocky
Review: https://review.openstack.org/623936

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/mistral 8.0.0.0b1

This issue was fixed in the openstack/mistral 8.0.0.0b1 development milestone.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to mistral (stable/rocky)

Reviewed: https://review.openstack.org/623936
Committed: https://git.openstack.org/cgit/openstack/mistral/commit/?id=ec06ccf65a69fdfe95f2f669591ff2a4a3aca7cc
Submitter: Zuul
Branch: stable/rocky

commit ec06ccf65a69fdfe95f2f669591ff2a4a3aca7cc
Author: Jose Castro Leon <email address hidden>
Date: Mon Feb 26 16:00:27 2018 +0100

    [Event-engine] Allow event_engine to work in HA

    A previous patch allows to make multiple event_engines to listen
    to a single queue, but the RPC calls on CRUD are still synchronous

    This patch modifies the calls and broadcasts them on all the event
    engines allow them to modify each independent listeners.

    Closes-Bug: #1715848
    Change-Id: Ia37831a03993f5a1bf980d62344d25377062788d
    (cherry picked from commit defff0877392400e572dca772d24b2c6fe49dca1)

tags: added: in-stable-rocky
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/mistral 7.1.0

This issue was fixed in the openstack/mistral 7.1.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.