Patch 1/2 just clears some syntax errors.
Patch 2/2 makes sure we stick to a fixed set of instance states and
actions, and that the state machines for each driver only use states and
actions from that. In particular, we had an ambiguity where some drivers
called 'stopping' 'shutting_down'. The latter state is not used anywhere
anymore.
We still need to do two more things before this whole issue can be laid to rest:
* add checks so that the states of instances we send to clients always
follows the state machine the driver reports
* look through all drivers to make sure the state machines are actually
sane (going through them I noticed that there's a dearth of 'destroy'
actions)
David
