Project

General

Profile

Actions
Copy link

Bug #9118

closed

[SDK] arv-put unbound variable 'output' after exception from API server

Added by Peter Amstutz almost 8 years ago. Updated almost 8 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Brett Smith
Category:
-
Target version:
2016-05-11 sprint
Story points:
-

Description

When the API server raises an error, arv-put fails with UnboundLocalError: local variable 'output' referenced before assignment:

arv-put: Error creating Collection on project: <HttpError 422 when requesting https://gcam1.example.com/arvados/v1/collections?ensure_unique_name=true&alt=json returned "#<NoMemoryError: failed to allocate memory>">.
Traceback (most recent call last):
 File "/usr/local/bin/arv-put", line 4, in <module>
   main()
 File "/usr/local/lib/python2.7/dist-packages/arvados/commands/put.py", line 533, in main
   stdout.write(output)
UnboundLocalError: local variable 'output' referenced before assignment

Subtasks 1 (0 open — 1 closed)

Task #9127: Review 9118-arv-put-nameerror-fix-wipResolvedBrett Smith05/02/2016Actions
Actions
Copy link
 #1

Updated by Peter Amstutz almost 8 years ago

  • Description updated (diff)
Actions
Copy link
 #2

Updated by Brett Smith almost 8 years ago

  • Status changed from New to In Progress
  • Assigned To set to Brett Smith
  • Target version set to 2016-05-11 sprint
Actions
Copy link
 #3

Updated by Peter Amstutz almost 8 years ago

@ e365fa2

Is there a particular reason to catch NameException instead of something like:

output = None
...
if output is not None:
   ...
else:
  status = status or 1
Actions
Copy link
 #4

Updated by Brett Smith almost 8 years ago

Peter Amstutz wrote:

Is there a particular reason to catch NameException instead of something like:

No.

Actions
Copy link
 #5

Updated by Brett Smith almost 8 years ago

  • Status changed from In Progress to Resolved
  • % Done changed from 0 to 100

Applied in changeset arvados|commit:96aad215edcc0f314216c07df9246799665dd19d.

Actions
Copy link

Also available in: Atom PDF