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

aws_dx_private_virtual_interface: 8500 MTU #8841

Closed
midacts opened this issue Jun 3, 2019 · 4 comments
Closed

aws_dx_private_virtual_interface: 8500 MTU #8841

midacts opened this issue Jun 3, 2019 · 4 comments

Comments

@midacts
Copy link

midacts commented Jun 3, 2019

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 "me too" comments, 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

Terraform Version

Terraform v0.11.10
provider.aws v2.12.0

Affected Resource(s)

  • aws_dx_private_virtual_interface

Terraform Configuration Files

resource "aws_dx_private_virtual_interface" "ex" {
  connection_id   = "${aws_dx_connection.ex.id}"
  name            = "ex"
  vlan            = 100
  address_family  = "ipv4"
  bgp_asn         = 64xxx
  mtu             = 8500
  dx_gateway_id   = "xxx"
}

Debug Output

Error: aws_dx_private_virtual_interface.ex: expected mtu to be one of [1500 9001], got 8500

Expected Behavior

Create/import the dx vif with jumbo frames (8500 MTU)

Actual Behavior

Errors out

Important Factoids

image

@ewbankkit
Copy link
Contributor

Hi @midacts, Is this a private virtual interface or transit virtual interface that you're trying to import?
The AWS Direct Connect docs are a bit hard to parse exactly but my understanding was that an MTU of 8500 was only for transit VIFs, where we are supporting 8500 MTUs in #8522.

@midacts
Copy link
Author

midacts commented Jun 3, 2019

@ewbankkit Thanks for the response. I think you are right.
Thanks for the link to the open issue. It makes sense that that is the option (I didn't know it was an option because it technically does not exist).

Want me to close this issue? I should be able to watch #8522

@ewbankkit
Copy link
Contributor

@midacts Thanks. Please 👍 #8522 if you didn't already.
We ought to add code to ensure that the terraform import is actually importing the correct type of VIF (private/public/transit) - I'll open an issue for that and link to this one.
Yes, please go ahead and close this issue.

@ghost
Copy link

ghost commented Mar 29, 2020

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!

@ghost ghost locked and limited conversation to collaborators Mar 29, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants