search

stfc / rucio

Rucio - Scientific Data Management
http://rucio.cern.ch
Apache License 2.0
0 stars 0 forks source link

user-interaction: Added VO option to the CLI #60 #63

Closed patrick-austin closed 4 years ago

patrick-austin commented 4 years ago

Added VO option to the CLI

Added the option to specify which VO you wish to authenticate against to the CLI (and admin CLI). If unspecified, will default to the previous behaviour of going off the config file. If a VO is given that the user doesn't have an account at authentication will fail.

However, this will run into issues if [client] auth_token_file_path is set, as in this case we use a single file for tokens (as specified by the user) which means attempts to swap VO will fail as the existing token is just used and whatever account it belonged to ends up being used (until that token expires). However this is also the case for swapping accounts at the same VO, so this could be intended/accepted behaviour. Discuss with Martin?

patrick-austin commented 4 years ago

It won't let me directly respond to your comment, but I thought we were going to add it to the token path directory rather than the file(s) themselves. But otherwise the change was fine.

elichad commented 4 years ago

I thought we were going to add it to the token path directory rather than the file(s) themselves.

Ah yes that's right, my mistake