Bug #15933

[API] user setup method missing "uuid" parameter in discovery document

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

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

100%

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

Description

User setup can't be done from command line because it is missing the "uuid" parameter.

Also, it accepts a "user" object parameter but I don't think it does anything. It should be removed.

Also, trying to invoke "setup" gave a weird error, apparently it turned the 'setup' request into a 'update_uuid' request. Maybe controller is rewriting the request?

{"errors":["request failed: http://localhost:8004/arvados/v1/users/ce8i5-tpzed-6oia07y9o4zvtl6/update_uuid: 422 Unprocessable Entity: #\u003cArgumentError: new_uuid parameter is required\u003e (req-92kuwtcxxmpmz2i94jr7)"]}

Subtasks

Task #15939: Review 15933-setup-uuid-paramResolvedPeter Amstutz

Associated revisions

Revision f3c89480
Added by Peter Amstutz 7 months ago

Merge branch '15933-setup-uuid-param' refs #15933

Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <>

History

#1 Updated by Peter Amstutz 7 months ago

  • Description updated (diff)
  • Subject changed from [API] "setup" missing uuid parameter to [API] user setup method missing "uuid" parameter in discovery document

#2 Updated by Peter Amstutz 7 months ago

  • Description updated (diff)

#3 Updated by Peter Amstutz 7 months ago

  • Target version changed from 2020-01-02 Sprint to 2020-01-15 Sprint

#4 Updated by Tom Clegg 7 months ago

Also, trying to invoke "setup" gave a weird error, apparently it turned the 'setup' request into a 'update_uuid' request. Maybe controller is rewriting the request?

This was fixed in #15922.

#5 Updated by Peter Amstutz 7 months ago

Also, it accepts a "user" object parameter but I don't think it does anything. It should be removed.

Actually it does still do something with the user object, it creates a new user with the supplied attributes and then set up the user.

#6 Updated by Peter Amstutz 7 months ago

  • Description updated (diff)

#7 Updated by Peter Amstutz 7 months ago

  • Target version changed from 2020-01-15 Sprint to 2020-01-02 Sprint
  • Assigned To set to Peter Amstutz
  • Status changed from New to In Progress

#8 Updated by Peter Amstutz 7 months ago

15933-setup-uuid-param @ 7e77dad2912c2eee85cc5e5876f35cf55a4cfc69

  • Add 'uuid' parameter
  • Remove unused 'openid_prefix' parameter

#9 Updated by Lucas Di Pentima 7 months ago

The changes LGTM. I've tried several times to start a new arvbox instance but for some reason the API server is failing, but don't think it's related to this branch:

...
2019-12-17_22:40:49.57017 + test -a /usr/src/arvados/services/api/config/arvados_config.rb
2019-12-17_22:40:49.57019 + rm -f config/application.yml config/database.yml
2019-12-17_22:40:49.58532 + test -f /var/lib/arvados/api_database_setup
2019-12-17_22:40:49.58534 + bundle exec rake db:setup
2019-12-17_22:40:51.29601 rake aborted!
2019-12-17_22:40:51.30017 LoadError: cannot load such file -- bundler/setup
2019-12-17_22:40:51.30030 /usr/src/arvados/services/api/config/boot.rb:8:in `require'
2019-12-17_22:40:51.30030 /usr/src/arvados/services/api/config/boot.rb:8:in `<top (required)>'
2019-12-17_22:40:51.30031 /usr/src/arvados/services/api/config/application.rb:5:in `require'
...

Local services/api test run was successful, please merge.

#10 Updated by Peter Amstutz 7 months ago

  • Status changed from In Progress to Resolved

#11 Updated by Peter Amstutz 6 months ago

  • Release set to 22

Also available in: Atom PDF