Merge lp:~termie/nova/remove_ioloop into lp:~hudson-openstack/nova/trunk
Proposed by
termie
Status: | Merged |
---|---|
Merged at revision: | 300 |
Proposed branch: | lp:~termie/nova/remove_ioloop |
Merge into: | lp:~hudson-openstack/nova/trunk |
Diff against target: |
265 lines (+64/-34) 7 files modified
nova/rpc.py (+2/-17) nova/test.py (+49/-5) nova/tests/access_unittest.py (+1/-1) nova/tests/auth_unittest.py (+1/-1) nova/tests/cloud_unittest.py (+8/-6) nova/tests/objectstore_unittest.py (+1/-1) nova/tests/rpc_unittest.py (+2/-3) |
To merge this branch: | bzr merge lp:~termie/nova/remove_ioloop |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Joshua McKenty (community) | Approve | ||
Review via email: mp+34913@code.launchpad.net |
Description of the change
Removes most traces of tornado from the codebase, leaving only the things that gundlach's branch will remove later.
The tests were passing even while the code was wrong during part of this process so I updated the TrialTestCase tests to emulate the behavior that BaseTestCase was providing of automatically turning test cases that return generators into defer.inlineCal
To post a comment you must log in.
lgtm, with a bit of either hesitation or confusion about how you're storing originalAttach as an instance property (in _monkeyPatchAtt ach). Isn't there some risk that rpc.Consumer. attach_ to_twisted will be called independently from some of the code during the test runs? (I suppose it's very unlikely...)