Skip to content

Commit

Permalink
Remove setting sequencer-http-endpoint in yaml
Browse files Browse the repository at this point in the history
The default value in nitro should work.
  • Loading branch information
Tristan-Wilson committed Oct 21, 2024
1 parent 57e2925 commit 9214877
Show file tree
Hide file tree
Showing 3 changed files with 18 additions and 8 deletions.
2 changes: 1 addition & 1 deletion docker-compose.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -168,7 +168,7 @@ services:
- "l1keystore:/home/user/l1keystore"
- "config:/config"
- "tokenbridge-data:/tokenbridge-data"
command: --conf.file /config/sequencer_config.json --node.feed.output.enable --node.feed.output.port 9642 --http.api net,web3,eth,txpool,debug --node.seq-coordinator.my-url ws://sequencer:8548 --graphql.enable --graphql.vhosts * --graphql.corsdomain * --execution.sequencer.timeboost.sequencer-http-endpoint http://sequencer:8547 --auth.addr 0.0.0.0 --auth.api auctioneer,eth,validation --auth.jwtsecret=/config/jwt.hex --auth.origins *
command: --conf.file /config/sequencer_config.json --node.feed.output.enable --node.feed.output.port 9642 --http.api net,web3,eth,txpool,debug --node.seq-coordinator.my-url ws://sequencer:8548 --graphql.enable --graphql.vhosts * --graphql.corsdomain * --auth.addr 0.0.0.0 --auth.api auctioneer,eth,validation --auth.jwtsecret=/config/jwt.hex --auth.origins *
depends_on:
- geth

Expand Down
16 changes: 11 additions & 5 deletions scripts/config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -257,10 +257,7 @@ function writeConfigs(argv: any) {
},
"execution": {
"sequencer": {
"enable": false,
"timeboost": {
"enable": false
}
"enable": false
},
"forwarding-target": "null",
},
Expand Down Expand Up @@ -312,6 +309,11 @@ function writeConfigs(argv: any) {
sequencerConfig.node["seq-coordinator"].enable = true
sequencerConfig.execution["sequencer"].enable = true
sequencerConfig.node["delayed-sequencer"].enable = true
if (argv.timeboost) {
sequencerConfig.execution.sequencer.timeboost = {
"enable": true
};
}
fs.writeFileSync(path.join(consts.configpath, "sequencer_config.json"), JSON.stringify(sequencerConfig))

let posterConfig = JSON.parse(baseConfJSON)
Expand Down Expand Up @@ -594,7 +596,11 @@ export const writeConfigCommand = {
describe: "DAS committee member B BLS pub key",
default: ""
},

timeboost: {
boolean: true,
describe: "run sequencer in timeboost mode",
default: false
},
},
handler: (argv: any) => {
writeConfigs(argv)
Expand Down
8 changes: 6 additions & 2 deletions test-node.bash
Original file line number Diff line number Diff line change
Expand Up @@ -483,6 +483,7 @@ if $force_init; then
fi # $force_init

anytrustNodeConfigLine=""
timeboostNodeConfigLine=""

# Remaining init may require AnyTrust committee/mirrors to have been started
if $l2anytrust; then
Expand Down Expand Up @@ -512,12 +513,15 @@ if $l2anytrust; then
fi

if $force_init; then
if $l2timeboost; then
timeboostNodeConfigLine="--timeboost"
fi
if $simple; then
echo == Writing configs
docker compose run scripts write-config --simple $anytrustNodeConfigLine
docker compose run scripts write-config --simple $anytrustNodeConfigLine $timeboostNodeConfigLine
else
echo == Writing configs
docker compose run scripts write-config $anytrustNodeConfigLine
docker compose run scripts write-config $anytrustNodeConfigLine $timeboostNodeConfigLine

echo == Initializing redis
docker compose up --wait redis
Expand Down

0 comments on commit 9214877

Please sign in to comment.