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

update for 0.6.4-cdk #45

Merged
merged 3 commits into from
Mar 31, 2024
Merged

update for 0.6.4-cdk #45

merged 3 commits into from
Mar 31, 2024

Conversation

lesterli
Copy link
Member

Summary

  • update images with cdk-validium-node:0.6.4-cdk and cdk-data-availability:0.0.7
  • fix bridge issues with the right config settings
    • config/dac/config.toml
    • config/node/config.toml

@lesterli lesterli requested a review from bap2pecs March 29, 2024 14:47
@lesterli lesterli requested a review from bap2pecs March 31, 2024 15:48
@bap2pecs bap2pecs merged commit 5bfff97 into v0.0.3 Mar 31, 2024
@@ -411,6 +412,7 @@ services:
zkevm-mock-l1-network:
container_name: zkevm-mock-l1-network
image: snapchain/geth-cdk-validium-contracts:v0.0.3

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

Hi Stefan, thanks. Which repo branch was the Docker image you mentioned built from?
The image we used was built from this repo branch in which we added some deployment scripts.

Choose a reason for hiding this comment

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

Hi, it is either the one you have used or this one: https://github.com/0xPolygonHermez/zkevm-contracts/tree/feature/verifier-v5.0.0-rc.4-fork.8 (I cannot remember anymore to be honest 🙁 ).

Copy link
Member Author

Choose a reason for hiding this comment

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

got it, thanks.

lesterli added a commit that referenced this pull request Apr 4, 2024
* support persistent db volumes

* fix `make ps-exited` command

* add command `make stop-clean`

* fix: add volume for mock l1 geth data

* fix visualizer proxy CORS error

without it, clicking on "View UML Diagram" will throw 500 error and you will see CORS error in console logs

* support gasless tx setup

* add make target: set-env

* nit

* merge explorer backend env for l1 and l2

* add prometheus metrics

* support wallet connect

* remove unnecessary envs

these envs override values in the node config.toml file. since they are the same, there is no need to override

* fix stats-l2 pointing to the wrong db

* rename

* add set-keystore.js

* add set-configs.js

* modify .keystore files passwords in set-configs.js

* nit

* modify L1 RPCs in set-configs.js

* modify Etherscan API key in set-configs.js

* clean up the compose file

- ZKEVM_NODE_MTCLIENT_URI and ZKEVM_NODE_EXECUTOR_URI are already defined in config.toml
- same for ZKEVM_NODE_STATE_DB_HOST
- ZKEVM_NODE_SEQUENCER_SENDER_ADDRESS doesn't exist. it should be ZKEVM_NODE_SEQUENCE_SENDER_SENDER_ADDRESS. by default cfg.SequenceSender.PrivateKey is used
- ZKEVM_NODE_AGGREGATOR_SENDER_ADDRESS was incorrect set. it's not the address in aggregator.keystore

* add set-configs.js

* fix bridge-ui envs

* sepolia make commands

* add make command run-sepolia-restart

* Update docker compose file to use bridge ui test image

* Revert the latest commit

* update images and conf

* update for 0.6.4-cdk (#45)

* update image 0.6.4-cdk.2

---------

Co-authored-by: XiangEnze <alwyn329@gmail.com>
Co-authored-by: lesterli <fulfilself@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants