On Mon, Sep 08, 2008 at 03:51:23PM +0100, Guido Trotter wrote: > > This new function checks whether an instance is running, before shutting > down its disks. This is what the Exec() of LUDeactivateInstanceDisks > did, so that is replaced by a call to this function. > ---
This naming is very misleading. I look at the code and I see "StopInstanceDisks" and "ShutdownInstanceDisks", which does what? Please name one "UnsafeShutdownInstancedisks" and one (with the check) "ShutdownInstanceDisks" so we can tell them apart... iustin
