VPNaaS UTs broken in neutron/tests/unit/extensions/test_l3.py

[Paul Michali]

Recently, VPNaaS repo UTs are failing in tests that inherit from Neutron tests. The tests worked 4/22/2015 and are broken on 4/24/2015. Will try to bisect to find change in Neutron that affects tests.

Example failure:

2015-04-24 06:40:39.838 | Captured pythonlogging:
2015-04-24 06:40:39.838 | ~~~~~~~~~~~~~~~~~~~~~~~
2015-04-24 06:40:39.838 | 2015-04-24 06:40:38,704 ERROR [neutron.api.extensions] Extension path 'neutron/tests/unit/extensions' doesn't exist!
2015-04-24 06:40:39.838 |
2015-04-24 06:40:39.838 |
2015-04-24 06:40:39.838 | Captured traceback:
2015-04-24 06:40:39.838 | ~~~~~~~~~~~~~~~~~~~
2015-04-24 06:40:39.838 | Traceback (most recent call last):
2015-04-24 06:40:39.838 | File "neutron_vpnaas/tests/unit/db/vpn/test_vpn_db.py", line 886, in test_delete_router_interface_in_use_by_vpnservice
2015-04-24 06:40:39.839 | expected_code=webob.exc.
2015-04-24 06:40:39.839 | File "/home/jenkins/workspace/gate-neutron-vpnaas-python27/.tox/py27/src/neutron/neutron/tests/unit/extensions/test_l3.py", line 401, in _router_interface_action
2015-04-24 06:40:39.839 | self.assertEqual(res.status_int, expected_code, msg)
2015-04-24 06:40:39.839 | File "/home/jenkins/workspace/gate-neutron-vpnaas-python27/.tox/py27/local/lib/python2.7/site-packages/testtools/testcase.py", line 350, in assertEqual
2015-04-24 06:40:39.839 | self.assertThat(observed, matcher, message)
2015-04-24 06:40:39.839 | File "/home/jenkins/workspace/gate-neutron-vpnaas-python27/.tox/py27/local/lib/python2.7/site-packages/testtools/testcase.py", line 435, in assertThat
2015-04-24 06:40:39.839 | raise mismatch_error
2015-04-24 06:40:39.839 | testtools.matchers._impl.MismatchError: 200 != 409
2015-04-24 06:40:39.839 |
2015-04-24 06:40:39.839 |
2015-04-24 06:40:39.839 | Captured pythonlogging:
2015-04-24 06:40:39.839 | ~~~~~~~~~~~~~~~~~~~~~~~
2015-04-24 06:40:39.840 | 2015-04-24 06:40:38,694 INFO [neutron.manager] Loading core plugin: neutron_vpnaas.tests.unit.db.vpn.test_vpn_db.TestVpnCorePlugin
2015-04-24 06:40:39.840 | 2015-04-24 06:40:38,694 INFO [neutron.manager] Service L3_ROUTER_NAT is supported by the core plugin
2015-04-24 06:40:39.840 | 2015-04-24 06:40:38,694 INFO [neutron.manager] Loading Plugin: neutron_vpnaas.services.vpn.plugin.VPNPlugin
2015-04-24 06:40:39.840 | 2015-04-24 06:40:38,704 ERROR [neutron.api.extensions] Extension path 'neutron/tests/unit/extensions' doesn't exist!

[Paul Michali (pcm)]

git bisect start
# bad: [e16d78e80e1ec9ed4a407bba2c9a29da941d6d12] Merge "OVS_LIB API addition - change bridge controller connection-mode"
git bisect bad e16d78e80e1ec9ed4a407bba2c9a29da941d6d12
# good: [0c1f96ad5a6606c1205bd50ea944c3a383892cde] Restrict subnet create/update to avoid DHCP resync
git bisect good 0c1f96ad5a6606c1205bd50ea944c3a383892cde
# good: [76d873a452e340944e2e3242e8bb1722e3c036e8] Merge "Fix super cleanUp for fullstack ProcessFixture"
git bisect good 76d873a452e340944e2e3242e8bb1722e3c036e8
# bad: [876717baf8f88dfc6633b9d58fc45468c8b86b0c] Merge "Add security groups events"
git bisect bad 876717baf8f88dfc6633b9d58fc45468c8b86b0c
# good: [fab814ecf2dc29766b80a1cdb8625d99714c93b0] Merge "ovs_neutron_agent: Remove a redundant assignment of ovs_status"
git bisect good fab814ecf2dc29766b80a1cdb8625d99714c93b0
# good: [05b9cb2c0560486294eb5a3144cbcab423033bf0] Merge "Ensure metadata network works with DVR"
git bisect good 05b9cb2c0560486294eb5a3144cbcab423033bf0
# good: [10fabf15de9d7a43b81afc359d27c0acfa118762] Merge "Remove dependency on weak reference for registry callbacks"
git bisect good 10fabf15de9d7a43b81afc359d27c0acfa118762
# good: [27ab8e619324a212b54c431a9837cebe8beb3618] Merge "L3 DB: Defer port DB subnet lookups"
git bisect good 27ab8e619324a212b54c431a9837cebe8beb3618
# bad: [868e67b480b08cc815d802cf950547c6b5ac0153] Add security groups events
git bisect bad 868e67b480b08cc815d802cf950547c6b5ac0153
# first bad commit: [868e67b480b08cc815d802cf950547c6b5ac0153] Add security groups events

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: master
Review: https://review.openstack.org/178306

[OpenStack Infra (hudson-openstack)]

Change abandoned by venkata anil (<email address hidden>) on branch: master
Review: https://review.openstack.org/178306

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/177451
Committed: https://git.openstack.org/cgit/openstack/neutron-vpnaas/commit/?id=5c1d1b0180f642faf4d8d2341386d04926d4b288
Submitter: Jenkins
Branch: master

commit 5c1d1b0180f642faf4d8d2341386d04926d4b288
Author: armando-migliaccio <email address hidden>
Date: Fri Apr 24 13:44:34 2015 -0700

    Fix failures for integration tests

    These unit tests were affected by the lack of callback registration.

    Callbacks can be wiped out depending on the order in which subscriptions
    and mocking of the callback registry manager takes place. By subscribing
    during setUp, this ensures that the integration is in place so that
    tests can pass.

    Closes-bug: #1448245

    Change-Id: I0dfa1ffa39403cbe00d2724fe21e6e22ef77e07d

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: stable/kilo
Review: https://review.openstack.org/202643

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/202643
Committed: https://git.openstack.org/cgit/openstack/neutron-vpnaas/commit/?id=0ec16689f907ecea0f357b9b0bb7d8709002a7dc
Submitter: Jenkins
Branch: stable/kilo

commit 0ec16689f907ecea0f357b9b0bb7d8709002a7dc
Author: armando-migliaccio <email address hidden>
Date: Fri Apr 24 13:44:34 2015 -0700

    Fix failures for integration tests

    These unit tests were affected by the lack of callback registration.

    Callbacks can be wiped out depending on the order in which subscriptions
    and mocking of the callback registry manager takes place. By subscribing
    during setUp, this ensures that the integration is in place so that
    tests can pass.

    Closes-bug: #1448245
    (cherry picked from commit 5c1d1b0180f642faf4d8d2341386d04926d4b288)

    Change-Id: I0dfa1ffa39403cbe00d2724fe21e6e22ef77e07d