Skip to content

Comments

chore(docdb): upgrade integ tests to engine 8.0.0 and add serverless region constraints#37059

Open
aemada-aws wants to merge 1 commit intomainfrom
fix/docdb-integ-tests
Open

chore(docdb): upgrade integ tests to engine 8.0.0 and add serverless region constraints#37059
aemada-aws wants to merge 1 commit intomainfrom
fix/docdb-integ-tests

Conversation

@aemada-aws
Copy link
Contributor

Issue # (if applicable)

N/A

Reason for this change

Four aws-docdb integration tests were failing due to region-incompatible engine versions:

  • integ.cluster / integ.cluster-copy-tags-to-snapshot / integ.cluster-removal-policy-snapshot: Used DocumentDB engine version 3.6.0, which is not available in all regions (e.g., eu-north-1 only supports 4.0.0+). Errors:
    • Resource handler returned message: "Internal Failure"
    • Cannot find version 3.6.0 for docdb
  • integ.cluster-serverless: DocumentDB Serverless (db.serverless) is not available in all regions (e.g., eu-north-1). Error:
    • The request includes a Serverless v2 ScalingConfiguration attribute, which isn't available for the engine version of your cluster (5.0.0)

Additionally, DocumentDB 3.6.0 standard support ends March 30, 2026, making the version upgrade timely.

Description of changes

Provisioned cluster tests (integ.cluster, integ.cluster-copy-tags-to-snapshot, integ.cluster-removal-policy-snapshot):

  • Upgraded engine version from 3.6.08.0.0 (latest, available in all DocDB regions)
  • Updated parameter group family from docdb3.6docdb8.0
  • No region constraints needed — 8.0.0 is universally available
  • Added IntegTest wrapper to integ.cluster.ts (previously used bare app.synth())

Serverless test (integ.cluster-serverless):

  • Added region constraints via IntegTest.regions (serverless only supports 5.0.0 and is not available in eu-north-1)

New shared constants file (docdb-integ-test-constraints.ts):

  • DOCDB_ENGINE_VERSION / DOCDB_PARAMETER_GROUP_FAMILY for provisioned tests
  • DOCDB_SERVERLESS_ENGINE_VERSION / DOCDB_SERVERLESS_SUPPORTED_REGIONS for serverless test
  • Includes verification commands in comments

Describe any new or updated permissions being added

N/A

Description of how you validated changes

All 4 tests deployed and destroyed successfully:

cd packages/@aws-cdk-testing/framework-integ && yarn integ \
  test/aws-docdb/test/integ.cluster.js \
  test/aws-docdb/test/integ.cluster-copy-tags-to-snapshot.js \
  test/aws-docdb/test/integ.cluster-removal-policy-snapshot.js \
  test/aws-docdb/test/integ.cluster-serverless.js \
  --disable-update-workflow \
  --update-on-failed \
  --force \
  --parallel-regions us-east-1 \
  --parallel-regions us-east-2 \
  --parallel-regions us-west-2 \
  --parallel-regions eu-west-1 \
  --verbose

Results: Tests: 4 passed, 4 total

⚠️ Destructive changes expected: Engine version upgrade from 3.6.0 to 8.0.0 requires cluster replacement. This is intentional.

Checklist


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

@github-actions github-actions bot added the p2 label Feb 23, 2026
@aws-cdk-automation aws-cdk-automation requested a review from a team February 23, 2026 15:02
@aemada-aws aemada-aws changed the title fix(docdb): upgrade integ tests to engine 8.0.0 and add serverless region constraints chore(docdb): upgrade integ tests to engine 8.0.0 and add serverless region constraints Feb 23, 2026
@mergify mergify bot added the contribution/core This is a PR that came from AWS. label Feb 23, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 23, 2026

⚠️ Experimental Feature: This security report is currently in experimental phase. Results may include false positives and the rules are being actively refined.
Please try merge from main to avoid findings unrelated to the PR.


TestsPassed ✅SkippedFailed
Security Guardian Results96 ran96 passed
TestResult
No test annotations available

@github-actions
Copy link
Contributor

github-actions bot commented Feb 23, 2026

⚠️ Experimental Feature: This security report is currently in experimental phase. Results may include false positives and the rules are being actively refined.
Please try merge from main to avoid findings unrelated to the PR.


TestsPassed ✅SkippedFailed
Security Guardian Results with resolved templates96 ran96 passed
TestResult
No test annotations available

@aemada-aws aemada-aws marked this pull request as ready for review February 23, 2026 23:40
@aemada-aws aemada-aws added the pr/needs-integration-tests-deployment Requires the PR to deploy the integration test snapshots. label Feb 23, 2026
@aws-cdk-automation aws-cdk-automation added the pr/needs-maintainer-review This PR needs a review from a Core Team Member label Feb 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contribution/core This is a PR that came from AWS. p2 pr/needs-integration-tests-deployment Requires the PR to deploy the integration test snapshots. pr/needs-maintainer-review This PR needs a review from a Core Team Member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants