Reviewed: https://review.openstack.org/620334 Committed: https://git.openstack.org/cgit/openstack/neutron-dynamic-routing/commit/?id=7f82a19155abfb88f6f1d463ac90de3198ad67c9 Submitter: Zuul Branch: master
commit 7f82a19155abfb88f6f1d463ac90de3198ad67c9 Author: YAMAMOTO Takashi <[email protected]> Date: Tue Nov 27 23:31:34 2018 +0900 Remove unnecessary mocking of get_admin_context Because: - They are not necessary. - Neutron agent scheduler code (reschedule_resources_from_down_agents) seems to (correctly) assume each get_admin_context() call returns a fresh context. This mocking breaks it. - Right now, Neutron agentdb uses the new engine facade [1] and Neutron agent scheduler doesn't. It causes caching issue in our unit tests. (See the Related-Bug) Stopping reusing contexts (thus sessions) would prevent the issue. [1] https://review.openstack.org/#/c/611462/ Related-Bug: #1803745 Change-Id: Ia986f98f8bda9c7cde33af3dfdf2e45a2685fd6d -- You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. https://bugs.launchpad.net/bugs/1803745 Title: neutron-dynamic-routing: unit test failures with master branch of neutron To manage notifications about this bug go to: https://bugs.launchpad.net/neutron/+bug/1803745/+subscriptions -- ubuntu-bugs mailing list [email protected] https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
