Skip to content

add initial version of cockroachdb extension#131

Open
HarshCasper wants to merge 3 commits intomainfrom
cockroachdb
Open

add initial version of cockroachdb extension#131
HarshCasper wants to merge 3 commits intomainfrom
cockroachdb

Conversation

@HarshCasper
Copy link
Member

@HarshCasper HarshCasper commented Mar 3, 2026

Summary

  • Add CockroachDB extension for LocalStack using Docker sidecar pattern
  • Run CockroachDB in start-single-node --insecure mode with in-memory store for fast, ephemeral local development
  • Route PostgreSQL wire protocol connections through the LocalStack gateway at cockroachdb.localhost.localstack.cloud:4566 via TCP protocol detection
  • Include integration tests covering basic connectivity, CRUD operations, and mixed TCP/HTTP traffic co-existence with AWS services
  • Add GitHub Actions workflow for CI with lint, build, and integration test steps
  • Provide Makefile automation for extension installation, testing, linting, and distribution

Copy link
Member

@whummer whummer left a comment

Choose a reason for hiding this comment

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

Looks great @HarshCasper 🚀 - just checked out the extension locally, and installation and make test are working nicely. 👌

Looks good to go from my side, just added a minor comment which would be great to address before merging.

Btw, just wondering - do we also want to add a simple sample app here (similar to what we did for the other recent extensions), or just get the functionality merged for now? (adding a sample could probably be tackled later 👍 )

"""
self.start_container()

if self.tcp_ports:
Copy link
Member

Choose a reason for hiding this comment

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

note: The custom logic in here should technically no longer be required, after the changes in #130 . Can you try removing this method, and trigger the CI tests again (which should pull in the latest version of localstack-extensions-utils).

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