test_openvswitch_plugin sometimes fail

Bug #1281481 reported by YAMAMOTO Takashi
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Low
Isaku Yamahata

Bug Description

depending on the sequence of tests, test_openvswitch_plugin sometimes ends up to kick
unexpected code via impl_fake rpc backend.

an example of the failure:
http://logs.openstack.org/91/71791/5/check/gate-neutron-python27/2317e6a/

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

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

Changed in neutron:
assignee: nobody → YAMAMOTO Takashi (yamamoto)
status: New → In Progress
Revision history for this message
Oleg Bondarev (obondarev) wrote :

Can't find the error using the provided link

Changed in neutron:
status: In Progress → Incomplete
Revision history for this message
Itsuro Oda (oda-g) wrote :

Oleg,

Test is failed by 'sudo /usr/local/jenkins/slave_scripts/jenkins-sudo-grep.sh post'.

Revision history for this message
Isaku Yamahata (yamahata) wrote :

The patch fixes only ovs plugin test case.
Similar fix would be necessary for other test case which is subclass of NeutronDbPluginV2TestCase?

Revision history for this message
YAMAMOTO Takashi (yamamoto) wrote :
Revision history for this message
YAMAMOTO Takashi (yamamoto) wrote :

yamahata,

probably.
ryu plugin test already has the similar mocking.
i haven't checked others.

Revision history for this message
Mark McClain (markmcclain) wrote :

Why don't we make the test clean-up the notifications vs adding a mock to cover up the issue.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix proposed to neutron (master)

Related fix proposed to branch: master
Review: https://review.openstack.org/76418

Changed in neutron:
assignee: YAMAMOTO Takashi (yamamoto) → Isaku Yamahata (yamahata)
status: Incomplete → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Related fix merged to neutron (master)

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

commit 4434882bc41c87c52192ab3ad01d3448868603cc
Author: Isaku Yamahata <email address hidden>
Date: Tue Feb 25 15:34:53 2014 +0900

    tests/unit: clean up notification driver

    Some tests doesn't cleanup notification driver. So some tests can
    run against unintended notification driver depending on execution order

    Change-Id: I7dbed47153be66684827b2693bbc995dddfb4c1b
    Related-Bug: #1281481
    Partial-Bug: #1284549

Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote : Fix merged to neutron (master)

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

commit a98dc7680d4689f5ce5f602f9c5cb3bcc77de215
Author: Isaku Yamahata <email address hidden>
Date: Wed Feb 26 11:36:56 2014 +0900

    tests/unit: refactor reading neutron.conf.test

    neutron.conf.test includes rpc_backend whose value is stashed.
    Thus it is required to reset when tearing down, otherwise the stale status
    will be used by succeeding tests causing random error.

    This patch refactors reading neutron.conf.test and resets the status of
    rpc_backend properly.

    Closes-Bug: #1281481
    Closes-Bug: #1284549
    Change-Id: I0fa5945b6adbb9945d353028ec88d00ccbf4e31a

Changed in neutron:
status: In Progress → Fix Committed
Kyle Mestery (mestery)
Changed in neutron:
milestone: none → juno-1
importance: Undecided → Low
Thierry Carrez (ttx)
Changed in neutron:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in neutron:
milestone: juno-1 → 2014.2
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers