Oslo - a Library of Common OpenStack Code

RPC Envelope does not work with impl_zmq

Reported by Eric Windisch on 2013-02-13
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
oslo
Undecided
Eric Windisch
Grizzly
Undecided
Eric Windisch

Bug Description

Having built a testsuite that enables and tests message envelopes for impl_zmq, I've found that the envelopes are presently broken when using this driver.

Tags: rpc Edit Tag help
Changed in oslo:
assignee: nobody → Eric Windisch (ewindisch)
status: New → In Progress

Reviewed: https://review.openstack.org/21790
Committed: http://github.com/openstack/oslo-incubator/commit/f1e5d569b6c9ceb6d7a4b338db9186e4f9c2fb7b
Submitter: Jenkins
Branch: master

commit f1e5d569b6c9ceb6d7a4b338db9186e4f9c2fb7b
Author: Eric Windisch <email address hidden>
Date: Tue Feb 12 11:46:24 2013 -0500

    Support RPC envelopes in impl_zmq

    This patch began as a set
    of tests verifying the functionality of
    sending and receiving RPC envelopes when
    using impl_zmq. It was discovered that
    when enabled, RPC envelopes were not
    actually working,

    The ZeroMQ driver includes its own envelopes.
    This patch introduce versioning to that
    envelope, eliminating the previously reserved
    'style' field.

    A new iteration of the zeromq-envelope is
    introduced, 'impl_zmq_v2'. It specifies
    that the zeromq-envelope should be followed
    by an unpacked array representing key value
    pairs of the standard RPC Envelope.

    Because the key-values of the RPC Envelope
    can be successfully transformed with bytes(),
    this prevents the need to double-serialize
    the content traversing the message bus.

    Also removes some unused imports.

    Closes bug 1123709
    Closes bug 1055446

    Change-Id: Ib04e3d092c9596146f1048d3502ac248496d313b

Changed in oslo:
status: In Progress → Fix Committed
Thierry Carrez (ttx) on 2013-02-21
Changed in oslo:
milestone: none → grizzly-3
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers