Skip to content
This repository was archived by the owner on Oct 23, 2025. It is now read-only.

add GlobalEndpoint - #611

Open
akwan224 wants to merge 2 commits into
aws-cloudformation:masterfrom
akwan224:addGlobalEndpoint
Open

add GlobalEndpoint#611
akwan224 wants to merge 2 commits into
aws-cloudformation:masterfrom
akwan224:addGlobalEndpoint

Conversation

@akwan224

Copy link
Copy Markdown

Description of changes:
Following up with the launch of global cluster endpoints https://aws.amazon.com/about-aws/whats-new/2024/10/amazon-aurora-global-database-writer-endpoint, adding global cluster endpoints to rds-global-cluster

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@Dmytrok-amz

Dmytrok-amz commented May 28, 2025

Copy link
Copy Markdown

The build is failing with checkstyle issues:

Trim Trailing Whitespace.................................................Failed

- hook id: trailing-whitespace

- exit code: 1

- files were modified by this hook
...

...
Error: Process completed with exit code 1.

@Dmytrok-amz
Dmytrok-amz enabled auto-merge July 4, 2025 00:48

@Dmytrok-amz Dmytrok-amz left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM - pending build verification

@cwoolum

cwoolum commented Sep 3, 2025

Copy link
Copy Markdown

@Dmytrok-amz , can we get this merged in?

@kcwinner

kcwinner commented Sep 3, 2025

Copy link
Copy Markdown

auto-merge was automatically disabled September 26, 2025 17:36

Pull request was closed

@Dmytrok-amz Dmytrok-amz reopened this Sep 26, 2025
@dbayardAWS

Copy link
Copy Markdown

I think line 75 of aws-rds-globalcluster/aws-rds-globalcluster.json is supposed to say "readOnlyProperties" as the tag, but it says "readOnly".

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants