Skip to content

Commit

Permalink
Fix bastian role (#23)
Browse files Browse the repository at this point in the history
* The AmazonSSMManagedInstanceCore was missing from the instance. Change
from using CDK level 1 to a level 2 construct seems to fix it.
* Existing agora instances are using `agora-access` keypair.
  • Loading branch information
zaro0508 authored Jan 30, 2025
1 parent 48504c7 commit 73c5f3d
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion app.py
Original file line number Diff line number Diff line change
Expand Up @@ -179,7 +179,7 @@
apex_stack.add_dependency(api_stack)

bastion_props = BastionProps(
key_name="agora-ci",
key_name="agora-access",
instance_type=ec2.InstanceType.of(ec2.InstanceClass.T3, ec2.InstanceSize.MICRO),
ami_id="ami-074a6fac5773fe883",
ami_region="us-east-1",
Expand Down
4 changes: 2 additions & 2 deletions src/bastion_stack.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,8 +33,8 @@ def __init__(
)
],
)
instance_profile = iam.CfnInstanceProfile(
self, "BastionInstanceProfile", roles=[role.role_name]
instance_profile = iam.InstanceProfile(
self, "BastionInstanceProfile", role=role
)

key_pair = ec2.KeyPair.from_key_pair_name(
Expand Down

0 comments on commit 73c5f3d

Please sign in to comment.