No neutron code coverage for v3 security groups

Bug #1229520 reported by Aaron Rosen
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Invalid
Medium
Unassigned

Bug Description

No neutron code coverage for v3 security groups

Aaron Rosen (arosen)
Changed in nova:
assignee: nobody → Aaron Rosen (arosen)
importance: Undecided → Medium
milestone: none → havana-rc1
tags: added: network
Changed in nova:
milestone: havana-rc1 → none
tags: added: havana-rc-potential
Revision history for this message
Matt Riedemann (mriedem) wrote :

Just for context later, see https://review.openstack.org/#/c/34825/ for background here.

tags: added: testing
Changed in nova:
status: New → Confirmed
Thierry Carrez (ttx)
tags: added: havana-backport-potential
removed: havana-rc-potential
Joe Gordon (jogo)
tags: removed: havana-backport-potential
Brent Eagles (beagles)
tags: added: neutron
Revision history for this message
Matt Riedemann (mriedem) wrote :

Means there wasn't test coverage for the v3 API change I believe, based on my old review comments:

Matt Riedemann Sep 25, 2013
Patch Set 10:
Why wasn't the v3 code coverage added in this patch rather than opening new bug 1229520 to track it which may or may not make havana (rc1 is scheduled for tomorrow I believe).
I'm assuming the reason is time, or lack thereof, but I'm not sure if that is a good reason to not have test coverage for the code change in this patch.

Matt Riedemann Sep 25, 2013
Patch Set 10: Looks good to me, but someone else must approve
Mulled this over a bit, the v3 API is experimental in Havana and I think this is a bug we want fixed, so given you have a bug opened to track the v3 security group test exposure for neutron in nova, we can follow up using that.

Revision history for this message
Matt Riedemann (mriedem) wrote :

Matt Riedemann Sep 19, 2013
Patch Set 8:
I know there are v3 tests similar to the v2 tests, is there nothing in the v3 tests for security groups? Regardless, there needs to be a test to cover the v3 API change also.

Sean Dague (sdague)
Changed in nova:
assignee: Aaron Rosen (arosen) → nobody
Sean Dague (sdague)
Changed in nova:
status: Confirmed → Invalid
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.