Skip to content

feat(opensearchservice): support DeploymentStrategyOptions on Domain#37987

Open
badmintoncryer wants to merge 5 commits into
aws:mainfrom
badmintoncryer:feat/opensearch-deployment-strategy
Open

feat(opensearchservice): support DeploymentStrategyOptions on Domain#37987
badmintoncryer wants to merge 5 commits into
aws:mainfrom
badmintoncryer:feat/opensearch-deployment-strategy

Conversation

@badmintoncryer
Copy link
Copy Markdown
Contributor

Issue # (if applicable)

Closes #37986.

Reason for this change

L1 CfnDomain.deploymentStrategyOptions already exists, but the L2 Domain construct does not expose it. Users have to use the defaultChild escape hatch.

Description of changes

  • Add DeploymentStrategy string enum (DEFAULT / CAPACITY_OPTIMIZED)
  • Add optional deploymentStrategy?: DeploymentStrategy on DomainProps, wired to CfnDomain.deploymentStrategyOptions
  • Pure additive change — when unset, no DeploymentStrategyOptions is emitted (byte-identical synthesis)

Describe any new or updated permissions being added

None.

Description of how you validated changes

  • Unit tests covering: prop absent, DEFAULT, CAPACITY_OPTIMIZED
  • New integration test integ.opensearch.deployment-strategy deployed successfully

Checklist


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

Copy link
Copy Markdown
Collaborator

@aws-cdk-automation aws-cdk-automation left a comment

Choose a reason for hiding this comment

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

(This review is outdated)

@badmintoncryer badmintoncryer changed the title feat(aws-opensearchservice): support DeploymentStrategyOptions on Domain feat(opensearchservice): support DeploymentStrategyOptions on Domain May 24, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 24, 2026

⚠️ Experimental Feature: This security report is currently in experimental phase. Results may include false positives and the rules are being actively refined.
This security report is NOT a review blocker. Please try merge from main to avoid findings unrelated to the PR.
To suppress a specific rule, see Suppressing Rules.


TestsPassed ✅SkippedFailed
Security Guardian Results48 ran48 passed
TestResult
No test annotations available

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 24, 2026

⚠️ Experimental Feature: This security report is currently in experimental phase. Results may include false positives and the rules are being actively refined.
This security report is NOT a review blocker. Please try merge from main to avoid findings unrelated to the PR.
To suppress a specific rule, see Suppressing Rules.


TestsPassed ✅SkippedFailed
Security Guardian Results with resolved templates48 ran48 passed
TestResult
No test annotations available

@aws-cdk-automation aws-cdk-automation dismissed their stale review May 24, 2026 14:32

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

README references DeploymentStrategy.CAPACITY_OPTIMIZED, but the rosetta
fixture did not import it, causing the Rosetta build step to fail with
"Cannot find name 'DeploymentStrategy'".
…ibe blue/green behavior

The DeploymentStrategy feature controls blue/green deployment behavior
(falling back to batch deployment when capacity is insufficient), not
Multi-AZ availability. Update the linked documentation page, the prose
description, and the README example engine version (now aligned with the
integ test) accordingly.
@aws-cdk-automation aws-cdk-automation added the pr/needs-community-review This PR needs a review from a Trusted Community Member or Core Team Member. label May 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

distinguished-contributor [Pilot] contributed 50+ PRs to the CDK p2 pr/needs-community-review This PR needs a review from a Trusted Community Member or Core Team Member.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

(aws-opensearchservice): support DeploymentStrategyOptions on Domain

2 participants