feat(route53): added EvaluateTargetHealth to Route53 Alias targets (#9481) #30664

wladyslawczyzewski · 2024-06-25T20:12:03Z

Issue # (if applicable)

Part of a bigger piece of work for #9481
Another PR for this ticket #30739

Reason for this change

At this moment constructing the Route53 health checks is available only via L1 construct, this PR is one of the series to introduce the R53 health checks via higher-level constructs.

Description of changes

The underlying L1 construct of a few L2 constructs for Alias targets already has the EvaluateTargetHealth property, so by adding this prop to the L2 construct we get the quick win to introduce the health checks for Alias records.

Description of how you validated changes

unit tests
integration tests

Checklist

My code adheres to the CONTRIBUTING GUIDE and DESIGN GUIDELINES

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

wladyslawczyzewski · 2024-06-25T20:13:35Z

hey guys, this is my first contribution to this repo, I would be grateful for any feedback 🤝

default configuration of Alias target health check

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.

…perty of Alias record

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

badmintoncryer

Thank you for your contribution!

Could you also create integ tests for targets other than ALB and Global Accelerator?

packages/aws-cdk-lib/aws-route53/lib/alias-record-target.ts

packages/aws-cdk-lib/aws-route53-targets/lib/bucket-website-target.ts

packages/aws-cdk-lib/aws-route53-targets/lib/classic-load-balancer-target.ts

packages/aws-cdk-lib/aws-route53-targets/lib/elastic-beanstalk-environment-target.ts

packages/aws-cdk-lib/aws-route53-targets/lib/global-accelerator-target.ts

packages/aws-cdk-lib/aws-route53-targets/lib/load-balancer-target.ts

packages/aws-cdk-lib/aws-route53-targets/lib/elastic-beanstalk-environment-target.ts

…rget.ts Co-authored-by: Kazuho Cryer-Shinozuka <[email protected]>

…ncer-target.ts Co-authored-by: Kazuho Cryer-Shinozuka <[email protected]>

…-environment-target.ts Co-authored-by: Kazuho Cryer-Shinozuka <[email protected]>

…r-target.ts Co-authored-by: Kazuho Cryer-Shinozuka <[email protected]>

…get.ts Co-authored-by: Kazuho Cryer-Shinozuka <[email protected]>

…s_health

packages/aws-cdk-lib/aws-route53-targets/lib/props.ts

GavinZZ

One more nit and I'm happy to approve.

…s_health

Pull request has been modified.

mergify · 2024-11-25T18:42:04Z

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).

mergify · 2024-11-25T21:55:08Z

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).

Pull request has been modified.

mergify · 2024-11-26T19:57:53Z

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).

Pull request has been modified.

mergify · 2024-11-28T19:10:27Z

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).

codecov · 2024-11-28T20:31:36Z

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.42%. Comparing base (8ccdff4) to head (2c78477).

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #30664      +/-   ##
==========================================
- Coverage   78.45%   78.42%   -0.03%     
==========================================
  Files         106      106              
  Lines        7208     7208              
  Branches     1323     1323              
==========================================
- Hits         5655     5653       -2     
- Misses       1365     1368       +3     
+ Partials      188      187       -1

Flag	Coverage Δ
suite.unit	`78.42% <ø> (-0.03%)`	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components	Coverage Δ
packages/aws-cdk	`78.42% <ø> (-0.03%)`	⬇️

GavinZZ · 2024-11-29T18:33:06Z

@mergify update

mergify · 2024-11-29T18:33:12Z

update

✅ Branch has been successfully updated

aws-cdk-automation · 2024-11-29T19:04:52Z

AWS CodeBuild CI Report

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

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

mergify · 2024-11-29T19:05:20Z

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).

feat(route53): added EvaluateTargetHealth to Route53 Alias targets

e7ceac5

github-actions bot added the beginning-contributor [Pilot] contributed between 0-2 PRs to the CDK label Jun 25, 2024

aws-cdk-automation requested a review from a team June 25, 2024 20:12

github-actions bot added the p2 label Jun 25, 2024

feat(route53): revert some previous changes to not introducing the

5d6aa97

default configuration of Alias target health check

aws-cdk-automation previously requested changes Jun 25, 2024

View reviewed changes

wladyslawczyzewski added 2 commits June 26, 2024 01:10

feat(route53): update integration tests for EvalueateTargetHealth pro…

070eda9

…perty of Alias record

feat(route53): update README file

d004bc9

aws-cdk-automation added the pr/needs-community-review This PR needs a review from a Trusted Community Member or Core Team Member. label Jun 25, 2024

