-
Notifications
You must be signed in to change notification settings - Fork 732
Add an option to dump/load configuration sandbox in cardano-testnet
#6239
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
Conversation
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.
Yes this is looking good 👍 . I am more concerned with getting everything working. My comments about factoring out functions etc are not important now.
cardano-testnet/test/cardano-testnet-test/Cardano/Testnet/Test/DumpConfig.hs
Outdated
Show resolved
Hide resolved
@@ -164,46 +165,22 @@ cardanoTestnet :: () | |||
=> HasCallStack | |||
=> CardanoTestnetOptions -- ^ The options to use | |||
-> GenesisOptions | |||
-> UserProvidedData ShelleyGenesis |
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.
💪
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.
Unfortunately, I had to put those back: some tests use it to create an env with custom parameters. Maybe I should modify the config files on disk since that's now possible?
ea046e0
to
f86dc4b
Compare
f86dc4b
to
3f192d1
Compare
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.
LGTM! A couple of comments.
@@ -106,17 +106,17 @@ hprop_gov_no_confidence = integrationWorkspace "no-confidence" $ \tempAbsBasePat | |||
|
|||
let conwayGenesisWithCommittee = defaultConwayGenesis { L.cgCommittee = committee } | |||
|
|||
createTestnetEnv |
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.
💪
Add an option to dump/load configuration sandbox in `cardano-testnet`
This disables checking for hashes when loading individual genesis files, which in turn allows for on-the-fly configuration modifications, without having to recompute the hashes each time.
fb0c257
to
ba03c10
Compare
Description
This PR introduces a new
create-env
sub-command tocardano-testnet
, which creates a sandboxed environment with default parameters, which can then be modified by hand and passed to the maincardano
subcommand with the--node-env
option.This PR should be reviewed commit by commit.
Checklist
See Runnings tests for more details
CHANGELOG.md
for affected packagehlint
. See.github/workflows/check-hlint.yml
to get thehlint
versionstylish-haskell
. See.github/workflows/stylish-haskell.yml
to get thestylish-haskell
versionghc-9.6
andghc-9.12
Note on CI
If your PR is from a fork, the necessary CI jobs won't trigger automatically for security reasons.
You will need to get someone with write privileges. Please contact IOG node developers to do this
for you.