OpenStack Heat

heat tests use oslo.messaging internal API

Bug #1410196 reported by Mehdi Abaakouk on 2015-01-13

This bug report is a duplicate of: Bug #1412836: unit tests mocking private symbol _CallContext. Edit Remove

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	OpenStack Heat	New	Undecided	Unassigned
	oslo.messaging	Fix Released	Critical	Doug Hellmann	oslo.messaging 1.6.0

Bug Description

Hi,

Heat use private code of oslo.messaging and will break on next release of oslo.messaging (1.5.2).

https://github.com/openstack/heat/blob/master/heat/tests/test_rpc_client.py#L23
https://github.com/openstack/heat/blob/master/heat/tests/test_fault_middleware.py#L20
https://github.com/openstack/heat/blob/master/heat/tests/test_api_openstack_v1.py#L18

Cheers,

Doug Hellmann (doug-hellmann) on 2015-01-13

Changed in oslo.messaging:
assignee:	nobody → Doug Hellmann (doug-hellmann)
importance:	Undecided → Critical
status:	New → Triaged
milestone:	none → next-kilo

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-01-13: Fix proposed to oslo.messaging (master)

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

Changed in oslo.messaging:
status:	Triaged → In Progress

Revision history for this message

OpenStack Infra (hudson-openstack) wrote on 2015-01-16: Fix merged to oslo.messaging (master)

Reviewed: https://review.openstack.org/146872
Committed: https://git.openstack.org/cgit/openstack/oslo.messaging/commit/?id=a6d068a984488e9d76ed20486446d62bf31a14dc
Submitter: Jenkins
Branch: master

commit a6d068a984488e9d76ed20486446d62bf31a14dc
Author: Doug Hellmann <email address hidden>
Date: Tue Jan 13 08:50:37 2015 -0500

Add oslo.messaging._drivers.common for heat tests

    Some of the heat unit tests import code from a private part of
    oslo.messaging. With the namespace package change, that private part was
    no longer being exposed. This change introduces the needed module so we
    can release oslo.messaging with the namespace changes without breaking
    heat, to give the heat team time to update their tests.

Partial-Bug: #1410196
Change-Id: Ib3ea106646f75b17665877b08ccbd382c5698714

Doug Hellmann (doug-hellmann) on 2015-01-27

Changed in oslo.messaging:
status:	In Progress → Fix Committed
status:	Fix Committed → Fix Released

Report a bug

This report contains Public information

Everyone can see this information.

Duplicate of bug #1412836 Remove

You are

Subscribing...

Edit bug mail

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.