deployment: add service ports to hybrid common service overlays#14209
Conversation
|
Warning This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
This stack of pull requests is managed by Graphite. Learn more about stacking. |
PR SummaryLow Risk Overview For core, nine named TCP ports are exposed (batcher, storage readers, class manager, consensus P2P, signature manager, and state-sync RPC/network/storage), matching the component and static config ports already set in the same overlay. For mempool, the service exposes the main mempool port and mempool-p2p-config (53200), consistent with existing Reviewed by Cursor Bugbot for commit e015994. Bugbot is set up for automated code reviews on this repo. Configure here. |
|
Reverted — include order conflict: common/common.yaml's service.ports=[monitoring-endpoint] is brought into the service include chain via common_node.yaml and overwrites the service-specific ports. The migration is not safe without addressing include ordering first. |

No description provided.