Code review comment for lp:~rconradharris/nova/server_progress

Revision history for this message
Naveed Massjouni (ironcamel) wrote :

I do not see where you are testing the new server progress feature you have added. For example, I see no new assertions in the tests. Is it because this is difficult to test and should be tested at a higher level?

Also, it looks like you have only added the progress feature for xen. Is this correct?

« Back to merge proposal