aws / aws-sdk

Landing page for the AWS SDKs on GitHub
https://aws.amazon.com/tools/
Other
71 stars 14 forks source link

VPCRegion of route53 service lacks support of cn-northwest-1 #787

Open Monster-Zhu opened 1 month ago

Monster-Zhu commented 1 month ago

Acknowledgements

Describe the bug

The type VPCRegion in route53 service lacks support of "cn-northwest-1" which prevents us from creating aws_route53_zone_association and aws_route53_vpc_association_authorization resources in cn-northwest-1 region by terraform which relies on aws-sdk-go-v2 to check the input of vpc_region:

Expected Behavior

aws_route53_zone_association and aws_route53_vpc_association_authorization can be created successfully

Current Behavior

aws_route53_zone_association and aws_route53_vpc_association_authorization failed to create with error:

│ Error: expected vpc_region to be one of ["us-east-1" "us-east-2" "us-west-1" "us-west-2" "eu-west-1" "eu-west-2" "eu-west-3" "eu-central-1" "eu-central-2" "ap-east-1" "me-south-1" "us-gov-west-1" "us-gov-east-1" "us-iso-east-1" "us-iso-west-1" "us-isob-east-1" "me-central-1" "ap-southeast-1" "ap-southeast-2" "ap-southeast-3" "ap-south-1" "ap-south-2" "ap-northeast-1" "ap-northeast-2" "ap-northeast-3" "eu-north-1" "sa-east-1" "ca-central-1" "cn-north-1" "af-south-1" "eu-south-1" "eu-south-2" "ap-southeast-4" "il-central-1" "ca-west-1"], got cn-northwest-1

Reproduction Steps

Refer to: https://github.com/hashicorp/terraform-provider-aws/issues/38403

Possible Solution

No response

Additional Information/Context

No response

AWS Go SDK V2 Module Versions Used

github.com/aws/aws-sdk-go-v2/service/route53 v1.42.3 github.com/aws/aws-sdk-go-v2 v1.30.3

Compiler and Version used

1.22.5

Operating System and version

macOS Sonoma 14.5

RanVaknin commented 1 month ago

Hi @Monster-Zhu ,

Thanks for opening this issue. I can see that this enum value is in fact missing from VPCRegion.

I have created an internal ticket with the R53 service team to fix their model (#P142774903)

Moving to Cross SDK repo for tracking. Thanks, Ran~