Notifier test case failing in parallel runs

Bug #1275154 reported by Ben Nemec
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
oslo-incubator
Fix Released
High
Ben Nemec

Bug Description

Another failure that's been showing up consistently in my parallel test runs. Doesn't appear to be related to the kombu issues, so this one should probably just be fixed if possible.

2014-01-31 20:13:28.852 | FAIL: tests.unit.test_notifier.NotifierTestCase.test_rpc_priority_queue
2014-01-31 20:13:28.853 | tags: worker-1
2014-01-31 20:13:28.853 | ----------------------------------------------------------------------
2014-01-31 20:13:28.853 | Empty attachments:
2014-01-31 20:13:28.853 | pythonlogging:''
2014-01-31 20:13:28.853 | stderr
2014-01-31 20:13:28.853 | stdout
2014-01-31 20:13:28.853 |
2014-01-31 20:13:28.854 | Traceback (most recent call last):
2014-01-31 20:13:28.854 | File "tests/unit/test_notifier.py", line 115, in test_rpc_priority_queue
2014-01-31 20:13:28.854 | ['testnotify', ])
2014-01-31 20:13:28.854 | File "/home/jenkins/workspace/gate-oslo-incubator-python27/.tox/py27/local/lib/python2.7/site-packages/stubout.py", line 118, in Set
2014-01-31 20:13:28.854 | old_child = getattr(parent, child_name)
2014-01-31 20:13:28.854 | File "/home/jenkins/workspace/gate-oslo-incubator-python27/.tox/py27/local/lib/python2.7/site-packages/oslo/config/cfg.py", line 1648, in __getattr__
2014-01-31 20:13:28.855 | raise NoSuchOptError(name)
2014-01-31 20:13:28.855 | NoSuchOptError: no such option: notification_topics

Ben Nemec (bnemec)
Changed in oslo:
importance: Undecided → High
status: New → Confirmed
Revision history for this message
Ben Nemec (bnemec) wrote :

This one was pretty easy to fix. We just need to make sure the option gets imported before this test runs. I believe that previously another test was taking care of it.

Changed in oslo:
assignee: nobody → Ben Nemec (bnemec)
Changed in oslo:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to oslo-incubator (master)

Reviewed: https://review.openstack.org/71133
Committed: https://git.openstack.org/cgit/openstack/oslo-incubator/commit/?id=66bcadfff78446d6d14ebd4a59f139c6e70adc54
Submitter: Jenkins
Branch: master

commit 66bcadfff78446d6d14ebd4a59f139c6e70adc54
Author: Ben Nemec <email address hidden>
Date: Mon Feb 3 18:50:38 2014 +0000

    Make sure rpc_notifier opts exist

    The notifier tests try to override config opts registered by
    rpc_notifier, but rpc_notifier may not be imported before those
    tests run. Use import_opt to ensure the options are registered
    before the test tries to override them.

    Change-Id: I5f01eed286c71ea2706225d2ca0380df153bd975
    Closes-Bug: #1275154

Changed in oslo:
status: In Progress → Fix Committed
Changed in oslo:
milestone: none → icehouse-3
Thierry Carrez (ttx)
Changed in oslo:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in oslo:
milestone: icehouse-3 → 2014.1
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.