Update comments in hroller code
authorKlaus Aehlig <aehlig@google.com>
Wed, 12 Jun 2013 09:42:24 +0000 (11:42 +0200)
committerKlaus Aehlig <aehlig@google.com>
Mon, 1 Jul 2013 07:52:40 +0000 (09:52 +0200)
hroller schedules moves of instances to have rebooted nodes
free of instances with this node as primary. Update the comments
to reflect that this move planning is for non-redundant instances
only.

Signed-off-by: Klaus Aehlig <aehlig@google.com>
Reviewed-by: Michele Tartara <mtartara@google.com>

src/Ganeti/HTools/Program/Hroller.hs

index 44411bc..1887a86 100644 (file)
@@ -96,13 +96,13 @@ move idx new_ndx (nl, il) = do
       il' = Container.add idx inst' il
   return (nl', il')
 
--- | Move an instance to one of the candidate nodes mentioned.
+-- | Move a non-redundant instance to one of the candidate nodes mentioned.
 locateInstance :: Idx -> [Ndx] -> (Node.List, Instance.List)
                   -> Result (Node.List, Instance.List)
 locateInstance idx ndxs conf =
   msum $ map (opToResult . flip (move idx) conf) ndxs
 
--- | Move a list of instances to some of the candidate nodes mentioned.
+-- | Move a list of non-redundant instances to some of the nodes mentioned.
 locateInstances :: [Idx] -> [Ndx] -> (Node.List, Instance.List)
                    -> Result (Node.List, Instance.List)
 locateInstances idxs ndxs conf =