Dear Klaus,

On 13.07.2015 14:54, Klaus Aehlig wrote:
> - Failover is not the only non-disk move. Think about shared/external storage
>    and failover-to-any.

It seems that checkInstanceMove function in Cluster.hs file is incorrect because it tries only Failover when disk moves are disallowed.

Am I right? Or maybe I misunderstand something.


aft_failover = if mir_type == MirrorInternal && use_secondary
               -- if drbd and allowed to failover
                    then checkSingleStep force disk_moves_factor ini_tbl
                           target ini_tbl Failover
                    else ini_tbl
primary_drained = Node.offline
                  . flip Container.find nl
                  $ Instance.pNode target
all_moves =
  if disk_moves
    then concatMap (possibleMoves mir_type use_secondary inst_moves
                    (rest_mig, primary_drained))
         nodes
     else []


--
Sincerely, Oleg Ponomarev <[email protected]>

Reply via email to