Code review comment for lp:~salvatore-orlando/neutron/bug804237
Revision history for this message
| Somik Behera (somikbehera) wrote | # |
review:
Approve
(netstack-core)
© 2004
Canonical Ltd.
•
Terms of use
•
Data privacy
•
Contact Launchpad Support
•
Blog
•
Careers
•
System status
•
0e1f616
(Get the code!)
Changes look good and uptil they pass the existing unit test, I dont see this change requiring full exhaustive tests, since if this change works ok for couple of API tests, it will work for everything else.
This seems to be a critical fix, so I vote that we merge it to trunk.