Skip to content

Fix test-node.bash - #719

Merged
rachel-bousfield merged 2 commits into
masterfrom
fix-seq-addr
Jun 24, 2022
Merged

rachel-bousfield merged 2 commits into
masterfrom
fix-seq-addr

Conversation

@gzeoneth

Copy link
Copy Markdown
Member

I am still unable to get the sequencer to run, failing at this line

return nil, fmt.Errorf("sequencer sender whitelist entry \"%v\" is not a valid address", address)

with

sequencer_1         | panic: sequencer sender whitelist entry "[]" is not a valid address

The config dump is

{"conf":{"dump":false,"env-prefix":"","file":["/config/sequencer_config.json"],"s3":{"access-key":"","bucket":"","object-key":"","region":"","secret-key":""},"string":""},"dev-init":false,"graphql":{"corsdomain":[],"enable":false,"vhosts":["localhost"]},"http":{"addr":"0.0.0.0","api":["net","web3","eth"],"corsdomain":[],"port":8547,"rpcprefix":"","vhosts":["localhost"]},"import-file":"","l1":{"chain-id":1337,"connection-attempts":15,"rollup":{"bridge":"0x86a54b72265b3539bcd2d3d6c99f5c0eb1b9e519","deployed-at":71,"inbox":"0x2215477271fbd5e08b47a36d4c1450a2489586c6","rollup":"0x90368d768778afcfce900324fa3a9c9813de637a","sequencer-inbox":"0x2e0d0ab8475a7de5382e2092945d83c43e34c7c3","validator-utils":"0xd92896a11ec85c731cad98a454ac258ae183f594","validator-wallet-creator":"0x0bf5a64aec707f2cb54d82b010e415984b84dbcf"},"url":"ws://geth:8546","wallet":{"account":"","only-create-key":false,"password":"","pathname":"/home/user/l1keystore","private-key":""}},"l2":{"chain-id":412346,"dev-wallet":{"account":"","only-create-key":false,"password":"","pathname":"","private-key":"e887f7d17d07cc7b8004053fb8826f6657084e88904bb61590e498ca04704cf2"},"wallet":{"password":"","private-key":""}},"log-level":3,"log-type":"plaintext","metrics":false,"metrics-server":{"addr":"127.0.0.1","port":6070,"update-interval":"3s"},"no-init":false,"node":{"archive":true,"batch-poster":{"compression-level":6,"das-retention-period":"360h0m0s","enable":false,"error-delay":"10s","high-gas-delay":"14h0m0s","high-gas-threshold":150,"max-interval":"30s","max-size":100000,"poll-delay":"10s"},"block-validator":{"concurrent-runs-limit":0,"current-module-root":"current","enable":false,"output-path":"./target/output","pending-upgrade-module-root":"latest","store-preimages":false},"dangerous":{"no-l1-listener":false},"data-availability":{"enable":false,"key":{"key-dir":"","priv-key":""},"l1-node-url":"","local-cache":{"enable":false,"expiration":"1h0m0s"},"local-db-storage":{"data-dir":"","discard-after-timeout":false,"enable":false},"local-file-storage":{"data-dir":"","enable":false},"redis-cache":{"enable":false,"redis-expiration":"1h0m0s","redis-key-config":"","redis-url":""},"request-timeout":"5s","rest-aggregator":{"enable":false,"max-per-endpoint-stats":20,"online-url-list":"","simple-explore-exploit-strategy":{"exploit-iterations":1000,"explore-iterations":20},"strategy":"simple-explore-exploit","strategy-update-interval":"10s","sync-to-storage":{"eager":false,"eager-lower-bound-block":"0","eager-stops-when-caught-up":false,"ignore-write-errors":true,"retention-period":"2562047h47m16.854775807s"},"urls":[],"wait-before-try-next":"2s"},"rpc-aggregator":{"assumed-honest":0,"backends":"","dump-keyset":false,"enable":false},"s3-storage":{"access-key":"","bucket":"","discard-after-timeout":false,"enable":false,"object-prefix":"","region":"","secret-key":""},"sequencer-inbox-address":""},"delayed-sequencer":{"enable":true,"finalize-distance":12,"time-aggregate":"1m0s"},"feed":{"input":{"timeout":"20s","url":[]},"output":{"addr":"","client-timeout":"15s","enable":false,"io-timeout":"5s","max-send-queue":4096,"ping":"5s","port":"9642","queue":100,"workers":100}},"forwarding-target":"null","inbox-reader":{"check-delay":"20s","delay-blocks":"0","hard-reorg":false},"l1-reader":{"enable":true,"poll-interval":"15s","poll-only":false,"tx-timeout":"5m0s"},"rpc":{"bloom-bits-blocks":"16384","evm-timeout":"5s","feehistory-max-block-count":"1024","gas-cap":"50000000","tx-fee-cap":"1"},"seq-coordinator":{"allowed-msg-lag":"200","chosen-healthcheck-addr":"","dangerous":{"disable-signature-verification":false},"enable":true,"lockout-duration":"30s","lockout-spare":"1s","msg-per-poll":"2000","my-url":"","redis-url":"redis://redis:6379","retry-interval":"0.5s","seq-num-duration":"24h0m0s","signing-key":"0123456789abcdef0123456789abcdef0123456789abcdef0123456789abcdef","update-interval":"3s"},"sequencer":{"dangerous":{"no-coordinator":false},"enable":true,"max-acceptable-timestamp-delta":"1h0m0s","max-block-speed":"100ms","max-revert-gas-reject":"31000","sender-whitelist":"[]"},"validator":{"confirmation-blocks":12,"dangerous":{"without-block-validator":false},"disable-challenge":false,"enable":false,"posting-strategy":{"high-gas-delay-blocks":0,"high-gas-threshold":"0"},"staker-interval":"10s","strategy":"MakeNodes","target-machine-count":4},"wasm":{"root-path":""}},"persistent":{"chain":"local","global-config":".arbitrum"},"ws":{"addr":"0.0.0.0","api":["net","web3","eth"],"expose-all":false,"origins":[],"port":8548,"rpcprefix":""}}

which show a "sender-whitelist":"[]" but I can't find where is that string "[]" introduced.

@cla-bot cla-bot Bot added the cla-signed label Jun 24, 2022
@codecov

codecov Bot commented Jun 24, 2022 •

Copy link
Copy Markdown

Codecov Report

Merging #719 (f297dae) into master (21ce481) will decrease coverage by 4.48%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #719      +/-   ##
==========================================
- Coverage   52.26%   47.78%   -4.49%     
==========================================
  Files         224      212      -12     
  Lines       26692    22576    -4116     
  Branches      494      494              
==========================================
- Hits        13950    10787    -3163     
+ Misses      11301    10334     -967     
- Partials     1441     1455      +14     

@PlasmaPower
PlasmaPower marked this pull request as ready for review June 24, 2022 11:55

@rachel-bousfield rachel-bousfield left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@rachel-bousfield
rachel-bousfield merged commit 78d22bf into master Jun 24, 2022
@rachel-bousfield
rachel-bousfield deleted the fix-seq-addr branch June 24, 2022 13:29
tsahee pushed a commit that referenced this pull request Jul 29, 2025
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.

3 participants