Code review comment for lp:~cerberus/nova/os-api-retune

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

Overall, lgtm.

> + inst_ref = self.compute_api.get_instance(ctxt, id)

This line is added in update(), but inst_ref isn't referenced later. Is this to trigger an exception as a side-effect? If that's the case, that might merit a comment.

« Back to merge proposal