Skip to content

Conversation

@kumvprat
Copy link
Contributor

@kumvprat kumvprat commented Jun 30, 2025

Reason for this change

Adding new aws partition to cdk : eusc-de-east-1

Description of changes

Changes here allow cdk user to build infrastructure in the new eusc-de-east-1 region :

  • Updated the route53 mappings needed to support a new region
  • Updated the assets/entities that are supported in the new region (like DLC repository accounts)
  • Integration test snapshots update to support the new region changes

Description of how you validated changes

Running integration tests

Checklist


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

@aws-cdk-automation aws-cdk-automation requested a review from a team June 30, 2025 16:58
@github-actions github-actions bot added the p2 label Jun 30, 2025
@mergify mergify bot added the contribution/core This is a PR that came from AWS. label Jun 30, 2025
@kumvprat kumvprat changed the title new partition aws chore(region-info): eusc-de-east-1 Jun 30, 2025
Jorge Diaz and others added 18 commits July 1, 2025 11:19
…gion

Update integration test snapshots to include the new eusc-de-east-1 region
in the LatestNodeRuntimeMap mapping with nodejs22.x runtime support.

This change reflects AWS's addition of a new region and ensures the
ivs-recording-configuration integration test passes with the updated
region mappings.
…-east-1 region

Update integration test snapshots to include eusc-de-east-1 region mapping
in LatestNodeRuntimeMap for the following successfully updated tests:

- integ.eks-addon
- integ.eks-al2023-nodegroup
- integ.eks-cluster-imported
- integ.eks-cluster-private-endpoint
- integ.eks-helm-asset
- integ.eks-inference-nodegroup
- integ.eks-oidc-provider
- integ.eks-standard-access-entry
- integ.eks-subnet-updates
- integ.eks-windows-ng
- integ.fargate-cluster

All tests now include the new region mapping:
"eusc-de-east-1":{"value":"nodejs22.x"}

Remaining tests (integ.alb-controller, integ.helm-chart-logging) require
additional updates and will be addressed separately.
…gion

- Updated LatestNodeRuntimeMap to include eusc-de-east-1 region with nodejs22.x runtime
- Updated asset references in helm-chart-logging integration test snapshots
…t for new eusc-de-east-1 region

- Add nodeType: redshift.NodeType.RA3_LARGE to fix deprecated dc2.large node type
- Update integration test snapshots to include eusc-de-east-1 region in LatestNodeRuntimeMap
- Test now shows UNCHANGED status indicating successful snapshot update
@kumvprat kumvprat requested a review from a team as a code owner July 1, 2025 17:39
@alvazjor alvazjor force-pushed the new_partition_aws branch from 121d691 to c1b8d54 Compare July 3, 2025 12:54
@vishaalmehrishi
Copy link
Contributor

aws/aws-cdk-cli#685 is expected to fix the failing integ tests here

@aws-cdk-automation
Copy link
Collaborator

AWS CodeBuild CI Report

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

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

@mergify
Copy link
Contributor

mergify bot commented Jul 3, 2025

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 mergify bot merged commit c3559b5 into main Jul 3, 2025
18 of 19 checks passed
@mergify mergify bot deleted the new_partition_aws branch July 3, 2025 20:33
@github-actions
Copy link
Contributor

github-actions bot commented Jul 3, 2025

Comments on closed issues and PRs are hard for our team to see.
If you need help, please open a new issue that references this one.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 3, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

contribution/core This is a PR that came from AWS. p2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants