-
Notifications
You must be signed in to change notification settings - Fork 9.6k
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 Elastic File System DNS name has (silently?) changed #10902
Comments
FWIW, this change was discreetly announced last week https://aws.amazon.com/fr/about-aws/whats-new/2016/12/simplified-mounting-of-amazon-efs-file-systems/ |
Hi @aisamu thanks for the bug report here - can you tell me if Thanks Paul |
Hey! Sadly, it doesn't =( |
Found the error! It will be fixed as of 0.8.3 :) |
Thanks for such awesome tool! |
Fixes:#10902 AWS introduced a change to the Mount Target DNS Name to remove the availability_zone from it - https://aws.amazon.com/about-aws/whats-new/2016/12/simplified-mounting-of-amazon-efs-file-systems/ This was because there used to be a limit of 1 mount target per AZ - this has been raised. ``` % make testacc TEST=./builtin/providers/aws TESTARGS='-run=TestAccAWSEFSMountTarget_' ==> Checking that code complies with gofmt requirements... go generate $(go list ./... | grep -v /terraform/vendor/) 2017/01/04 10:45:35 Generated command/internal_plugin_list.go TF_ACC=1 go test ./builtin/providers/aws -v -run=TestAccAWSEFSMountTarget_ -timeout 120m === RUN TestAccAWSEFSMountTarget_importBasic --- PASS: TestAccAWSEFSMountTarget_importBasic (236.19s) === RUN TestAccAWSEFSMountTarget_basic --- PASS: TestAccAWSEFSMountTarget_basic (445.52s) === RUN TestAccAWSEFSMountTarget_disappears --- PASS: TestAccAWSEFSMountTarget_disappears (228.31s) PASS ok github.com/hashicorp/terraform/builtin/providers/aws 910.044s ```
Fixes:#10902 AWS introduced a change to the Mount Target DNS Name to remove the availability_zone from it - https://aws.amazon.com/about-aws/whats-new/2016/12/simplified-mounting-of-amazon-efs-file-systems/ This was because there used to be a limit of 1 mount target per AZ - this has been raised. ``` % make testacc TEST=./builtin/providers/aws TESTARGS='-run=TestAccAWSEFSMountTarget_' ==> Checking that code complies with gofmt requirements... go generate $(go list ./... | grep -v /terraform/vendor/) 2017/01/04 10:45:35 Generated command/internal_plugin_list.go TF_ACC=1 go test ./builtin/providers/aws -v -run=TestAccAWSEFSMountTarget_ -timeout 120m === RUN TestAccAWSEFSMountTarget_importBasic --- PASS: TestAccAWSEFSMountTarget_importBasic (236.19s) === RUN TestAccAWSEFSMountTarget_basic --- PASS: TestAccAWSEFSMountTarget_basic (445.52s) === RUN TestAccAWSEFSMountTarget_disappears --- PASS: TestAccAWSEFSMountTarget_disappears (228.31s) PASS ok github.com/hashicorp/terraform/builtin/providers/aws 910.044s ```
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 have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further. |
Hi there!
It turns out, as per the AWS Console and new EC2 mounting instructions, AWS has (silently?) dropped the availability zone from the EFS DNS name.
The former
us-east-1a.fs-xxxxxxxx.efs.us-east-1.amazonaws.com
becomes justfs-xxxxxxxx.efs.us-east-1.amazonaws.com
.Unfortunately,
aws_efs_mount_target.xxx.dns_name
still outputs the old format.For the moment, just dropping the AZ from the DNS works, as in
replace(aws_efs_mount_target.xxx.dns_name, "/\\A[^\\.]*\\./", "")
, but I figured it'd be useful to let you know!Thanks a lot for this great tool!
Affected Resource(s)
The text was updated successfully, but these errors were encountered: