Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Listing all accounts/scope while at VO "def" #86

Open
patrick-austin opened this issue Jun 22, 2020 · 1 comment
Open

Listing all accounts/scope while at VO "def" #86

patrick-austin opened this issue Jun 22, 2020 · 1 comment
Labels
bug Something isn't working priority: low

Comments

@patrick-austin
Copy link
Member

Motivation

Currently when listing accounts/scopes while at VO def (i.e. when super_root) we return all accounts/scopes irrespective of VO. Normally we'd filter using .like('%@tst') at the core, but as we don't append the VO when at def, the string used is just '%' which returns all accounts.

In itself this isn't a bad thing, but because we strip the VO from the end of account names at the API level the list returned is somewhat confusing, for example it can have multiple root accounts and there's no indication of their respective VO.

There are other places we use .like() that may exhibit similar behaviour.

Modification

  1. Do nothing, as this should only affect super_root who should/might not be running these kinds of commands on def anyway.

  2. Limit the results to def (i.e. we should only see super_root in the list of accounts.

  3. Include the VO in the returned list (for this set of circumstances only) so that super_root can use list functions to monitor across all VOs meaningfully.

Discuss at the next M-VO meeting? Course of action may depend on how we/users plan on using the M-VO setup.

@patrick-austin patrick-austin added bug Something isn't working priority: low labels Jun 22, 2020
@patrick-austin
Copy link
Member Author

This should only list the accounts etc. at def (option 2).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working priority: low
Projects
None yet
Development

No branches or pull requests

1 participant