feat(storage): add AWS DynamoDB storage provider - #5458
edersonbrilhante wants to merge 6 commits into
Conversation
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.OpenSSF ScorecardScorecard details
Scanned Files
|
| RunnerStateActivation, | ||
| RunnerStateFilter, | ||
| RunnerStateRecord, | ||
| RunnerStateStore, |
There was a problem hiding this comment.
I think this split still needs a small prerequisite from the shared storage contracts. At this head, the core doesn't export the lifecycle types imported here, and it also lacks RunnerConfigRecord.accessScope and GitHubWebhookSecretStore. A targeted TypeScript check reported TS2305 for those missing exports and TS2339 for accessScope.
I understand that the provider wiring is intentionally outside this PR. Maybe we should bring over just the required type contracts, or stack this on a prerequisite PR that supplies them, so we can compile and test the provider independently. Is there another planned base for this split that I should take into account?
There was a problem hiding this comment.
I am bringing the code from the other PR. The other PR is too dirty to rebase and merge. Right is just raw code
Description
Add the AWS DynamoDB storage-provider Terraform module and Lambda implementation extracted from #5279. This draft is intentionally limited to:
modules/storage-providers/aws/dynamodblambdas/libs/storage-providers/aws/dynamodbThe source PR remains open and untouched; provider wiring, shared storage abstractions, documentation, and lockfile changes are not included.
Test Plan
Related Issues