Skip to content

Commit b42ff8f

Browse files
committed
testsys-launcher(docs): update README to use CFN context
the stack switched from using parameterized roles for assuming the testsys-admin role to assuming this role from context. Update the README instructions for deploying to reflect this. Signed-off-by: Gavin Inglis <[email protected]>
1 parent fad39d2 commit b42ff8f

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

deploy/testsys-launcher/README.md

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,9 @@ cdk deploy
1313
```
1414

1515
The default role that can assume the `testsys-admin` role is "Administrator".
16-
During deployment, use `--parameter TestsysAssumerRole={MY-ROLE-NAME}` to specify
17-
a different role that can assume the `testsys-admin` role.
16+
To specify other roles to assume `testsys-admin`, modify `cdk.json` or use
17+
a comma separated set of roles via CDK context like
18+
`--context testsysAdminAssumedBy=<role1>,<role2>,<role3>`.
1819

1920
To get the kubeconfig, assuming you are acting as the role that can assume `testsys-admin`,
2021
use the `aws eks update-kubeconfig` command with the `testsys-admin` role:
@@ -42,5 +43,8 @@ Bottlerocket nodes on the cluster consume the latest releases and stay up to dat
4243

4344
## Optional Parameters
4445

45-
* `TestsysAssumerRole` name of the role that can assume the `testsys-admin` role to perform cluster operations _(string)_ - Default: "Administrator"
4646
* `TestsysNodegroupSize` number of instances for the testsys node-group _(number)_ - Default: "3"
47+
48+
## Optional Context
49+
50+
* `testsysAdminAssumedBy`: a comma separated list of roles that can assume the `testsys-admin` role - Default: "Administrator"

0 commit comments

Comments
 (0)