deployment: move network and consensus ports to hybrid environment overlays#14183
Conversation
ad8129b to
7b0b0a0
Compare
PR SummaryLow Risk Overview Each environment now sets Reviewed by Cursor Bugbot for commit 7b0b0a0. Bugbot is set up for automated code reviews on this repo. Configure here. |
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 7b0b0a0. Configure here.
| state_sync_config.static_config.network_config.port: 55010 | ||
| state_sync_config.static_config.p2p_sync_client_config.#is_none: true | ||
| state_sync_config.static_config.rpc_config.port: 8090 | ||
| state_sync_config.static_config.storage_reader_server_static_config.port: 55012 |
There was a problem hiding this comment.
Port 55012 conflict between proof_manager and state_sync storage reader
High Severity
The newly added state_sync_config.static_config.storage_reader_server_static_config.port: 55012 conflicts with the pre-existing components.proof_manager.port: 55012. Both create TCP listeners on port 55012 within the same core service pod — the proof_manager creates a gRPC remote server bound to 0.0.0.0:55012, and the storage reader server creates an HTTP server also on 0.0.0.0:55012. This will cause an "address already in use" error at startup in all three environments.
Additional Locations (2)
Reviewed by Cursor Bugbot for commit 7b0b0a0. Configure here.



Corresponding devops PR: https://github.com/starkware-industries/sequencer-devops/pull/706