Skip to content
This repository was archived by the owner on Mar 23, 2026. It is now read-only.

DynamoDB: Implement DescribeContributorInsights#13220

Merged
viren-nadkarni merged 10 commits into
localstack:mainfrom
aleslash:feat-dynamodb-contributorinsights
Oct 7, 2025
Merged

DynamoDB: Implement DescribeContributorInsights#13220
viren-nadkarni merged 10 commits into
localstack:mainfrom
aleslash:feat-dynamodb-contributorinsights

Conversation

@aleslash

@aleslash aleslash commented Oct 5, 2025

Copy link
Copy Markdown
Contributor

Motivation

To get better experience when using localstack with ACK when using dynamodb controller.

Changes

It implements the DescribeContributorInsights function as described in issue #13165

Testing

Run localstack with ack, create any dynamodb resource by applying a manifest like

apiVersion: dynamodb.services.k8s.aws/v1alpha1
kind: Table
metadata:
  name: my-dynamodb-table
spec:
  attributeDefinitions:
    - attributeName: id
      attributeType: S
  keySchema:
    - attributeName: id
      keyType: HASH
  provisionedThroughput:
    readCapacityUnits: 5
    writeCapacityUnits: 5
  tableName: my-dynamodb-table

And verify its status by command:

kubectl get tables.dynamodb.services.k8s.aws

status:

NAME                CLASS   STATUS   SYNCED   AGE
my-dynamodb-table           ACTIVE   True     33m

Closes #13165

@aleslash aleslash changed the title Feat dynamodb contributorinsights DynamoDB: Implement DescribeContributorInsights Oct 5, 2025
@alexrashed alexrashed mentioned this pull request Oct 6, 2025
Closed
@alexrashed alexrashed added semver: minor Non-breaking changes which can be included in minor releases, but not in patch releases review: merge when ready Signals to the reviewer that a PR can be merged if accepted docs: skip Pull request does not require documentation changes notes: needed Pull request should be mentioned in the release notes labels Oct 7, 2025
@alexrashed alexrashed added this to the 4.10 milestone Oct 7, 2025
@alexrashed

alexrashed commented Oct 7, 2025

Copy link
Copy Markdown
Member

@viren-nadkarni: As discussed in #13166, this PR is basically continuing where you guys left off there. Would you mind taking this one over here / picking it up here with the review? Thanks!

viren-nadkarni
viren-nadkarni approved these changes Oct 7, 2025
View reviewed changes

@viren-nadkarni viren-nadkarni left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for contributing @aleslash !

@viren-nadkarni viren-nadkarni merged commit e0ec7f6 into localstack:main Oct 7, 2025
54 of 57 checks passed
@aleslash aleslash deleted the feat-dynamodb-contributorinsights branch October 7, 2025 23:26
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@giograno giograno Awaiting requested review from giograno

1 more reviewer

@viren-nadkarni viren-nadkarni viren-nadkarni approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@viren-nadkarni viren-nadkarni

Labels

docs: skip Pull request does not require documentation changes notes: needed Pull request should be mentioned in the release notes review: merge when ready Signals to the reviewer that a PR can be merged if accepted semver: minor Non-breaking changes which can be included in minor releases, but not in patch releases

Projects

None yet

Milestone

4.10

Development

Successfully merging this pull request may close these issues.

feature request: DynamoDB DescribeContributorInsights implementation

3 participants

@aleslash @alexrashed @viren-nadkarni