Comment 2 for bug 1660747

Revision history for this message
melanie witt (melwitt) wrote :

As mriedem pointed out in IRC, the reset-state API simply changes the vm_state and does an instance.save() immediately and returns. So we shouldn't need to wait for an error state. In cells v1, the instance.save() will trigger a sync to the child cell to update the vm_state. This should be fine since the API reads from the top/parent, but I think we have a bug in our compute/api get_all cells v2 code where we're reading instances from child cells instead of the top/parent cell. We have conditionals where "if cells v1, read from the top" and it looks like get_all was missed.