Merge lp:~cbehrens/nova/lp766404 into lp:~hudson-openstack/nova/trunk
Proposed by
Chris Behrens
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Josh Kearney | ||||
Approved revision: | 1067 | ||||
Merged at revision: | 1070 | ||||
Proposed branch: | lp:~cbehrens/nova/lp766404 | ||||
Merge into: | lp:~hudson-openstack/nova/trunk | ||||
Diff against target: |
234 lines (+76/-74) 4 files modified
nova/tests/test_xenapi.py (+23/-0) nova/tests/xenapi/stubs.py (+11/-28) nova/utils.py (+2/-0) nova/virt/xenapi_conn.py (+40/-46) |
||||
To merge this branch: | bzr merge lp:~cbehrens/nova/lp766404 | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Vish Ishaya (community) | Approve | ||
Josh Kearney (community) | Approve | ||
Review via email: mp+60543@code.launchpad.net |
Description of the change
XenAPI was not implemented to allow for multiple simultaneous XenAPI requests. A single XenAPIConnection (and thus XenAPISession) is used for all queries. XenAPISession's wait_for_task method would set a self.loop = for looping calls to _poll_task until task completion. Subsequent (parallel) calls to wait_for_task for another query would overwrite this. XenAPISession.
To post a comment you must log in.
Are there tests that can show this fix in action?