Comment 3 for bug 1046358

Revision history for this message
dan wendlandt (danwent) wrote :

Note: one other thing you should do to make this patch "correct" was that you should update the def validate_networks() method to make sure that this IP address is valid for the network, and has not already been allocated. the validate_networks() method is called by the API layer to confirm that the params passed into the network are valid. This lets us return a 400 if the network data is invalid. If you don't do this, the VM boot API call will "succeed", but the VM will end up in error state when the compute layer later calls allocate_for_instance and reaches out to Quantum.