Code review comment for lp:~tamura-yoshiaki/nova/bug-764236

Revision history for this message
Jesse Andrews (anotherjesse) wrote :

A unit test would be adding a test to nova/tests/test_compute.py that resulted in network_info being set and causing exceptions to occur until your patch is applied.

Sometimes errors only really manifest in deployed systems (for which we have smoketests - a series of tests that run against the API of a deployed system).

How did you happen upon the bug in the first place - that might help with determining how we can add a test :)

« Back to merge proposal