From 2cb687ad2340762190997a4a28bbff69fa3d3ac9 Mon Sep 17 00:00:00 2001
From: Iustin Pop <iustin@google.com>
Date: Thu, 3 Jul 2008 12:06:46 +0000
Subject: [PATCH] Reuse the luxi client in cli.SubmitOpCode
By a mistake, we don't reuse the luxi client. As such, we open and close
the connection at each poll cycle and spam the server logs.
Reviewed-by: ultrotter
---
lib/cli.py | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/lib/cli.py b/lib/cli.py
index 167f28cb8..7b8581fa6 100644
--- a/lib/cli.py
+++ b/lib/cli.py
@@ -380,7 +380,7 @@ def SubmitOpCode(op, proc=None, feedback_fn=None):
# TODO: Fix feedback_fn situation.
cl = luxi.Client()
job = opcodes.Job(op_list=[op])
- jid = SubmitJob(job)
+ jid = SubmitJob(job, cl)
query = {
"object": "jobs",
@@ -389,7 +389,7 @@ def SubmitOpCode(op, proc=None, feedback_fn=None):
}
while True:
- jdata = SubmitQuery(query)
+ jdata = SubmitQuery(query, cl)
if not jdata:
# job not found, go away!
raise errors.JobLost("Job with id %s lost" % jid)
@@ -400,7 +400,7 @@ def SubmitOpCode(op, proc=None, feedback_fn=None):
time.sleep(1)
query["fields"].extend(["op_list", "op_status", "op_result"])
- jdata = SubmitQuery(query)
+ jdata = SubmitQuery(query, cl)
if not jdata:
raise errors.JobLost("Job with id %s lost" % jid)
status, op_list, op_status, op_result = jdata[0]
--
GitLab