Skip to content

deployment: add recorder_url to hybrid environment overlays#14180

Merged
nimrod-starkware merged 1 commit into
mainfrom
05-25-deployment_add_recorder_url_to_hybrid_environment_overlays
May 26, 2026
Merged

deployment: add recorder_url to hybrid environment overlays#14180
nimrod-starkware merged 1 commit into
mainfrom
05-25-deployment_add_recorder_url_to_hybrid_environment_overlays

Conversation

@nimrod-starkware
Copy link
Copy Markdown
Contributor

@nimrod-starkware nimrod-starkware commented May 25, 2026

Add recorder_url to config.sequencerConfig in each production hybrid environment overlay:

  • mainnet/common.yaml: http://starknet-mainnet.cende-recorder-proxy.starknet.io/
  • sepolia-alpha/common.yaml: http://starknet-sepolia-alpha.cende-recorder-proxy.starknet.io/
  • sepolia-integration/common.yaml: http://starknet-sepolia-integration.cende-recorder-proxy.starknet.io/

Corresponding devops PR (removes the same key from the devops overlays): starkware-industries/sequencer-devops#702

@reviewable-StarkWare
Copy link
Copy Markdown

This change is Reviewable

@nimrod-starkware nimrod-starkware force-pushed the 05-25-deployment_add_recorder_url_to_hybrid_environment_overlays branch from 0e34ea9 to 1bc33cd Compare May 25, 2026 07:44
@nimrod-starkware nimrod-starkware force-pushed the 05-25-deployment_add_starknet_url_to_hybrid_environment_overlays branch from 3b60e9c to bfe2921 Compare May 25, 2026 07:44
@nimrod-starkware nimrod-starkware marked this pull request as ready for review May 25, 2026 07:53
@cursor
Copy link
Copy Markdown

cursor Bot commented May 25, 2026

PR Summary

Low Risk
Deployment-only config additions with environment-specific URLs; no application code or auth changes.

Overview
Adds recorder_url under config.sequencerConfig in the three hybrid production overlays (mainnet, sepolia-alpha, sepolia-integration), each pointing at the matching cende-recorder-proxy host on starknet.io.

This wires hybrid sequencers to the Cende recorder service per network; it aligns with moving the same setting out of devops overlays (see linked devops PR).

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

Copy link
Copy Markdown
Collaborator

@dorimedini-starkware dorimedini-starkware left a comment

Choose a reason for hiding this comment

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

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

@nimrod-starkware nimrod-starkware changed the base branch from 05-25-deployment_add_starknet_url_to_hybrid_environment_overlays to main May 26, 2026 07:49
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@nimrod-starkware nimrod-starkware force-pushed the 05-25-deployment_add_recorder_url_to_hybrid_environment_overlays branch from 1bc33cd to 8be1563 Compare May 26, 2026 07:52
@graphite-app
Copy link
Copy Markdown

graphite-app Bot commented May 26, 2026

Merge activity

  • May 26, 7:52 AM UTC: Graphite rebased this pull request, because this pull request is set to merge when ready.

@nimrod-starkware nimrod-starkware added this pull request to the merge queue May 26, 2026
Merged via the queue into main with commit ad0c176 May 26, 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