feat(route53): latency based routing

[badmintoncryer]

In this PR, I have made the RecordSet compatible with latency-based routing.
You can set it by providing a Region name to the region argument.

new route53.ARecord(this, 'ARecord', {
  zone: myZone,
  target: route53.RecordTarget.fromIpAddresses('1.2.3.4'),
  region: 'ap-northeast-1', // added
})

Closes #28722.

---

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

[aws-cdk-automation]

The pull request linter has failed. See the aws-cdk-automation comment below for failure reasons. If you believe this pull request should receive an exemption, please comment and provide a justification.

A comment requesting an exemption should contain the text Exemption Request. Additionally, if clarification is needed add Clarification Request to a comment.

✅ Updated pull request passes all PRLinter validations. Dismissing previous PRLinter review.

[lpizzinidev]

Looks good overall 👍
I left some notes for minor adjustments.

[badmintoncryer]

@lpizzinidev Thank you for your review! I have addressed your comments.

[lpizzinidev]

Nice 👍

[aaythapa]

Thank you for your contribution! Looks great overall, only concern is should the region names be enum values? The docs clearly define which regions are allowed in this property so maybe we should do the same

[badmintoncryer]

@aaythapa Thank you for your review. I'll add enum!

[aws-cdk-automation]

The pull request linter has failed. See the aws-cdk-automation comment below for failure reasons. If you believe this pull request should receive an exemption, please comment and provide a justification.

A comment requesting an exemption should contain the text Exemption Request. Additionally, if clarification is needed add Clarification Request to a comment.

✅ Updated pull request passes all PRLinter validations. Dismissing previous PRLinter review.

[badmintoncryer]

@aaythapa I've added LatencyBasedRoutingRegion. Please comfirm it.

[aaythapa]

@aaythapa I've added LatencyBasedRoutingRegion. Please confirm it.

Hi @badmintoncryer thank you for making the changes. I talked it over with the team and we decided we don't want to add enums for regions. Our reasoning is users already use strings when defining region in CDK, for example here and with references like stack.env.region. On top of that, new regions are constantly being added so we need to manually maintain the list of enums which add additional overhead.

Sorry about the extra work, I should've discussed this with the team before suggesting you add enums. Hopefully it isn't too much work to go back.

[badmintoncryer]

@aaythapa I've reverted modification! Please confirm it again.

[aws-cdk-automation]

AWS CodeBuild CI Report

• CodeBuild project: AutoBuildv2Project1C6BFA3F-wQm2hXv2jqQv
• Commit ID: afdc41e
• Result: SUCCEEDED
• Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

[mergify]

Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).