Bug #17492

Exceptions thrown by the Ruby SDK should include the request id if possible

Added by Peter Amstutz 10 months ago. Updated 6 days ago.

Status:
In Progress
Priority:
Normal
Assigned To:
Category:
SDKs
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
(Total: 0.00 h)
Story points:
-

Subtasks

Task #17798: ReviewNewTom Clegg


Related issues

Blocked by Arvados - Bug #17830: Controller doesn't propagate back X-Request-Id headers on "new code path" endpointsResolved07/22/2021

History

#1 Updated by Peter Amstutz 9 months ago

  • Target version set to 2021-05-26 sprint

#2 Updated by Peter Amstutz 9 months ago

  • Target version changed from 2021-05-26 sprint to 2021-06-09 sprint

#3 Updated by Peter Amstutz 8 months ago

  • Target version changed from 2021-06-09 sprint to 2021-06-23 sprint

#4 Updated by Peter Amstutz 8 months ago

  • Subject changed from Errors should always include the request id if possible to Exceptions thrown by the Ruby SDK should include the request id if possible

#5 Updated by Peter Amstutz 8 months ago

  • Assigned To set to Lucas Di Pentima

#6 Updated by Lucas Di Pentima 7 months ago

  • Target version changed from 2021-06-23 sprint to 2021-07-07 sprint

#7 Updated by Lucas Di Pentima 7 months ago

  • Blocked by Bug #17830: Controller doesn't propagate back X-Request-Id headers on "new code path" endpoints added

#8 Updated by Lucas Di Pentima 7 months ago

  • Target version changed from 2021-07-07 sprint to 2021-07-21 sprint

#9 Updated by Lucas Di Pentima 6 months ago

  • Target version changed from 2021-07-21 sprint to 2021-08-04 sprint

#10 Updated by Lucas Di Pentima 6 months ago

  • Target version changed from 2021-08-04 sprint to 2021-08-18 sprint

#11 Updated by Lucas Di Pentima 5 months ago

  • Target version changed from 2021-08-18 sprint to 2021-09-01 sprint

#12 Updated by Lucas Di Pentima 5 months ago

  • Target version changed from 2021-09-01 sprint to 2021-09-15 sprint

#13 Updated by Lucas Di Pentima 4 months ago

  • Target version changed from 2021-09-15 sprint to 2021-09-29 sprint

#14 Updated by Peter Amstutz 4 months ago

  • Target version changed from 2021-09-29 sprint to 2021-10-13 sprint

#15 Updated by Lucas Di Pentima 3 months ago

  • Target version changed from 2021-10-13 sprint to 2021-10-27 sprint

#16 Updated by Lucas Di Pentima 3 months ago

  • Target version changed from 2021-10-27 sprint to 2021-11-24 sprint

#17 Updated by Lucas Di Pentima 2 months ago

  • Target version changed from 2021-11-24 sprint to 2021-12-08 sprint

#18 Updated by Lucas Di Pentima about 2 months ago

  • Status changed from New to In Progress

#19 Updated by Lucas Di Pentima about 2 months ago

  • Target version changed from 2021-12-08 sprint to 2022-01-05 sprint

#20 Updated by Lucas Di Pentima about 1 month ago

  • Target version changed from 2022-01-05 sprint to 2022-01-19 sprint

#21 Updated by Lucas Di Pentima 6 days ago

Updates at 1859bc3 - branch 17492-ruby-sdk-req-ids

Status update:

The arv CLI command currently implements its own GoogleAPIClient subclass to add custom behavior. I've replaced that subclass with a new one from the arvados gem.

The new Arvados::ArvadosClient class overrides the execute() method, passing a newly created X-Request-Id: req-xxxxx header to every request and also adding the request-id to the exception message if it's not there.

Pending:

  • Add some testing to this new ArvadosClient code. It would be ideal to do some unit testing to check the different failure modes, although it's not clear if it's a simple task.
  • The arv CLI testing machinery relies on installing a released version of the arvados gem, so the gem won't reflect any changes on the CLI test run until we do a proper release.

#22 Updated by Lucas Di Pentima 6 days ago

  • Target version changed from 2022-01-19 sprint to 2022-02-02 sprint

Also available in: Atom PDF