Skip to content

fix(bb-distributed-data): register DSQL runtime config#232

Open
ikenyal wants to merge 1 commit into
aws-devtools-labs:mainfrom
ikenyal:fix/distributed-data-runtime-config
Open

fix(bb-distributed-data): register DSQL runtime config#232
ikenyal wants to merge 1 commit into
aws-devtools-labs:mainfrom
ikenyal:fix/distributed-data-runtime-config

Conversation

@ikenyal

@ikenyal ikenyal commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

Problem

DistributedDatabase registers its DSQL endpoint and region directly on the shared Blocks Lambda. Resource mappings should use the shared config registry so they do not consume the Lambda environment-variable budget.

Issue #, if available: N/A

Changes

  • Register the DSQL endpoint and region through registerConfig().
  • Keep the migration custom-resource Lambda environment unchanged; it is separate from the customer-facing Blocks Lambda.
  • Update CDK synthesis coverage to verify the application handler receives config-registry wiring, has no direct endpoint or region variables, and the deployed config includes both values.
  • Add a patch changeset for @aws-blocks/bb-distributed-data.

Validation

  • npm run build --workspace @aws-blocks/bb-distributed-data
  • npm test --workspace @aws-blocks/bb-distributed-data (146 passing tests)
  • npm run lint:deps
  • changeset status --since=origin/main

Checklist

  • PR description included
  • Tests are changed or added
  • Relevant documentation is changed or added (not applicable: no user-facing API or workflow change)

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

@ikenyal
ikenyal requested a review from a team as a code owner July 18, 2026 06:52
@changeset-bot

changeset-bot Bot commented Jul 18, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 64d725e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@aws-blocks/bb-distributed-data Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant