remove race condition in detach_vol

detach_vol had an issue where if the state of the image went from
'in-use' (or anything) to 'available' after we checked for the shortcut
path, but before the 'detach-vol' call was received, then it would
return failure.

This now does:
 * check state of vol
   * if vol in 'available', return 0
 * detach vol
   * if detach failed
     * check state
       * if vol is available, return 0
       * else return failure

It leaves in the 'detach-vol' by design as euca calls are just so much
faster.