Project

General

Profile

Actions

Bug #16104

closed

Can't load properties for tag editing in wb1

Added by Peter Amstutz about 4 years ago. Updated about 4 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
API
Target version:
Story points:
-

Description

It just says "Loading tags..." forever.

https://workbench.ce8i5.arvadosapi.com/collections/ce8i5-4zz18-m1nql1u0n2hooqt#Tags

(From chat) might be a different error, on requesting the collection record:

Jan 30 19:03:20 ce8i5.arvadosapi.com arvados-controller[69348]: {"PID":69348,"RequestID":"req-td85nh8ihu6u1c6kzauv","level":"info","msg":"request","remoteAddr":"127.0.0.1:42758","reqBytes":0,"reqForwardedFor":"152.170.154.130","reqHost":"ce8i5.arvadosapi.com","reqMethod":"GET","reqPath":"arvados/v1/collections","reqQuery":"filters=%5B%5B%22uuid%22%2C%22%3D%22%2C%22ce8i5-4zz18-m1nql1u0n2hooqt%22%5D%5D\u0026select=%5B%22properties%22%5D","time":"2020-01-30T19:03:20.364084667Z"}
Jan 30 19:03:20 ce8i5.arvadosapi.com arvados-controller[69348]: {"PID":69348,"RequestID":"req-td85nh8ihu6u1c6kzauv","level":"info","msg":"response","remoteAddr":"127.0.0.1:42758","reqBytes":0,"reqForwardedFor":"152.170.154.130","reqHost":"ce8i5.arvadosapi.com","reqMethod":"GET","reqPath":"arvados/v1/collections","reqQuery":"filters=%5B%5B%22uuid%22%2C%22%3D%22%2C%22ce8i5-4zz18-m1nql1u0n2hooqt%22%5D%5D\u0026select=%5B%22properties%22%5D","respBody":"{\"errors\":[\"cannot make progress in federated list query: cluster \\\"ce8i5\\\" returned 1 items but none had the requested UUIDs\"]}\n","respBytes":129,"respStatus":"Bad Gateway","respStatusCode":502,"time":"2020-01-30T19:03:20.377458078Z","timeToStatus":0.013358,"timeTotal":0.013371,"timeWriteBody":0.000012}

Seems to be a controller bug.


Subtasks 1 (0 open1 closed)

Task #16105: Review 16104-select-uuidResolvedPeter Amstutz02/03/2020Actions

Related issues

Blocks Arvados - Idea #16015: Release 2.0ResolvedPeter Amstutz01/28/2020Actions
Actions #1

Updated by Peter Amstutz about 4 years ago

  • Description updated (diff)
Actions #2

Updated by Peter Amstutz about 4 years ago

  • Description updated (diff)
Actions #3

Updated by Peter Amstutz about 4 years ago

  • Description updated (diff)
Actions #4

Updated by Peter Amstutz about 4 years ago

  • Description updated (diff)
Actions #5

Updated by Peter Amstutz about 4 years ago

  • Assigned To set to Tom Clegg
  • Description updated (diff)
  • Subject changed from If vocabulary.json is not found tags cannot be edited to Can't load properties for tag editing in wb1
Actions #6

Updated by Peter Amstutz about 4 years ago

  • Target version changed from 2020-02-26 Sprint to 2020-02-12 Sprint
Actions #7

Updated by Peter Amstutz about 4 years ago

Actions #8

Updated by Tom Clegg about 4 years ago

  • Status changed from New to In Progress
  • Category changed from Workbench to API
Actions #9

Updated by Tom Clegg about 4 years ago

The federated query handler (triggered by filters=[["uuid","in",...]]) was allowing a deselected UUID column because there was only one cluster to query -- but the single-cluster query was handled by the generic case anyway, which hit the "no progress" problem responsible for the "can't deselect UUID" rule.

Fixed two issues:
  1. Do a single request to the local cluster (rather than using the generic case) when all UUIDs are local. This allows queries like filters=[[uuid,in,[x,y,z]]], order=uuid, limit=2 to work properly -- instead of skipping the "allowed query" checks but then falling through to the merge procedure that depends on those checks.
  2. Remove the "can't deselect uuid" rule. Instead, add "uuid" to the select param when doing remote queries. This means internally the response will have the UUID field populated, but the "uuid" field will still disappear in the HTTP response because that is handled by source:lib/controller/router.

16104-select-uuid @ 4d969d35d1e1f022bd5ddcebb2de915bedd01334 -- developer-run-tests: #1713

Actions #11

Updated by Tom Clegg about 4 years ago

  • Status changed from In Progress to Resolved
Actions

Also available in: Atom PDF