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

DV alone with CLI - docker run command missing flags #342

Closed
1 task
KaloyanTanev opened this issue Mar 26, 2024 · 2 comments
Closed
1 task

DV alone with CLI - docker run command missing flags #342

KaloyanTanev opened this issue Mar 26, 2024 · 2 comments
Assignees
Labels
documentation Improvements or additions to documentation next-sprint-candidate protocol streamline

Comments

@KaloyanTanev
Copy link
Contributor

🎯 Problem to be solved

While trying to launch a cluster alone following the CLI in the docs, it didn't seem as streamlined as I hoped for.

In step 1.2 there is code snippet that I don't quite get:

WITHDRAWAL_ADDR=[ENTER YOUR WITHDRAWAL ADDRESS HERE]
FEE_RECIPIENT_ADDR=[ENTER YOUR FEE RECIPIENT ADDRESS HERE]
NETWORK="holesky"

Those values are not exported, neither do they exist in the .env.

Furthermore in step 1.4 the command:

docker run --rm -v "$(pwd):/opt/charon" --env-file .env obolnetwork/charon:v0.19.0 create cluster

returns:

20:43:32.172 ERRO cmd        Fatal error: missing --nodes flag       
	cmd/createcluster.go:310 .validateCreateConfig
	cmd/createcluster.go:145 .runCreateCluster
	cmd/createcluster.go:91 .func1
	cmd/cmd.go:80 .func1
	main.go:19 .main

while I do understand (after diving into the charon code) that there should be either --nodes flag or --definition-file, from where they are fetched, I believe it should be a bit more clear for the user what to do, with the example code snippets running out of the box.

🛠️ Proposed solution

  1. Remove
WITHDRAWAL_ADDR=[ENTER YOUR WITHDRAWAL ADDRESS HERE]
FEE_RECIPIENT_ADDR=[ENTER YOUR FEE RECIPIENT ADDRESS HERE]
NETWORK="holesky"

if it is not required.

  1. Add the necessary flags to the docker run --rm -v "$(pwd):/opt/charon" --env-file .env obolnetwork/charon:v0.19.0 create cluster command so the user can see what is the bare minimum config required to run. Potentially again with similar [ENTER YOUR WITHDRAWAL ADDRESS HERE] values where necessary.

Or

Add example definition file that the user can use and pass to the docker run --rm -v "$(pwd):/opt/charon" --env-file .env obolnetwork/charon:v0.19.0 create cluster command. Potentially again with similar [ENTER YOUR WITHDRAWAL ADDRESS HERE] values where necessary.

🧪 Tests

Be able to seamlessly follow the instructions 1:1 for running cluster alone via the CLI.

  • Netlify deployment build tests passing
@slugmann321
Copy link
Contributor

This is a bit confusing... it seems that you started on the "launchpad" flow of the docs, but then switched to the CLI flow.

The "git clone" part is missing from the launchpad tab. This is being fixed with PR #343. Other than that, the launchpad flow works fine.

For the CLI flow, it is true that we should be more clear that WITHDRAWAL_ADDR and FEE_RECIPIENT_ADDR are not found in the .env file but rather they are flags that could be added to the "docker run" command. I agree that we should present the docker run command with these flags.

Will implement in PR #343

@boulder225 boulder225 added documentation Improvements or additions to documentation protocol labels Mar 27, 2024
@slugmann321
Copy link
Contributor

Ok I fixed the CLI flow for "create a cluster alone" so that it's clear. Added the mandatory flags so that the command can be run easily.

See #343

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation next-sprint-candidate protocol streamline
Projects
None yet
Development

No branches or pull requests

3 participants