From 729164d0cc2bcebefdf2bc6444ad11d97ec98218 Mon Sep 17 00:00:00 2001 From: Iustin Pop Date: Fri, 9 Jan 2009 14:26:53 +0000 Subject: [PATCH] burnin: do not use offline nodes This patch makes burnin skip the offline nodes in it's builtin node selection. It also removes an extra line. Reviewed-by: imsnah --- tools/burnin | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/tools/burnin b/tools/burnin index 018396c..1170108 100755 --- a/tools/burnin +++ b/tools/burnin @@ -231,13 +231,13 @@ class Burner(object): else: names = [] try: - op = opcodes.OpQueryNodes(output_fields=["name"], names=names) + op = opcodes.OpQueryNodes(output_fields=["name", "offline"], names=names) result = self.ExecOp(op) except errors.GenericError, err: err_code, msg = cli.FormatError(err) Log(msg) sys.exit(err_code) - self.nodes = [data[0] for data in result] + self.nodes = [data[0] for data in result if not data[1]] result = self.ExecOp(opcodes.OpDiagnoseOS(output_fields=["name", "valid"], names=[])) @@ -452,7 +452,6 @@ class Burner(object): Log("- Remove instance %s" % instance) self.ExecOp(op) - def Rename(self): """Rename the instances.""" rename = self.opts.rename -- 1.7.10.4