GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — DynamoDB: Implement DescribeContributorInsights by aleslash · Pull Request #13220 · localstack/localstack

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

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

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
aleslash added 10 commits September 18, 2025 15:23
@aleslash
dynamodb describecontributorinsights impl
6f0e2ab
@aleslash
dynamodb describecontributorinsights impl
e45f29d
@aleslash
test: add describe contributor insights test
451363c
@aleslash
refactor: adjust tests and feature contrib insights
d6354aa
@aleslash
Merge branch 'main' of https://github.com/aleslash/localstack into fe…
affb026 
…at-dynamodb-contributorinsights
@aleslash
fix: contributor insights signature
3937557
@aleslash
refactor: describe contributor insights
3d8878b
@aleslash
feat: describe contributor insights at v2
c6b59b7
@aleslash
chore: fix format style
8adcb22
@aleslash
Merge branch 'localstack:main' into feat-dynamodb-contributorinsights
85984e2
@aleslash
aleslash requested review from giograno and viren-nadkarni as code owners October 5, 2025 00:45
@aleslash aleslash changed the title Feat dynamodb contributorinsights DynamoDB: Implement DescribeContributorInsights Oct 5, 2025
@alexrashed alexrashed mentioned this pull request Oct 6, 2025
DynamoDB: Implement DescribeContributorInsights #13166
Closed
@alexrashed alexrashed assigned viren-nadkarni Oct 7, 2025
@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 reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
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

There was a problem hiding this 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 !

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@viren-nadkarni
viren-nadkarni merged commit e0ec7f6 into localstack:main Oct 7, 2025
@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

Uh oh!

There was an error while loading. Please reload this page.

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

Uh oh!

There was an error while loading. Please reload this page.

3 participants

@aleslash @alexrashed @viren-nadkarni
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.