Comment 2 for bug 1229520

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.