Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Verifier runs in Sequencer mode #1076

Closed
CAPtheorem opened this issue Jun 13, 2021 · 4 comments
Closed

Verifier runs in Sequencer mode #1076

CAPtheorem opened this issue Jun 13, 2021 · 4 comments
Labels
A-cannon Area: cannon C-bug Category: bugs

Comments

@CAPtheorem
Copy link
Contributor

Describe the bug
Using the standard command for spinning up the system with a Verifier, (docker-compose up --scale verifier=1 --build), a second gethl2 starts called "verifier_1", but based on its debug output and behavior, it thinks it's a sequencer, leading to confusion for all parties.

To Reproduce
Steps to reproduce the behavior:

  1. Run docker-compose up --scale verifier=1 --build per Readme.
  2. Inspect system log output:
verifier_1           | INFO [06-13|00:49:53.875] Regenerated local transaction journal    transactions=0 accounts=0
verifier_1           | INFO [06-13|00:49:53.875] Running in sequencer mode                sync-backend=l1
verifier_1           | INFO [06-13|00:49:53.875] Sanitizing timestamp refresh threshold to 3 minutes 

Expected behavior
Running docker-compose up --scale verifier=1 should yield a system with one sequencer and one verifier.

Fix
In the docker-compose.yaml, replace IS_VERIFIER: "true" with ROLLUP_VERIFIER_ENABLE: "true"

@snario
Copy link
Contributor

snario commented Jun 14, 2021

There is a PR fixing this currently under review: #981

@CAPtheorem
Copy link
Contributor Author

Great, thanks!

@smartcontracts
Copy link
Contributor

cc @annieke @tynes is this issue fixed?

@smartcontracts smartcontracts added C-bug Category: bugs A-cannon Area: cannon labels Nov 4, 2021
@annieke
Copy link
Contributor

annieke commented Nov 4, 2021

yup fixed with the linked PR! thanks for catching

@annieke annieke closed this as completed Nov 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-cannon Area: cannon C-bug Category: bugs
Projects
None yet
Development

No branches or pull requests

4 participants