-
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
provider/aws: Allow lightsail resources to work in other regions #14685
Conversation
Previously lightsail was limited to `us-east-1` only. This restriction has now been lifted to new regions. ``` $ make testacc TEST=./builtin/providers/aws TESTARGS='-run=TestAccAWSLightsailInstance_euRegion' ==> Checking that code complies with gofmt requirements... go generate $(go list ./... | grep -v /terraform/vendor/) 2017/05/19 16:40:48 Generated command/internal_plugin_list.go TF_ACC=1 go test ./builtin/providers/aws -v -run=TestAccAWSLightsailInstance_euRegion -timeout 120m === RUN TestAccAWSLightsailInstance_euRegion --- PASS: TestAccAWSLightsailInstance_euRegion (45.31s) PASS ok github.com/hashicorp/terraform/builtin/providers/aws 45.319s ``` Fixes: #14668
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, as long as we include a clear BC notice in the changelog once this is merged.
* provider/aws: Update Lightsail supported regions This commit complements (#14621)[#14621] and (#14685)[#14685]. * Revert "provider/aws: Update Lightsail supported regions" This reverts commit 545c3d6. * This commit complements #14621 and #14685. * Link to AWS docs instead of listing regions Instead of explicitly listing supported Lightsail regions in the docs, we now link to the Lightsail docs.
* provider/aws: Update Lightsail supported regions This commit complements (#14621)[#14621] and (#14685)[#14685]. * Revert "provider/aws: Update Lightsail supported regions" This reverts commit 545c3d6. * This commit complements #14621 and #14685. * Link to AWS docs instead of listing regions Instead of explicitly listing supported Lightsail regions in the docs, we now link to the Lightsail docs.
* provider/aws: Update Lightsail supported regions This commit complements (#14621)[hashicorp/terraform#14621] and (#14685)[hashicorp/terraform#14685]. * Revert "provider/aws: Update Lightsail supported regions" This reverts commit 545c3d6e6e7a9b665542ecc3b5e4d857faac749b. * This commit complements #14621 and #14685. * Link to AWS docs instead of listing regions Instead of explicitly listing supported Lightsail regions in the docs, we now link to the Lightsail docs.
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. |
Previously lightsail was limited to
us-east-1
only. This restriction has now been lifted to new regions.Fixes: #14668