ZeroMQ driver fails to send casts if processing during exit

Bug #1097927 reported by Erica Windisch
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
oslo.messaging
Invalid
Low
Unassigned

Bug Description

The ZeroMQ driver only completes a cast() when a remote host's olso-zmq-receiver process consumes a message, or if the cast() times out (via rpc_cast_timeout or the per-cast Timeout kwarg).

Because cast() is spawned inside a greenthread, Python will happily exit with unsent messages.

At a minimum, impl_zmq's cleanup() method should wait for all cast() messages to send or timeout.

Tags: zmq
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to oslo-incubator (master)

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

Changed in oslo:
assignee: nobody → Eric Windisch (ewindisch)
status: New → In Progress
Revision history for this message
Erica Windisch (ewindisch) wrote :

Note that this doesn't have much impact on long running tasks such as services, but is apparent in short-running ones such as command line rpc clients. There aren't any of those in common use, I caught this using my own rpc client which I put together to test another bug.

Mark McLoughlin (markmc)
Changed in oslo:
importance: Undecided → Medium
Revision history for this message
Mark McLoughlin (markmc) wrote :

Patch was abandoned 2 months ago with "Better implementation to follow"

Changed in oslo:
assignee: Eric Windisch (ewindisch) → nobody
status: In Progress → Triaged
affects: oslo-incubator → oslo.messaging
Changed in oslo.messaging:
importance: Medium → Low
Li Ma (nick-ma-z)
tags: added: zmq
Li Ma (nick-ma-z)
Changed in oslo.messaging:
assignee: nobody → Li Ma (nick-ma-z)
Li Ma (nick-ma-z)
Changed in oslo.messaging:
assignee: Li Ma (nick-ma-z) → nobody
Revision history for this message
Oleksii Zamiatin (ozamiatin) wrote :

If you find a similar bug in new driver please report a new one
https://review.openstack.org/#/c/223877/

Changed in oslo.messaging:
status: Triaged → Invalid
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.