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

Refresh DendrETH documentation #210

Open
stefan-nikolov96 opened this issue Aug 14, 2023 · 0 comments
Open

Refresh DendrETH documentation #210

stefan-nikolov96 opened this issue Aug 14, 2023 · 0 comments

Comments

@stefan-nikolov96
Copy link
Contributor

stefan-nikolov96 commented Aug 14, 2023

Remove the following code snippets and text from -

In How does the relayer work?

INITIAL_SLOT=5355991
SLOTS_JUMP=64

...

We have set up a recurring job that repeats itself after a specified time interval (slotsjump) and starts from an initial slot. The job follows a specific network, currently supporting Pratter and Mainnet. To run this job, execute the following command in the beacon-light-client/solidity folder:
yarn hardhat run-update --initialslot $INITIAL_SLOT --slotsjump $SLOTS_JUMP --follownetwork pratter

@Dimo99, is this step completely redundant or should it be replaced by something else?

In Running DendrETH relay node

This passage -

You can also build a custom image yourself by executing the make dendreth-relay-node command within the development environment provided by this repository.

implies this step is optional while it is the default way to set up the docker environment in which we can deploy the relayer.

In Running the test suites

The WebAssembly tests of the Nim light client:
yarn test-emcc seems to be deprecated and should be replaced with yarn test

Additionally

Solidity tests fail if ETHEREUM_MAINNET_RPC is not set in .env

wasm nim-lc tests fail if

CUDOS_MNEMONIC
CUDOS_PUBLIC_KEY
CUDOS_CONTRACT_ADDRESS
CUDOS_RPC_ENDPOINT
COSMOS_LOCAL_MNEMONIC
COSMOS_LOCAL_RPC_ENDPOINT
MALAGA_ADDRESS
MALAGA_RPC_ENDPOINT
MALAGA_MNEMONIC

are missing from .env
These should be included in the README.md or better yet - provide the user with a preset .env

The docker image requires more than the default partition size - steps to increase the docker partition size should be included -

sudo systemctl stop docker.service docker.socket
sudo zfs umount -f zfs_root/nixos/var/lib/docker
sudo zfs destroy -fr zfs_root/nixos/var/lib/docker
sudo zfs create zfs_root/nixos/var/lib/docker -o canmount=on -o quota=100G
systemctl start docker.service docker.socket

sudo zfs set "zfs_root/nixos/var/lib/docker" -o canmount=on -o quota=300G
sudo zfs quota=300G "zfs_root/nixos/var/lib/docker"
  • The supervisor is central to the execution and common commands to interact with it should be provided.
    This is how you should shutdown the supervisor - sudo unlink /tmp/supervisor.sock

  • Add expected gas fee under deployed smart contracts table

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

No branches or pull requests

1 participant