optimism icon indicating copy to clipboard operation
optimism copied to clipboard

op-node: app should should retry establishing connection with l2 node untill it suceeds

Open vladopajic opened this issue 2 years ago • 1 comments

When op-node is started it will halt process if connection with l2 could not be established.

Log example:

t=2023-08-14T09:31:26+0000 lvl=eror msg="Unable to create the rollup node"      error="Post \"http://eth-node-1:8545\": context deadline exceeded"
t=2023-08-14T09:31:26+0000 lvl=crit msg="Application failed"                    message="Post \"http://eth-node-1:8545\": context deadline exceeded"

Instead of halting processes it might be better if node keeps running and tries reconnecting with l2 node until it succeeds, and printing log messages with current status.

vladopajic avatar Aug 14 '23 10:08 vladopajic

Def open to a PR implementing this

tynes avatar Aug 15 '23 15:08 tynes

Stale issue, closing this. The op-node by now does retry, and with backoff, and a maximum number of attempts.

protolambda avatar Jun 17 '24 19:06 protolambda

Correction: it does retry, but only for L1 connection, not L2.

protolambda avatar Jun 17 '24 19:06 protolambda