ovs_neutron_agent is missing test coverage

Bug #1285893 reported by Anita Kuno
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
neutron
Expired
Medium
Unassigned

Bug Description

neutron/tests/unit/openvswitch/test_ovs_neutron_agent.py is missing test coverage

Tags: ovs
Changed in neutron:
importance: Undecided → Medium
status: New → Triaged
tags: added: havana-backport-potential ovs
Changed in neutron:
milestone: none → next
milestone: next → icehouse-rc1
Changed in neutron:
assignee: nobody → Manish Godara (manishatyhoo)
status: Triaged → In Progress
tags: added: icehouse-rc-potential ova
removed: ovs
Changed in neutron:
milestone: icehouse-rc1 → none
tags: added: icehouse-backport-potential
removed: icehouse-rc-potential
tags: added: ovs
removed: ova
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on neutron (master)

Change abandoned by Manish Godara (<email address hidden>) on branch: master
Review: https://review.openstack.org/76717

Changed in neutron:
assignee: Manish Godara (manishatyhoo) → nobody
Revision history for this message
Federico Ressi (federico-ressi-9) wrote :
Download full text (3.8 KiB)

I have just run tests on master branch with following commands:

 coverage erase
 coverage run --source=neutron.plugins.ml2.drivers.openvswitch.agent -m unittest neutron.tests.unit.plugins.ml2.drivers.openvswitch.agent.test_ovs_neutron_agent
 coverage html
 coverage report -m

Produced coverage HTML is attached as tarball.

I had following report:

 Name Stmts Miss Branch BrMiss Cover Missing
 -----------------------------------------------------------------------------------------------------------------------------
 neutron/plugins/ml2/drivers/openvswitch/agent/__init__ 0 0 0 0 100%
 neutron/plugins/ml2/drivers/openvswitch/agent/common/__init__ 0 0 0 0 100%
 neutron/plugins/ml2/drivers/openvswitch/agent/common/config 13 0 0 0 100%
 neutron/plugins/ml2/drivers/openvswitch/agent/common/constants 35 0 0 0 100%
 neutron/plugins/ml2/drivers/openvswitch/agent/extension_drivers/__init__ 0 0 0 0 100%
 neutron/plugins/ml2/drivers/openvswitch/agent/extension_drivers/qos_driver 38 38 4 4 0% 15-76
 neutron/plugins/ml2/drivers/openvswitch/agent/main 18 18 0 0 0% 18-47
 neutron/plugins/ml2/drivers/openvswitch/agent/openflow/__init__ 0 0 0 0 100%
 neutron/plugins/ml2/drivers/openvswitch/agent/openflow/ovs_ofctl/__init__ 0 0 0 0 100%
 neutron/plugins/ml2/drivers/openvswitch/agent/openflow/ovs_ofctl/br_dvr_process 16 8 0 0 50% 44, 52, 59, 68, 75-80, 88-91
 neutron/plugins/ml2/drivers/openvswitch/agent/openflow/ovs_ofctl/br_int 53 34 12 12 29% 32-34, 37, 40-46, 49-53, 59-63, 67-70, 74-75, 83-84, 89, 97, 103, 110, 116-125, 130-132
 neutron/plugins/ml2/drivers/openvswitch/agent/openflow/ovs_ofctl/br_phys 20 9 2 2 50% 33-34, 37-45, 52, 55, 60
 neutron/plugins/ml2/drivers/openvswitch/agent/openflow/ovs_ofctl/br_tun 76 49 14 14 30% 55-137, 141-145, 153, 158, 161-162, 168-169, 173-174, 182-187, 192-197, 205-211, 217-218, 224-225, 230, 237, 241, 257-260
 neutron/plugins/ml2/drivers/openvswitch/agent/openflow/ovs_ofctl/main 9 9 0 0 0% 17-36
 neutron/plugins/ml2/drivers/openvswitch/agent/openflow/ovs_ofctl/ofswitch 58 20 16 8 62% 44-47, 50, 53, 56, 59, 65, 71, 77, 85-89, 92-93, 96-97, 107, 112
 neutron/plugins/ml2/drivers/openvswitch/agent/openflow/ovs_ofctl/ovs_bridge 8 3 0 0 63% 27-28, 31
 neutron/plugins/ml2/drivers/openvswitch/agent/ovs_dvr_neutron_agent 392 69 136 45 78% 44, 86, 98, 140, 214, 240-248, 256-272, 320-325, 329-331, 336, 340-341, 356, 374-379, 385-387, 412-430, 469, 475, 488-489, 516-527, 539-542, 551, 566-569, 601, 611-612, 652, 657, 673, 678, 695-696
 neutron/plugins/ml2/drivers/ope...

Read more...

Changed in neutron:
assignee: nobody → Slawek Kaplonski (slaweq)
Alan Pevec (apevec)
tags: removed: havana-backport-potential icehouse-backport-potential
Changed in neutron:
assignee: Slawek Kaplonski (slaweq) → nobody
Revision history for this message
Armando Migliaccio (armando-migliaccio) wrote :

no assignee

Changed in neutron:
status: In Progress → Incomplete
Revision history for this message
Launchpad Janitor (janitor) wrote :

[Expired for neutron because there has been no activity for 60 days.]

Changed in neutron:
status: Incomplete → Expired
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.