Project

General

Profile

Actions
Copy link

Bug #15933

closed

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

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

Status:
Resolved
Priority:
Normal
Assigned To:
Peter Amstutz
Category:
-
Target version:
2020-01-02 Sprint
Story points:
-
Release:
Arvados 2.0 - Early 2020
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 1 (0 open — 1 closed)

Task #15939: Review 15933-setup-uuid-paramResolvedPeter Amstutz12/17/2019Actions
Actions
Copy link
 #1

Updated by Peter Amstutz almost 5 years ago

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

Updated by Peter Amstutz almost 5 years ago

  • Description updated (diff)
Actions
Copy link
 #3

Updated by Peter Amstutz almost 5 years ago

  • Target version changed from 2020-01-02 Sprint to 2020-01-15 Sprint
Actions
Copy link
 #4

Updated by Tom Clegg almost 5 years 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.

Actions
Copy link
 #5

Updated by Peter Amstutz almost 5 years 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.

Actions
Copy link
 #6

Updated by Peter Amstutz almost 5 years ago

  • Description updated (diff)
Actions
Copy link
 #7

Updated by Peter Amstutz almost 5 years 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
Actions
Copy link
 #8

Updated by Peter Amstutz almost 5 years ago

15933-setup-uuid-param @ 7e77dad2912c2eee85cc5e5876f35cf55a4cfc69

  • Add 'uuid' parameter
  • Remove unused 'openid_prefix' parameter
Actions
Copy link
 #9

Updated by Lucas Di Pentima almost 5 years 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.

Actions
Copy link
 #10

Updated by Peter Amstutz almost 5 years ago

  • Status changed from In Progress to Resolved
Actions
Copy link
 #11

Updated by Peter Amstutz almost 5 years ago

  • Release set to 22
Actions
Copy link

Also available in: Atom PDF