Skip to content

DynamoDB: Implement DescribeContributorInsights #13220

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
viren-nadkarni merged 10 commits into from
Oct 7, 2025
Merged

DynamoDB: Implement DescribeContributorInsights #13220

viren-nadkarni merged 10 commits into from
Oct 7, 2025
+77 −0

Conversation

@aleslash
Copy link
Contributor

@aleslash aleslash commented Oct 5, 2025

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
Copy link
Member

alexrashed commented Oct 7, 2025

@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
Copy link
Member

@viren-nadkarni viren-nadkarni left a comment

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 join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

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

@giograno giograno Awaiting requested review from giograno giograno is a code owner

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