-
Notifications
You must be signed in to change notification settings - Fork 103
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
test: add regen-sandbox docker setup #1550
Conversation
i'm seeing this output from the command below, i'm guessing i'm probably doing something wrong:
|
@wgwz running docker with the
This error happened to me because I set the env variable |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is great!!
One thing I'd like to have is an easy way to run this without persisting the data. I'd like to be able to re-run scripts without rm -r .regen
to get fresh state. Right now if you run the data
script twice you get an error defining the resolver a second time:
INFO: Defining resolver http://resolver.mydataservice.com
ERROR: (code 24) failed to execute message; message index: 0: a resolver with the same URL and manager already exists: unique key violation
The solution is almost there.. if you don't provide a /regen
volume mount then the chain data isn't persisted. But without a /regen
volume mount you can't run scripts nor share the keyring-backend
directory. Maybe we could make a separate volume mount for config, data and scripts? Although I don't want to make the docker run
command too complex
Or maybe a flag could be passed that is checked at the beginning of |
@clevinson are you wanting someone to pick up the changes/improvements on this? |
Co-authored-by: Kyle Lawlor-Bagcal <[email protected]>
Co-authored-by: Kyle Lawlor-Bagcal <[email protected]>
Co-authored-by: Ryan Christoffersen <[email protected]>
Co-authored-by: Ryan Christoffersen <[email protected]>
…ev/add-docker-sandbox
👍 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested the following:
- successfully spun up with setting mnemonic
- successfully spun up without setting mnemonic
- successfully spun up with existing home directory
- successfully spun up overwriting home directory
- successfully spun up with specific setup scripts
All minor suggestions/questions but before approving:
bridge
is not listed as a setup option in the readme- running
bridge
andecocredit
together produces inaccurate logs
Co-authored-by: Ryan Christoffersen <[email protected]>
@ryanchristo addressed your comments and ready for another review. Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The scripts have been removed from the pull request:
INFO: Initializing state from './setup/ecocredit.sh'
/usr/bin/bootstrap.sh: line 124: ./setup/ecocredit.sh: No such file or directory
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nvm on the last comment, just need to update the example command.
Co-authored-by: Ryan Christoffersen <[email protected]>
Description
This container (and bootstrapping scripts) is intended to aid with end-to-end testing of client libraries, applications, manual local network tests, and other scenarios where you want to be able to quickly bootstrap a single node regen network.
We could potentially be publishing versioned images of this container as part of our release cycle. And projects like Regen-web and regen-js could pull from those versioned containers for their integration tests.
The hope is to have an easily buildable docker setup for regen-ledger that:
--keyring-backend test
keyring from a single mnemonic provided through an env variable.regen
home directory persisting as a mounted volume by the host of the containerAuthor Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...
!
to the type prefix if API or client breaking changeCHANGELOG.md
Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...
!
in the type prefix if API or client breaking change