Am 28. Februar 2011 16:27 schrieb Rene Nussbaumer <[email protected]>: > On Mon, Feb 28, 2011 at 4:15 PM, Michael Hanselmann <[email protected]> wrote: > [snip] >> + master = qa_config.GetMasterNode() >> + failovermaster = qa_config.AcquireNode(exclude=master) >> + >> + # Ensure queue is not drained >> + for node in [master, failovermaster]: >> + AssertCommand(drain_check, node=node, fail=True) >> + >> + # Drain queue on failover master >> + AssertCommand(["touch", constants.JOB_QUEUE_DRAIN_FILE], >> node=failovermaster) >> + >> + cmd = ["gnt-cluster", "master-failover"] >> + try: >> + AssertCommand(drain_check, node=failovermaster) > > You don't check master here, but just failovermaster? Probably a good > habit to do it on the current master node too? :)
It's done just a few lines before and only the new master node is not used in between. Michael
