Skip to content

fix(localmode): include DynamoDB for function logs#8

Open
swkeever wants to merge 1 commit into
mainfrom
skeever/update-localmode-dynamodb-assets
Open

fix(localmode): include DynamoDB for function logs#8
swkeever wants to merge 1 commit into
mainfrom
skeever/update-localmode-dynamodb-assets

Conversation

@swkeever

@swkeever swkeever commented Jun 12, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Update the embedded local-mode Docker Compose template to include DynamoDB Local.
  • Pass the function invocation and log-event table env vars expected by the updated hosting server.
  • Keep the CLI-owned compose asset free of server-owned first-party local secrets.

Architecture

  • volcano start now brings up DynamoDB Local alongside the existing local-mode services.
  • The hosting server receives DYNAMODB_ENDPOINT, FUNCTION_INVOCATIONS_TABLE_NAME, and LOG_EVENTS_HOT_PATH_TABLE_NAME so local mode uses the same DynamoDB-backed function logging hot path as the hosting PR.
  • The DynamoDB data volume is declared in the embedded compose template so local log/invocation tables survive container restarts.

Related PRs

Validation

  • GitHub Actions green on this PR, including check / check, check / localmode-e2e, and CodeQL.
  • docker compose -f internal/localmode/assets/docker-compose.template.yml config --quiet
  • go test ./internal/localmode
  • go test ./...

@swkeever swkeever marked this pull request as ready for review June 12, 2026 21:47
@swkeever swkeever requested a review from a team as a code owner June 12, 2026 21:47
Copilot AI review requested due to automatic review settings June 12, 2026 21:47

Copilot AI 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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@swkeever swkeever changed the title fix(localmode): include DynamoDB in embedded compose fix(localmode): include DynamoDB for function logs Jun 12, 2026
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.

2 participants