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

Bump L0 API instance to t3.small #652

Closed
alqvia opened this issue Oct 30, 2019 · 4 comments · May be fixed by #653
Closed

Bump L0 API instance to t3.small #652

alqvia opened this issue Oct 30, 2019 · 4 comments · May be fixed by #653
Assignees

Comments

@alqvia
Copy link
Collaborator

alqvia commented Oct 30, 2019

Expected behavior

Deploying to us-west-2 succeeds with all AZs.

Actual behavior

Deploying L0 instance to us-west-2 gives the following error message:

Description:Description
Launching a new EC2 instance. 
Status Reason: Your requested instance type (t2.small) is not supported in your requested Availability Zone (us-west-2d). 
Please retry your request by not specifying an Availability Zone or choosing us-west-2a, us-west-2b, us-west-2c. Launching EC2 instance failed.

Currently, the EC2 instance_type defined in aws_launch_configuration (environment.tf) is set to t2.small, which is no longer supported by the us-west-2d AZ. t2.small is an older type that should be replaced with t3.small.

Steps to reproduce the behavior

Deploy L0 to us-west-2

@alqvia alqvia self-assigned this Oct 30, 2019
@alqvia alqvia changed the title Bump L0 API instance to t3 Bump L0 API instance to t3.small Oct 30, 2019
@diemonster
Copy link
Contributor

@alqvia we suspect this is a terraform-specific bug. If you run l0-setup apply twice, the second time running will apply another change including the 2d subnets into the VPC configuration.

If you follow the suggestion above, do you still get the same error message?

for reference, a similar error: terraform-aws-modules/terraform-aws-ec2-instance#68

@alqvia
Copy link
Collaborator Author

alqvia commented Oct 30, 2019

I'll give it a try

@alqvia
Copy link
Collaborator Author

alqvia commented Oct 30, 2019

Okay, so this seemed to work on the first l0-setup apply...

@alqvia alqvia closed this as completed Oct 31, 2019
@alqvia
Copy link
Collaborator Author

alqvia commented Oct 31, 2019

Re-running init and apply worked.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants