deployment: move network and consensus ports to hybrid environment overlays#14183
Conversation
ad8129b to
7b0b0a0
Compare
PR SummaryMedium Risk Overview Each overlay now sets This matches the paired devops work to expose the same ports in Kubernetes. Reviewed by Cursor Bugbot for commit e267b09. 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.
1c33c4c to
ea09641
Compare
7b0b0a0 to
6a1e8cc
Compare
dorimedini-starkware
left a comment
There was a problem hiding this comment.
@dorimedini-starkware reviewed 3 files and all commit messages, and made 2 comments.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on idan-starkware, matanl-starkware, and nimrod-starkware).
a discussion (no related file):
general question: is there a config that is common for all networks? because all these values are identical.. is this a planned feature? part of the jsonnet migration?
6a1e8cc to
e267b09
Compare
ea09641 to
277ebfe
Compare
nimrod-starkware
left a comment
There was a problem hiding this comment.
@nimrod-starkware made 2 comments.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on dorimedini-starkware, idan-starkware, and matanl-starkware).
a discussion (no related file):
Previously, dorimedini-starkware wrote…
general question: is there a config that is common for all networks? because all these values are identical.. is this a planned feature? part of the jsonnet migration?
Yes, see this.
It's in this stack...
dorimedini-starkware
left a comment
There was a problem hiding this comment.
@dorimedini-starkware resolved 2 discussions.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on idan-starkware and matanl-starkware).


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