Skip to content

[AD] Fix race condition in AD 1-click template that caused stack creation failure#7348

Merged
gmarciani merged 1 commit intoaws:developfrom
gmarciani:wip/mgiacomo/3160/fix-ad-test-0422-1
Apr 22, 2026
Merged

[AD] Fix race condition in AD 1-click template that caused stack creation failure#7348
gmarciani merged 1 commit intoaws:developfrom
gmarciani:wip/mgiacomo/3160/fix-ad-test-0422-1

Conversation

@gmarciani
Copy link
Copy Markdown
Contributor

@gmarciani gmarciani commented Apr 22, 2026

Description of changes

Fix race condition in AD 1-click template that caused stack creation failure by making the Post custom resource wait for the AdAdminNode to complete the registration of users.

The Post custom resource executes operations on AD users, which are created by the AdAdminNode. So it requires the AdAdminNode to complete its user data. So far the custom resource depends on AdAdminNode, but this is not enough because CloudFormation considered the EC2 instance created as soon as it launched, which unblocks the custom resource too early. To make the custom resource wait for the node3 to complete its user data, the custom resource must waith for the wait condition signaled by the end of the user data.

Tests

SUCCESS test_ad_integration

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

…tion failure.

The fix consists making the Post custom resource wait for the AdAdminNode to complete the registration of users.
@gmarciani gmarciani requested review from a team as code owners April 22, 2026 21:00
@gmarciani gmarciani enabled auto-merge (rebase) April 22, 2026 21:03
@gmarciani gmarciani added skip-changelog-update Disables the check that enforces changelog updates in PRs 3.x labels Apr 22, 2026
@gmarciani gmarciani merged commit ee0ea6a into aws:develop Apr 22, 2026
47 of 49 checks passed
@gmarciani gmarciani deleted the wip/mgiacomo/3160/fix-ad-test-0422-1 branch April 22, 2026 22:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3.x skip-changelog-update Disables the check that enforces changelog updates in PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants