Bug #15851

[controller] returning 'null' instead of '[]' for empty result lists

Added by Peter Amstutz 7 months ago. Updated 5 months ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
-
Target version:
Start date:
11/21/2019
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
-
Release relationship:
Auto

Description

federation is breaking here 22:42:23 /var/lib/cwl/stg7a0e7345-aadf-4b47-9dc1-253daca1b104/prepare.py
22:42:26 Traceback (most recent call last):
22:42:26 File "/var/lib/cwl/stg7a0e7345-aadf-4b47-9dc1-253daca1b104/prepare.py", line 32, in <module>
22:42:26 for c in cols["items"]:
22:42:26 TypeError: 'NoneType' object is not iterable

(09:22:33 AM) Fernando Monserrat: and run-cwl-test-xxx here 22:42:35 """, category=CWLToolDeprecationWarning)
22:42:35 ERROR Workflow error, try again with --debug for more information:
22:42:35 'items'

(09:24:34 AM) tetron: did we turn on the EnableBetaController around that time?

(09:31:33 AM) ward: tetron: Nov 6 17:06 UTC is when we set EnableBetaController14287: true on our dev clusters


Subtasks

Task #15859: Review 15851-empty-items-arrayResolvedPeter Amstutz

Associated revisions

Revision e087a608
Added by Tom Clegg 7 months ago

Merge branch '15851-empty-items-array'

fixes #15851

Arvados-DCO-1.1-Signed-off-by: Tom Clegg <>

Revision 607d2b11
Added by Tom Clegg 7 months ago

Merge branch '15851-bundle'

refs #15851

Arvados-DCO-1.1-Signed-off-by: Tom Clegg <>

History

#1 Updated by Peter Amstutz 7 months ago

  • Description updated (diff)

#2 Updated by Tom Clegg 7 months ago

  • Assigned To set to Tom Clegg

#5 Updated by Tom Clegg 7 months ago

  • Subject changed from [controller] Possibly returning 'null' instead of '[]' for empty result lists to [controller] returning 'null' instead of '[]' for empty result lists
  • Status changed from New to In Progress

#7 Updated by Peter Amstutz 7 months ago

Tom Clegg wrote:

15851-empty-items-array @ 92d7d779d4f0415d09f536d286972bd953e7b566 -- https://ci.curoverse.com/view/Developer/job/developer-run-tests/1660/

Confirmed with manual test arv collection list --filters '[["uuid", "in", []]]' and switching between branches, LGTM.

#8 Updated by Tom Clegg 7 months ago

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

#9 Updated by Peter Amstutz 5 months ago

  • Release set to 22

Also available in: Atom PDF