Skip to content

Expose GRPC port for peer service#33

Merged
galt-tr merged 1 commit into
bsv-blockchain:masterfrom
galt-tr:peerGRPC
Dec 5, 2025
Merged

Expose GRPC port for peer service#33
galt-tr merged 1 commit into
bsv-blockchain:masterfrom
galt-tr:peerGRPC

Conversation

@galt-tr
Copy link
Copy Markdown
Contributor

@galt-tr galt-tr commented Dec 5, 2025

This pull request adds support for a new gRPC port to the peer service and improves logging in the asset reconciler. The main changes include introducing a new constant for the gRPC port, updating the peer service specification to expose this port, and enhancing observability during asset reconciliation.

Peer service enhancements:

  • Added a new constant PeerGRPCPort (9904) to constants.go to define the gRPC port for peer services.
  • Updated the defaultPeerServiceSpec function in peer_service.go to expose the new gRPC port in the peer service, allowing gRPC traffic to be routed to the peer pods.

Observability improvements:

  • Added an info log in the Reconcile method of asset_controller.go to indicate when an asset is being reconciled, including the asset's name for easier debugging and monitoring.

@galt-tr galt-tr requested a review from mrz1836 as a code owner December 5, 2025 18:57
@github-actions github-actions Bot added fork-pr PR originated from a forked repository requires-manual-review PR or issue requires manual review by a maintainer or security team labels Dec 5, 2025
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Dec 5, 2025

👋 Thanks, @galt-tr!

This pull request comes from a fork. For security, our CI runs in a restricted mode.
A maintainer will triage this shortly and run any additional checks as needed.

  • 🏷️ Labeled: fork-pr, requires-manual-review
  • 👀 We'll review and follow up here if anything else is needed.

Thanks for contributing to bsv-blockchain/teranode-operator! 🚀

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Dec 5, 2025

@galt-tr galt-tr merged commit da53f27 into bsv-blockchain:master Dec 5, 2025
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fork-pr PR originated from a forked repository requires-manual-review PR or issue requires manual review by a maintainer or security team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants