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
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.