Skip to content

Commit 677818c

Browse files
committed
remove unecessary config from docker-compose
Signed-off-by: Igor Braga <5835477+bragaigor@users.noreply.github.com>
1 parent bbeb358 commit 677818c

3 files changed

Lines changed: 9 additions & 16 deletions

File tree

README.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -62,16 +62,16 @@ Adds a **regular follower node** (`regular-follower-node`) that runs consensus a
6262

6363
### `--run-consensus-and-execution-in-different-processes`
6464

65-
Adds a **split-process follower** where consensus and execution run as separate containers communicating over authenticated RPC.
65+
Adds a **split-process follower** where consensus and execution run as separate containers communicating over RPC.
6666

6767
```bash
6868
./test-node.bash --init --dev --run-consensus-and-execution-in-different-processes
6969
```
7070

7171
| Service | Role | Ports |
7272
|---------|------|-------|
73-
| `consensus-follower-node` | Consensus (inbox, feed, validation) | HTTP `8547` -> `7147`, WS `8552` -> `8552` |
74-
| `execution-follower-node` | Execution (EVM, RPC, GraphQL) | HTTP `8547` -> `7247`, WS `9682` -> `9682` |
73+
| `consensus-only-follower-node` | Consensus (inbox, feed, validation) | HTTP `8547` -> `7147`, WS `8552` -> `8552` |
74+
| `execution-only-follower-node` | Execution (EVM, RPC, GraphQL) | HTTP `8547` -> `7247`, WS `9682` -> `9682` |
7575

7676
Both flags can be combined to run all three follower configurations simultaneously.
7777

docker-compose.yaml

Lines changed: 4 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -154,7 +154,7 @@ services:
154154
- "consensus:/consensus"
155155
- "config:/config"
156156

157-
execution-follower-node:
157+
execution-only-follower-node:
158158
pid: host # allow debugging
159159
image: nitro-node-dev-testnode
160160
entrypoint: /usr/local/bin/nitro
@@ -168,9 +168,6 @@ services:
168168
- "tokenbridge-data:/tokenbridge-data"
169169
command:
170170
- --conf.file=/config/base_node_config.json
171-
- --node.feed.output.enable
172-
- --http.api=net,web3,eth,txpool,debug,timeboost,auctioneer
173-
- --node.seq-coordinator.my-url=http://sequencer:8547
174171
- --graphql.enable
175172
- --graphql.vhosts=*
176173
- --graphql.corsdomain=*
@@ -180,15 +177,15 @@ services:
180177
- --execution.rpc-server.enable
181178
- --execution.rpc-server.public
182179
- --execution.rpc-server.authenticated=false
183-
- --execution.consensus-rpc-client.url=ws://consensus-follower-node:8552
180+
- --execution.consensus-rpc-client.url=ws://consensus-only-follower-node:8552
184181
- --execution.consensus-rpc-client.jwtsecret=/config/jwt.hex
185182
- --execution.consensus-rpc-client.retries=3
186183
- --execution.consensus-rpc-client.connection-wait=15s
187184
- --execution.consensus-rpc-client.retry-delay=1s
188185
depends_on:
189186
- geth
190187

191-
consensus-follower-node:
188+
consensus-only-follower-node:
192189
pid: host # allow debugging
193190
image: nitro-node-dev-testnode
194191
entrypoint: /usr/local/bin/nitro
@@ -199,10 +196,8 @@ services:
199196
- "consensus-follower-data:/home/user/.arbitrum/local/nitro"
200197
- "l1keystore:/home/user/l1keystore"
201198
- "config:/config"
202-
- "tokenbridge-data:/tokenbridge-data"
203199
command:
204200
- --conf.file=/config/base_node_config.json
205-
- --node.seq-coordinator.my-url=http://sequencer:8547
206201
- --http.api=net,web3,eth,txpool,debug,timeboost,auctioneer
207202
- --graphql.enable
208203
- --graphql.vhosts=*
@@ -213,7 +208,7 @@ services:
213208
- --node.rpc-server.enable
214209
- --node.rpc-server.public
215210
- --node.rpc-server.authenticated=false
216-
- --node.execution-rpc-client.url=ws://execution-follower-node:9682
211+
- --node.execution-rpc-client.url=ws://execution-only-follower-node:9682
217212
- --node.execution-rpc-client.jwtsecret=/config/jwt.hex
218213
- --node.execution-rpc-client.retries=3
219214
- --node.execution-rpc-client.connection-wait=15s
@@ -232,10 +227,8 @@ services:
232227
- "regular-follower-data:/home/user/.arbitrum/local/nitro"
233228
- "l1keystore:/home/user/l1keystore"
234229
- "config:/config"
235-
- "tokenbridge-data:/tokenbridge-data"
236230
command:
237231
- --conf.file=/config/base_node_config.json
238-
- --node.seq-coordinator.my-url=http://sequencer:8547
239232
- --http.api=net,web3,eth,txpool,debug,timeboost,auctioneer
240233
- --graphql.enable
241234
- --graphql.vhosts=*

test-node.bash

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -354,7 +354,7 @@ while [[ $# -gt 0 ]]; do
354354
echo --build-utils rebuild scripts, rollupcreator, token bridge docker images
355355
echo --no-build-utils don\'t rebuild scripts, rollupcreator, token bridge docker images
356356
echo --follower-node run a follower node
357-
echo --run-consensus-and-execution-in-different-processes run consensus and execution nodes in different processes communicating over RPC
357+
echo --run-consensus-and-execution-in-different-processes run a Nitro node with consensus and execution in different processes communicating over RPC
358358
echo --force-build-utils force rebuilding utils, useful if NITRO_CONTRACTS_BRANCH or TOKEN_BRIDGE_BRANCH changes
359359
echo
360360
echo script runs inside a separate docker. For SCRIPT-ARGS, run $0 script --help
@@ -366,7 +366,7 @@ NODES="sequencer"
366366
INITIAL_SEQ_NODES="sequencer"
367367

368368
if $run_consensus_and_execution_in_different_processes; then
369-
NODES="$NODES consensus-follower-node execution-follower-node"
369+
NODES="$NODES consensus-only-follower-node execution-only-follower-node"
370370
fi
371371

372372
if $follower_node; then

0 commit comments

Comments
 (0)