AWS Wavelength support

[ewbankkit]

Community Note

• Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
• Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request
• If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description

AWS Wavelength is now available in us-east-1 and us-west-2.

New or Affected Resource(s)

New

• aws_ec2_carrier_gateway (version 3.21.0)

Updated

• aws_route_table (version 3.35.0)
• aws_route (version 3.35.0)
• aws_eip (network_border_group added in version 3.14.0, carrier_ip_address in version 3.22.0)
• aws_launch_template (version 3.21.0)
• aws_network_interface (version 3.22.0)
• aws_availability_zone (version 3.22.0)

aws_default_route_table

Error: error creating route: RouteNotSupported: Routes that target carrier gateways are not supported in main route table.

Potential Terraform Configuration

resource "aws_ec2_carrier_gateway" "example" {
  vpc_id = aws_vpc.example.id

  tags = {
    Name = "Example"
  }
}

References

Announcement.
Blog post.

Requires AWS SDK v1.33.21:

• Update module aws/aws-sdk-go to v1.33.21 #14000

Related:

• aws_eip support new attributes NetworkBorderGroup and address #13998
• r/aws_eip - support NetworkBorderGroup attribute #14028

[jukie]

I can work on this if you haven't already started @ewbankkit

[ewbankkit]

@jukie If you have bandwidth, a PR for the the aws_ec2_carrier_gateway resource (and then one for the corresponding data source) would be great.
I'm not sure if you need to have an account with Wavelength enabled to actually create a Carrier Gateway.
I will tackle the routing enhancements as part of another set of PRs I am working on.
Thanks.

[FireballDWF]

@ewbankkit If you have made any progress and would like someone to help test or troubleshoot, I'm willing. I'm AWS ProServe and willing to help push things within AWS if you need any support.

[ewbankkit]

@FireballDWF Thanks for the offer.
Would you be able to suggest a priority order for the changes?

[FireballDWF]

route target of carrier gateway would be next and will prioritize the remainder tonight

On Tue, Nov 17, 2020 at 10:21 AM Kit Ewbank ***@***.***> wrote: @FireballDWF <https://github.com/FireballDWF> Thanks for the offer. Would you be able to suggest a priority order for the changes? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#14518 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAAEWSKILPJLWMKYQGJG3MLSQKIH3ANCNFSM4PX52M4A> .

-- Sent from Gmail Mobile

[FireballDWF]

Proposed priority

1. aws_route
2. aws_route_table
3. aws_eip
4. aws_launch_template
5. aws_network_interface
6. aws_default_route_table

[FireballDWF]

I received access an hour ago to a wavelength account. I can attempt to build on Saturday morning. If you can include the aws_route resource change, that is sufficient for a minimal practical test.

[ewbankkit]

@FireballDWF I have added (hopefully) sufficient support to aws_route in #16252.
Please add any results to that PR.
Thanks.

[bflad]

Support for the new aws_ec2_carrier_gateway resource has been merged and will release with version 3.21.0 of the Terraform AWS Provider, likely tomorrow. Thanks to @ewbankkit for the implementation.

[ewbankkit]

The aws_eip resource changes for Wavelength (adding network_border_group attribute) was implemented via #14028, released in version 3.14.0 of the Terraform AWS Provider.

[FireballDWF]

@FireballDWF I have added (hopefully) sufficient support to aws_route in #16252.
Please add any results to that PR.
Thanks.

I did not see any changes to aws_route in that PR, and don't see the changes for aws_route anywhere else in the latest release. However I successfully tested aws_ec2_carrier_gateway in the latest release.

[ewbankkit]

@FireballDWF I pulled out the commit that updated aws_route before #16252 was merged so as reduce the scope to one resource per PR.
I have a couple of precursor PRs (#14014 and then #16930) that could be merged first for aws_route or we could add the (relatively minor) change to add a carrier_gateway_id attribute on top of the current code - Brian can make that call.

[ghost]

This has been released in version 3.35.0 of the Terraform AWS provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading.

For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template for triage. Thanks!

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. Thanks!