Conversation
|
|
||
| fund_geth_account() { | ||
| local foundry_key="0xac0974bec39a17e36ba4a6b4d238ff944bacb478cbed5efcae784d7bf4f2ff80" | ||
| local geth_account="0x821b55d8abe79bc98f05eb675fdc50dfe796b7ab" |
There was a problem hiding this comment.
There is variables defined for the address you should not hard code it.
| command: | ||
| - start | ||
| - reader-node |
There was a problem hiding this comment.
Who runs the merger here exactly? Also, here is very little gain in having all the component separated in the case of simple tests like this.
It would be good though if you could bring this more extensive deployment to the firehose-docs repository, improving the HA component (but using dummy-blockchain as demo).
| @@ -0,0 +1,46 @@ | |||
| # Monad Devnet Setup for Battlefield Testing | |||
|
|
|||
| Before running the Monad battlefield tests, ensure you have a monad nude configured. ([Monad docs](https://docs.monad.xyz/node-ops/full-node-installation)) | |||
There was a problem hiding this comment.
| Before running the Monad battlefield tests, ensure you have a monad nude configured. ([Monad docs](https://docs.monad.xyz/node-ops/full-node-installation)) | |
| Before running the Monad battlefield tests, ensure you have a monad node configured. ([Monad docs](https://docs.monad.xyz/node-ops/full-node-installation)) |
Do we also really need this? I thought the whole idea of the monad-devnet Docker compose was to avoid much of this need and only simpler config was needed on a physical machine for huge pages?
| ### Configuration Origins | ||
|
|
||
| Configuration files can be obtained from: | ||
|
|
||
| Validators: `https://bucket.monadinfra.com/validators/devnet/validators.toml` | ||
| Forkpoint: `https://bucket.monadinfra.com/forkpoint/devnet/forkpoint.toml` |
There was a problem hiding this comment.
Are those "source" configs? This section is not super clear to me.
|
|
||
| ## Running Tests | ||
|
|
||
| Once setup is complete, run the Monad battlefield tests: |
There was a problem hiding this comment.
If it's docker, we should the instruction to run it (and the machine we use internally should use the same instructions).
test/lib/constants.ts
Outdated
| "beginOrdinal", | ||
| "endOrdinal", |
There was a problem hiding this comment.
What's the status here?
There was a problem hiding this comment.
Almost done, Im double checking suicides and the rest of the cleanup
| "gasChanges": [ | ||
| { | ||
| "oldValue": "0", | ||
| "newValue": "900000", | ||
| "reason": "REASON_TX_INITIAL_BALANCE", | ||
| "ordinal": "1" | ||
| }, |
There was a problem hiding this comment.
Didn't we agreed to not have gas change on Monad?
There was a problem hiding this comment.
yes! They're not handled, stale data, will be removed in my next passover. They're in the exclude too
2 participants
No description provided.