On Wed, May 25, 2011 at 04:52:58PM +0200, Michael Hanselmann wrote: > Am 25. Mai 2011 16:51 schrieb Iustin Pop <[email protected]>: > > On Wed, May 25, 2011 at 04:45:50PM +0200, Michael Hanselmann wrote: > >> --- a/lib/cmdlib.py > >> +++ b/lib/cmdlib.py > >> @@ -6648,7 +6648,7 @@ class LUNodeMigrate(LogicalUnit): > >> REQ_BGL = False > >> > >> def CheckArguments(self): > >> - _CheckIAllocatorOrNode(self, "iallocator", "remote_node") > >> + pass > > > > Is this still needed? > > Yes, neither iallocator nor remote node are mandatory as a general > case. If they are needed, the respective instance's tasklet will fail > in CheckPrereq.
LGTM then, thanks. iustin
