Skip to content

Fix kafka-console-shared and add protos from teranode repo#84

Merged
oskarszoon merged 1 commit into
bsv-blockchain:mainfrom
oskarszoon:bugfix/kafka-console
Oct 30, 2025
Merged

Fix kafka-console-shared and add protos from teranode repo#84
oskarszoon merged 1 commit into
bsv-blockchain:mainfrom
oskarszoon:bugfix/kafka-console

Conversation

@oskarszoon

Copy link
Copy Markdown
Contributor

Fixes #60

@github-actions

github-actions Bot commented Oct 30, 2025

Copy link
Copy Markdown
Contributor

🤖 Claude Code Review

Status: Complete

Summary: No issues found. The changes properly fix the kafka-console-shared configuration error reported in issue #60.

What Changed:

  • Pinned Redpanda console image to stable version (v3.2.2)
  • Replaced inline config with externalized config file
  • Added protobuf file mappings for all Kafka topics
  • Removed deprecated schemaRegistry configuration

Verified:

  • All proto file paths exist in the repository
  • All proto type names match the actual message definitions
  • Config syntax is valid YAML
  • Volume mounts use read-only mode appropriately

The implementation follows the same approach as teranode-teratestnet PR #36.

@sonarqubecloud

Copy link
Copy Markdown

@oskarszoon oskarszoon merged commit 108cde3 into bsv-blockchain:main Oct 30, 2025
8 checks passed
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.

[BUG] kafka-console-shared from deploy/docker erroring out

2 participants