LGTM, thanks
On Mon, Dec 16, 2013 at 4:27 PM, Klaus Aehlig <[email protected]> wrote: > Luxid is more strict with closing the connection after receiving > a syntactically incorrect request, gnt-debug cannot use the same > client for several successive tests verifying that a syntactically > incorrect request is recognized as such. > > Signed-off-by: Klaus Aehlig <[email protected]> > --- > lib/client/gnt_debug.py | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > > diff --git a/lib/client/gnt_debug.py b/lib/client/gnt_debug.py > index da1dbef..e243a80 100644 > --- a/lib/client/gnt_debug.py > +++ b/lib/client/gnt_debug.py > @@ -188,9 +188,8 @@ def _TestJobDependency(opts): > """ > ToStdout("Testing job dependencies") > > - cl = cli.GetClient() > - > try: > + cl = cli.GetClient() > SubmitOpCode(opcodes.OpTestDelay(duration=0, depends=[(-1, None)]), > cl=cl) > except errors.GenericError, err: > if opts.debug: > @@ -218,6 +217,7 @@ def _TestJobDependency(opts): > ht.TOr(ht.TNonEmptyString, > ht.TJobId)]))) > > + cl = cli.GetClient() > result = cl.SubmitManyJobs(jobs) > if not check_fn(result): > raise errors.OpExecError("Job submission doesn't match %s: %s" % > @@ -287,8 +287,6 @@ def _TestJobSubmission(opts): > (4, 2, priority + offset), > ]) > > - cl = cli.GetClient() > - > for before, after, failpriority in testdata: > ops = [] > ops.extend([opcodes.OpTestDelay(duration=0) for _ in range(before)]) > @@ -296,6 +294,7 @@ def _TestJobSubmission(opts): > ops.extend([opcodes.OpTestDelay(duration=0) for _ in range(after)]) > > try: > + cl = cli.GetClient() > cl.SubmitJob(ops) > except errors.GenericError, err: > if opts.debug: > @@ -312,6 +311,7 @@ def _TestJobSubmission(opts): > ops, > ] > try: > + cl = cli.GetClient() > cl.SubmitManyJobs(jobs) > except errors.GenericError, err: > if opts.debug: > -- > 1.8.5.1 > >
