Comment 8 for bug 1432958

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to neutron (master)

Reviewed: https://review.openstack.org/167414
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=c0289458fa9055ee488989828951d15fa2a06da3
Submitter: Jenkins
Branch: master

commit c0289458fa9055ee488989828951d15fa2a06da3
Author: armando-migliaccio <email address hidden>
Date: Tue Mar 24 15:09:35 2015 -0700

    Fix intermittent failure in TestNetworksFailover UT

    Ensure that the periodic check does not get in the way of method
    calls being tested, by stopping the periodic task from running.

    This patch moves the mock for the periodic check into the
    setup_coreplugin call so it gets called by unit tests that
    use the core plugin.

    The previous location after the construction of the API router was
    too late because the core plugin was already constructed by the
    neutron manager. This led to random failures because the periodic
    tasks leaked by all of the unit tests would occasionally preempt
    test_reschedule_network_from_down_agent in automatically removing
    a network from an agent.

    Co-Author: Jenkins <email address hidden>

    Change-Id: I60ad7fa8ca874f93b7f806a0e035be84180a5de9
    Closes-bug: #1432958