wladyslawczyzewski changed the title ~~feat(route53): added EvaluateTargetHealth to Route53 Alias targets~~ feat(route53): added EvaluateTargetHealth to Route53 Alias targets (#9481) Jun 28, 2024

This was referenced Jul 1, 2024

Monthly PR metrics report #30715

Closed

Weekly PR metrics report #30720

Closed

Merge branch 'main' into route53_evaluate_alias_health

3ea3441

wladyslawczyzewski mentioned this pull request Jul 2, 2024

feat(route53): added L2 construct for Route53's health checks #30739

Open

1 task

wladyslawczyzewski added 7 commits July 11, 2024 09:45

Merge branch 'main' into route53_evaluate_alias_health

83f2a95

Merge branch 'main' into route53_evaluate_alias_health

c93b2c9

Merge branch 'main' into route53_evaluate_alias_health

127acb3

Merge branch 'main' into route53_evaluate_alias_health

cab0113

Merge branch 'main' into route53_evaluate_alias_health

c524487

Merge branch 'main' into route53_evaluate_alias_health

a804541

Merge branch 'main' into route53_evaluate_alias_health

222d3da

lukepafford mentioned this pull request Sep 2, 2024

feat(route53): new Route53 CrossAccountRecordSet construct #31281

Closed

badmintoncryer suggested changes Oct 31, 2024

View reviewed changes

wladyslawczyzewski and others added 5 commits November 10, 2024 14:41

Update packages/aws-cdk-lib/aws-route53-targets/lib/bucket-website-ta…

dd2b02f

…rget.ts Co-authored-by: Kazuho Cryer-Shinozuka <[email protected]>

Update packages/aws-cdk-lib/aws-route53-targets/lib/classic-load-bala…

98c02af

…ncer-target.ts Co-authored-by: Kazuho Cryer-Shinozuka <[email protected]>

Update packages/aws-cdk-lib/aws-route53-targets/lib/elastic-beanstalk…

953964a

…-environment-target.ts Co-authored-by: Kazuho Cryer-Shinozuka <[email protected]>

Update packages/aws-cdk-lib/aws-route53-targets/lib/global-accelerato…

cf76e59

…r-target.ts Co-authored-by: Kazuho Cryer-Shinozuka <[email protected]>

Update packages/aws-cdk-lib/aws-route53-targets/lib/load-balancer-tar…

272294e

…get.ts Co-authored-by: Kazuho Cryer-Shinozuka <[email protected]>

wladyslawczyzewski added 3 commits November 22, 2024 09:57

fix CI build

0ed30d1

Merge remote-tracking branch 'origin/main' into route53_evaluate_alia…

1883fc1

…s_health

added docstring

963ba80

wladyslawczyzewski requested a review from GavinZZ November 22, 2024 11:06

GavinZZ reviewed Nov 22, 2024

View reviewed changes

packages/aws-cdk-lib/aws-route53-targets/lib/props.ts Outdated Show resolved Hide resolved

GavinZZ previously requested changes Nov 22, 2024

View reviewed changes

wladyslawczyzewski added 2 commits November 23, 2024 23:29

rename the file with shared interface

4ef99bb

Merge remote-tracking branch 'origin/main' into route53_evaluate_alia…

823e263

…s_health

fix import paths

d145d85

wladyslawczyzewski requested a review from GavinZZ November 23, 2024 23:28

GavinZZ previously approved these changes Nov 25, 2024

View reviewed changes

aws-cdk-automation removed the pr/needs-maintainer-review This PR needs a review from a Core Team Member label Nov 25, 2024

Merge branch 'main' into route53_evaluate_alias_health

a309400

GavinZZ previously approved these changes Nov 26, 2024

View reviewed changes

Merge branch 'main' into route53_evaluate_alias_health

ae19705

GavinZZ approved these changes Nov 28, 2024

View reviewed changes

Merge branch 'main' into route53_evaluate_alias_health

2081fc5

Merge branch 'main' into route53_evaluate_alias_health

2c78477

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(route53): added EvaluateTargetHealth to Route53 Alias targets (#9481) #30664

feat(route53): added EvaluateTargetHealth to Route53 Alias targets (#9481) #30664

wladyslawczyzewski commented Jun 25, 2024 •

edited

Loading

wladyslawczyzewski commented Jun 25, 2024

aws-cdk-automation left a comment

badmintoncryer left a comment

GavinZZ left a comment

mergify bot commented Nov 25, 2024

mergify bot commented Nov 25, 2024

mergify bot commented Nov 26, 2024

mergify bot commented Nov 28, 2024

codecov bot commented Nov 28, 2024 •

edited

Loading

GavinZZ commented Nov 29, 2024

mergify bot commented Nov 29, 2024

aws-cdk-automation commented Nov 29, 2024

mergify bot commented Nov 29, 2024

feat(route53): added EvaluateTargetHealth to Route53 Alias targets (#9481) #30664

Are you sure you want to change the base?

feat(route53): added EvaluateTargetHealth to Route53 Alias targets (#9481) #30664

Conversation

wladyslawczyzewski commented Jun 25, 2024 • edited Loading

Issue # (if applicable)

Reason for this change

Description of changes

Description of how you validated changes

Checklist

wladyslawczyzewski commented Jun 25, 2024

aws-cdk-automation left a comment

Choose a reason for hiding this comment

badmintoncryer left a comment

Choose a reason for hiding this comment

GavinZZ left a comment

Choose a reason for hiding this comment

mergify bot commented Nov 25, 2024

mergify bot commented Nov 25, 2024

mergify bot commented Nov 26, 2024

mergify bot commented Nov 28, 2024

codecov bot commented Nov 28, 2024 • edited Loading

Codecov Report

GavinZZ commented Nov 29, 2024

mergify bot commented Nov 29, 2024

✅ Branch has been successfully updated

aws-cdk-automation commented Nov 29, 2024

AWS CodeBuild CI Report

mergify bot commented Nov 29, 2024

wladyslawczyzewski commented Jun 25, 2024 •

edited

Loading

codecov bot commented Nov 28, 2024 •

edited

Loading