-
Notifications
You must be signed in to change notification settings - Fork 9.1k
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
TechDebt: Migrate lightsail
service to aws-go-sdk-v2
#31853
TechDebt: Migrate lightsail
service to aws-go-sdk-v2
#31853
Conversation
…esource to aws-go-sdk-v2
# Conflicts: # internal/conns/awsclient_gen.go # internal/service/lightsail/bucket.go # internal/service/lightsail/bucket_access_key.go # internal/service/lightsail/bucket_access_key_test.go # internal/service/lightsail/bucket_resource_access.go # internal/service/lightsail/bucket_resource_access_test.go # internal/service/lightsail/bucket_test.go # internal/service/lightsail/certificate.go # internal/service/lightsail/certificate_test.go # internal/service/lightsail/container_service.go # internal/service/lightsail/container_service_deployment_version.go # internal/service/lightsail/container_service_deployment_version_test.go # internal/service/lightsail/container_service_test.go # internal/service/lightsail/database.go # internal/service/lightsail/database_test.go # internal/service/lightsail/disk.go # internal/service/lightsail/disk_attachment.go # internal/service/lightsail/disk_attachment_test.go # internal/service/lightsail/disk_test.go # internal/service/lightsail/distribution.go # internal/service/lightsail/distribution_test.go # internal/service/lightsail/domain.go # internal/service/lightsail/domain_entry.go # internal/service/lightsail/domain_entry_test.go # internal/service/lightsail/domain_test.go # internal/service/lightsail/instance.go # internal/service/lightsail/instance_public_ports.go # internal/service/lightsail/instance_public_ports_test.go # internal/service/lightsail/instance_test.go # internal/service/lightsail/key_pair.go # internal/service/lightsail/key_pair_test.go # internal/service/lightsail/lb.go # internal/service/lightsail/lb_attachment.go # internal/service/lightsail/lb_attachment_test.go # internal/service/lightsail/lb_certificate.go # internal/service/lightsail/lb_certificate_attachment.go # internal/service/lightsail/lb_certificate_test.go # internal/service/lightsail/lb_https_redirection_policy.go # internal/service/lightsail/lb_stickiness_policy.go # internal/service/lightsail/lb_stickiness_policy_test.go # internal/service/lightsail/lb_test.go # internal/service/lightsail/static_ip.go # internal/service/lightsail/static_ip_attachment.go # internal/service/lightsail/static_ip_attachment_test.go # internal/service/lightsail/static_ip_test.go # internal/service/lightsail/sweep.go # internal/service/lightsail/tags_gen.go
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 🚀.
% make testacc TESTARGS='-run=TestAccLightsail.*_basic$$' PKG=lightsail ACCTEST_PARALLELISM=2
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test ./internal/service/lightsail/... -v -count 1 -parallel 2 -run=TestAccLightsail.*_basic$ -timeout 180m
=== RUN TestAccLightsailBucketAccessKey_basic
=== PAUSE TestAccLightsailBucketAccessKey_basic
=== RUN TestAccLightsailBucketResourceAccess_basic
=== PAUSE TestAccLightsailBucketResourceAccess_basic
=== RUN TestAccLightsailBucket_basic
=== PAUSE TestAccLightsailBucket_basic
=== RUN TestAccLightsailCertificate_basic
=== PAUSE TestAccLightsailCertificate_basic
=== RUN TestAccLightsailContainerServiceDeploymentVersion_container_basic
=== PAUSE TestAccLightsailContainerServiceDeploymentVersion_container_basic
=== RUN TestAccLightsailContainerService_basic
=== PAUSE TestAccLightsailContainerService_basic
=== RUN TestAccLightsailDatabase_basic
=== PAUSE TestAccLightsailDatabase_basic
=== RUN TestAccLightsailDiskAttachment_basic
=== PAUSE TestAccLightsailDiskAttachment_basic
=== RUN TestAccLightsailDisk_basic
=== PAUSE TestAccLightsailDisk_basic
=== RUN TestAccLightsailDomainEntry_basic
=== PAUSE TestAccLightsailDomainEntry_basic
=== RUN TestAccLightsailDomain_basic
=== PAUSE TestAccLightsailDomain_basic
=== RUN TestAccLightsailInstancePublicPorts_basic
=== PAUSE TestAccLightsailInstancePublicPorts_basic
=== RUN TestAccLightsailInstance_basic
=== PAUSE TestAccLightsailInstance_basic
=== RUN TestAccLightsailKeyPair_basic
=== PAUSE TestAccLightsailKeyPair_basic
=== RUN TestAccLightsailStaticIPAttachment_basic
=== PAUSE TestAccLightsailStaticIPAttachment_basic
=== RUN TestAccLightsailStaticIP_basic
=== PAUSE TestAccLightsailStaticIP_basic
=== CONT TestAccLightsailBucketAccessKey_basic
=== CONT TestAccLightsailDisk_basic
--- PASS: TestAccLightsailBucketAccessKey_basic (59.52s)
=== CONT TestAccLightsailInstance_basic
--- PASS: TestAccLightsailDisk_basic (68.81s)
=== CONT TestAccLightsailStaticIP_basic
--- PASS: TestAccLightsailStaticIP_basic (23.10s)
=== CONT TestAccLightsailStaticIPAttachment_basic
--- PASS: TestAccLightsailInstance_basic (69.52s)
=== CONT TestAccLightsailKeyPair_basic
--- PASS: TestAccLightsailKeyPair_basic (33.46s)
=== CONT TestAccLightsailContainerServiceDeploymentVersion_container_basic
--- PASS: TestAccLightsailStaticIPAttachment_basic (97.09s)
=== CONT TestAccLightsailDiskAttachment_basic
--- PASS: TestAccLightsailDiskAttachment_basic (218.95s)
=== CONT TestAccLightsailDatabase_basic
--- PASS: TestAccLightsailContainerServiceDeploymentVersion_container_basic (505.36s)
=== CONT TestAccLightsailContainerService_basic
--- PASS: TestAccLightsailContainerService_basic (120.14s)
=== CONT TestAccLightsailBucket_basic
--- PASS: TestAccLightsailBucket_basic (39.70s)
=== CONT TestAccLightsailCertificate_basic
--- PASS: TestAccLightsailCertificate_basic (58.28s)
=== CONT TestAccLightsailBucketResourceAccess_basic
--- PASS: TestAccLightsailBucketResourceAccess_basic (125.06s)
=== CONT TestAccLightsailDomain_basic
acctest.go:848: skipping tests; AWS_DEFAULT_REGION (us-west-2) not supported. Supported: [us-east-1]
--- SKIP: TestAccLightsailDomain_basic (0.00s)
=== CONT TestAccLightsailInstancePublicPorts_basic
--- PASS: TestAccLightsailInstancePublicPorts_basic (77.94s)
=== CONT TestAccLightsailDomainEntry_basic
acctest.go:848: skipping tests; AWS_DEFAULT_REGION (us-west-2) not supported. Supported: [us-east-1]
--- SKIP: TestAccLightsailDomainEntry_basic (0.00s)
--- PASS: TestAccLightsailDatabase_basic (955.82s)
PASS
ok github.com/hashicorp/terraform-provider-aws/internal/service/lightsail 1370.575s
@brittandeyoung Thanks for the contribution 🎉 👏. |
This functionality has been released in v5.3.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. Thank you! |
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. |
Description
This pull request upgrades the lightsail service to aws go sdk v2 and updates all resources to work with the new sdk.
Relations
Closes #0000
References
Output from Acceptance Testing