Skip to content

isolate consensus RPC on an optional second IPC socket - #456

Closed
MitkoTschimev wants to merge 1 commit into
circlefin:mainfrom
1inch:isolated-consensus-ipc
Closed

MitkoTschimev wants to merge 1 commit into
circlefin:mainfrom
1inch:isolated-consensus-ipc

Conversation

@MitkoTschimev

Copy link
Copy Markdown

Summary

Public HTTP/WS and arc-consensus share one Reth EthApi, so they share one eth_call semaphore (--rpc.max-blocking-io-requests, default 256). When client eth_call fills that queue, consensus get_signing_validator_set waits, then ETH_DEFAULT_TIMEOUT (1s) kills the CL.

--consensus-ipcpath opens a second jsonrpsee IPC on its own EthApi (16 permits) and serves eth_* plus reth_subscribePersistedBlock there. Leave it unset and consensus keeps using --ipcpath; nothing else changes.

Test plan

  • arc-node-execution node --help shows --consensus-ipcpath and parsing tests pass
  • Unset flag: consensus still connects to --ipcpath
  • Set flag: consensus eth_call still succeeds while the public HTTP eth_call queue is saturated
  • Engine API on --auth-ipc.path is unchanged

Public HTTP/WS and arc-consensus share one EthApi eth_call semaphore.
Under load, get_signing_validator_set waits out ETH_DEFAULT_TIMEOUT and
the CL exits. --consensus-ipcpath serves consensus on a second EthApi
with 16 permits. Unset, consensus still uses --ipcpath.
@github-actions

Copy link
Copy Markdown
Contributor

Hi @MitkoTschimev,

Thank you for your interest in contributing to Arc Node.

This PR has been automatically closed because it does not reference a GitHub issue. All PRs must reference an existing issue using the format Closes: #XXX.

To contribute properly:

  1. Find an existing issue you'd like to work on, or open a new issue describing your proposed change
  2. Comment on the issue requesting assignment and wait for maintainer approval
  3. Only submit a PR after you have been assigned to the issue

Please see our CONTRIBUTING.md for more details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant