Skip to content

deployment: extract gateway common config#14246

Merged
nimrod-starkware merged 1 commit into
main-v0.14.3from
05-28-deployment_extract_gateway_common_config
Jun 1, 2026
Merged

deployment: extract gateway common config#14246
nimrod-starkware merged 1 commit into
main-v0.14.3from
05-28-deployment_extract_gateway_common_config

Conversation

@nimrod-starkware

Copy link
Copy Markdown
Contributor

No description provided.

@reviewable-StarkWare

Copy link
Copy Markdown

This change is Reviewable

@nimrod-starkware nimrod-starkware self-assigned this May 28, 2026
@nimrod-starkware nimrod-starkware marked this pull request as ready for review May 28, 2026 13:26
@cursor

cursor Bot commented May 28, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Pure config layout refactor with no per-environment overrides removed; risk is only if the include mechanism resolves paths differently than inlined YAML.

Overview
Moves duplicated hybrid gateway service deployment settings into a single shared overlay at configs/overlays/hybrid/common/services/gateway.yaml.

mainnet, sepolia-alpha, and sepolia-integration gateway.yaml files now only include that common file, matching how other hybrid services (e.g. committer, l1) are structured. Intended behavior is unchanged; this is consolidation only.

Reviewed by Cursor Bugbot for commit aa0ff92. Bugbot is set up for automated code reviews on this repo. Configure here.

@dorimedini-starkware dorimedini-starkware left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dorimedini-starkware reviewed 4 files and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on idan-starkware and matanl-starkware).

@nimrod-starkware nimrod-starkware force-pushed the 05-28-deployment_extract_gateway_common_config branch from 6d6d330 to 55d1db0 Compare May 31, 2026 14:15
@nimrod-starkware nimrod-starkware force-pushed the 05-28-deployment_extract_l1_common_config branch from 02b6bbc to 3882ce4 Compare May 31, 2026 14:15
@graphite-app graphite-app Bot changed the base branch from 05-28-deployment_extract_l1_common_config to graphite-base/14246 June 1, 2026 07:56
@nimrod-starkware nimrod-starkware force-pushed the 05-28-deployment_extract_gateway_common_config branch from 55d1db0 to 7186fcc Compare June 1, 2026 08:15
@nimrod-starkware nimrod-starkware changed the base branch from graphite-base/14246 to 05-28-deployment_extract_l1_common_config June 1, 2026 08:15
@nimrod-starkware nimrod-starkware changed the base branch from 05-28-deployment_extract_l1_common_config to main-v0.14.3 June 1, 2026 08:20
@nimrod-starkware nimrod-starkware force-pushed the 05-28-deployment_extract_gateway_common_config branch from 7186fcc to aa0ff92 Compare June 1, 2026 08:30
@nimrod-starkware nimrod-starkware added this pull request to the merge queue Jun 1, 2026
Merged via the queue into main-v0.14.3 with commit 92451c2 Jun 1, 2026
27 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.

3 participants