Skip to content
This repository was archived by the owner on Jan 16, 2026. It is now read-only.

fix(docker): Correctly set RPC port in node recipe#2035

Merged
refcell merged 1 commit intomainfrom
cl/fix-rpc-port
Jun 6, 2025
Merged

fix(docker): Correctly set RPC port in node recipe#2035
refcell merged 1 commit intomainfrom
cl/fix-rpc-port

Conversation

@clabby
Copy link
Contributor

@clabby clabby commented Jun 6, 2025

Overview

Corrects the RPC port in the kona-node recipe. Also removes the --p2p.redial 0 flag following @theochap's recent fix that resolved #1854.

@clabby clabby self-assigned this Jun 6, 2025
@clabby clabby added K-fix Kind: fix A-docker Area: Dockerfiles W-node Workstream: kona-node labels Jun 6, 2025
Copilot AI review requested due to automatic review settings June 6, 2025 15:51
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR corrects the RPC port configuration in the kona-node Docker recipe and removes the redundant --p2p.redial 0 flag.

  • Correctly adds a new --rpc.port flag with the updated port value.
  • Removes the unnecessary --p2p.redial 0 flag based on recent fixes.

@codecov
Copy link

codecov bot commented Jun 6, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.1%. Comparing base (777f4a9) to head (53df546).
Report is 2 commits behind head on main.

✅ All tests successful. No failed tests found.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@refcell refcell added this pull request to the merge queue Jun 6, 2025
Merged via the queue into main with commit 3d9387e Jun 6, 2025
23 checks passed
@refcell refcell deleted the cl/fix-rpc-port branch June 6, 2025 16:14
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Dec 10, 2025
## Overview

Corrects the RPC port in the `kona-node` recipe. Also removes the
`--p2p.redial 0` flag following @theochap's recent fix that resolved
op-rs/kona#1854.
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Jan 14, 2026
## Overview

Corrects the RPC port in the `kona-node` recipe. Also removes the
`--p2p.redial 0` flag following @theochap's recent fix that resolved
#1854.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

A-docker Area: Dockerfiles K-fix Kind: fix W-node Workstream: kona-node

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(node/p2p): remove infinite redial by default

3 participants