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

Revision history for this message
Rick Harris (rconradharris) wrote :

> Long story short, I think including progress counters is a very strong
> statement regarding the type of cloud you expect it to be, and in this case, I
> don't think synchronous features like progress are indicative of the sort of
> scale we expect Nova to achieve.

Nailed it.

Feels weird to argue against my own patch, but you're absolutely right.

If we get a consensus that this patch should be rejected, I'll go ahead and extract out the misc fixes I included in this patch so we don't lose that work.

« Back to merge proposal