Open arya2 opened 1 year ago
These are just some notes for if we eventually do this ticket. No action is needed at this time.
We might also want to take a look at:
It seems like some of these changes would benefit from the Regtest variant containing a consensus rule configuration. (Like zcashd has for all its network variants.)
It would also be a lot easier to make this change if we refactored all our network conditions into methods on the Network enum. For example, a has_slow_start() method, rather than a match statement buried in the zebra-consensus code.
- update
From<Network> for &'static str
fordb_path()
We either need to work out how to make sure the consensus rules match the cached state, or make regtest ephemeral by default.
This isn't an issue for mainnet or testnet because their rules are fixed and can't be configured.
- Set
peerset_initial_target_size
to 0 (Zebra should not make any peer connections)
This change isn't enough to stop Zebra making outbound connections, because Zebra will try to connect to new peers every minute or so. Instead, the initial seed peers list needs to be empty for regtest.
- Update prefixes
Which prefixes?
- Match other differences between zcashd testnet vs regtest in chainparams
If we're not going to connect to other nodes by default, then compatibility with zcashd seems like a lower priority. So we might want to categorise this ticket into:
That way, when we schedule this work, we can split the ticket, give the categories different priorities, then do the estimates.
Here's one request for regtest in Zebra: https://twitter.com/thecodebuffet/status/1674606998908727297
Screenshot in case twitter goes down:
This is currently blocked by some kind of network refactor. (It doesn't have to be #1135 specifically.)
Hey team! Please add your planning poker estimate with Zenhub @arya2 @oxarbitrage @teor2345 @upbqdn
@mpguerra this ticket is outdated. @arya2 and I have had some private conversations about it, and I think we should update it before we estimate it. (Or replace it with another ticket.)
For example:
I'd like to split the tasks in this ticket based on what the ZSA team needs for their testing, then do the rest of regtest later.
- Add a
Regtest
variant toNetwork
, updateFrom<Network> for &'static str
fordb_path()
, default port, and genesis hash
It will be much easier to gradually add network settings inside the existing Testnet
variant. Adding new variants requires changing a lot of code.
- Add config field or CLI option for accepting network upgrade heights
This should be split into a different ticket, it's not required to match the default zcashd
regtest.
I added these issues to the ticket description, we might need to fix them depending on the required scope:
https://github.com/ZcashFoundation/zebra/pull/7770#discussion_r1364969449
Personally, the lack of regtest support blocks me from moving to zebrad because it is too hard to write integration tests.
Personally, the lack of regtest support blocks me from moving to zebrad because it is too hard to write integration tests.
@hhanh00 Is it enough for Zebra to support NU5 on its Regtest, or is it important that Zebra support Canopy and prior network upgrades on Regtest as well?
Just nu5 is enough but I would also need the ability to generate, invalidate and reconsider blocks. This is how the project I work on tests forks and reorganizations.
@arya2 it seems like there are still some meaningful gaps between zebra regtest mode and zcashd regtest mode. See: https://discord.com/channels/809218587167293450/811004767043977288/1280987474918772858 Do we know what these are? If so we should update this issue with the remaining work to make them compatible
Motivation
This has been requested, and could be useful for testing.
Depends-On: #7845
Possible Design
NetworkUpgrade::activation_height
to use zcashd defaults or user-defined heightsPoWLimit
must be less than or equal to(2^256 - 1) / PoWAveragingWindow
(approximately2^251
) to avoid overflowSLOW_START_INTERVAL
checks (nSubsidySlowStartInterval
is 0 for regtest in zcashd)