Skip to content

v3.14.0
Compare
Choose a tag to compare
@americk0 americk0 released this 16 Nov 15:27
· 0 commits to 4b30d6bf0877349ddcf083068065547fbf413a74 since this release
v3.14.0
fa2fd21

Description

We didn't have this work planned but I've been wanting to do this for some time now. This adds support for passing aliases anywhere that you can pass an account ID or account string. For example, before when calling alks sessions open you could pass the -a/--account and -r/--role and the value of the account field could be any of these options:

  • 012345678910
  • 012345678910/ALKSAdmin
  • 012345678910/ALKSAdmin - awstest

but now you can also pass just the alias (awstest), so you don't have to memorize account IDs anymore. To do this I added a function that looks up account info based on whatever you passed to get the account ID, alias, and label from ALKS. This added an extra call to get all accounts that a user has via the getAlksAccounts function, which slowed performance, so I added some very short-lived caching to that function call to speed things up since it was getting called multiple times per command

Rally # US852948: ALKS CLI - Accept alias in place of account ID

How has this been tested?

More unit tests were added to cover just about all the code that I touched, and I spent a while testing the commands manually to ensure that they still functioned correctly

Assets 2
Loading