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

Users SABT use “run-docker-validator-network" script that works #1898

Closed
clrblmt opened this issue Nov 26, 2019 · 1 comment
Closed

Users SABT use “run-docker-validator-network" script that works #1898

clrblmt opened this issue Nov 26, 2019 · 1 comment

Comments

@clrblmt
Copy link
Contributor

clrblmt commented Nov 26, 2019

Expected Behavior

Fix or remove it from the running-a-validator.md file.

Current Behavior

The “run-docker-validator-network.sh” does not work / bugs.

./run-docker-validator-network.sh: /usr/bin/bash: bad interpreter: No such file or directory

  • Shebang should be #!/usr/bin/env bash instead of #!/usr/bin/bash

Does not support manually running individual commands:

  • ./run-docker-validator-network.sh pull (OK)
  • ./run-docker-validator-network.sh accounts (OK)
  • ./run-docker-validator-network.sh deploy (FAILS with ./run-docker-validator-network.sh: line 153: CELO_PROXY_ADDRESS: unbound variable)

./run-docker-validator-network.sh pull,accounts,deploy,run-validator,status fails if screen command is not installed

jeanregisser added a commit to aaitor/celo-monorepo that referenced this issue Nov 26, 2019
nambrot pushed a commit that referenced this issue Nov 26, 2019
* adapting to comments and changing ports

* comments about database and other clarifications

* adding reference to baklava and alfajores faucet.
TOC updated

* full node

* adding the command run-fullnode to the script

* running fullnode

* fixing wrong references to ATTESTATION_PRIVATE_KEY

* setup of nodekey removed, fixes #1888

* Adding instructions to stop and remove containers (preserving data dir).
Fixes #1882

* putting together the commands creating the accounts and the assignation to env variables for clarity

* Adds instructions about how to execute long run commands using "screen". Fixes #1890

* adding docker run -d option

* consolidating the instructions for running a validator in one page. Fixes #1884

* Adding independent page about baklava network

* make run docker validator network also run on mac

* using /bin/bash

* adding command to check if docker is installed and running. Fixes #1892

* changing reference to baklava testnet markdown document

* separating the sections about running the proxy & the validator. TOC updated.
Fixes #1868

* fixing issue with bad enode port. Fixes #1879

* account proof-of-possession takes two arguments. Fixes #1873

* adding -d for running docker and removing reference to alfajores

* different changes

* fixing pos command

* removing reference to an invalid page. Fixes #1883

* changing warning text. Fixes #1867

* removing unnecessary use of static-nodes.json in validator. Fixes #1869

* Fix incorrect value echoed for CELO_VALIDATOR_GROUP_ADDRESS

* Fix shebang, #1898

* attestation service command

* adapting attestation service command

* removing echo

* removing alfajores references

* removing not necessary flags

* moving block to the bottom

* added clarification about stopped containers

* typo and rephrasing a couple of things
@diminator
Copy link
Contributor

fixed in #1934

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants