DevOps: Define recipe for FastNet. #4066
Conversation
egieseke
requested review from
cce,
onetechnical,
algobarb,
algojack and
algolucky
test/testdata/deployednettemplates/recipes/fastnet/genesis.json
Outdated
Show resolved
Hide resolved
test/testdata/deployednettemplates/recipes/fastnet/genesis.json
Outdated
Show resolved
Hide resolved
| "VersionModifier": "", | ||
| "ConsensusProtocol": "future", | ||
| "FirstPartKeyRound": 0, | ||
| "LastPartKeyRound": 30000, |
There was a problem hiding this comment.
Is this too short? This would have been about 36 hours for a perf network, but for a long-lived network we want this to be much, much longer unless we're going to go through and renew partkeys ourselves. At 3m rounds it's only about 87 days if our rounds are 2.5s
There was a problem hiding this comment.
agreed, we should choose a bigger value for this longer-running network. It will take several hours to make all the keys though (depending on how many cores you have when running netgoal)
There was a problem hiding this comment.
Reverted back to 3 million rounds.
| @@ -0,0 +1,32 @@ | |||
| node_types = {"R":6, "N":12, "NPN":6} | |||
There was a problem hiding this comment.
For this round I'm kind of okay with mimicking betanet in terms of composition and distribution. Is this 24 non-co-located nodes? This seems like a lot until we can iron out some details.
@cce Any thoughts on this? I'm kind of okay with the 5 instance (1 relay + 4 partnode) setup to launch at this round while we evaluate our changes.
There was a problem hiding this comment.
Sure, however you think is best to roll it out, I assume it is relatively easy to update and grow the network as needed
There was a problem hiding this comment.
OK, reduced to 1 relay, 4 nodes, and 2 NPNs.
| @@ -0,0 +1,104 @@ | |||
| { | |||
| "NetworkName": "", | |||
There was a problem hiding this comment.
Might as well name this alphanet right from the start!
There was a problem hiding this comment.
Renamed to alphanet.
There was a problem hiding this comment.
Agreed with @onetechnical on his points and would like to see EnableRuntimeMetrics added to config
| @@ -0,0 +1,32 @@ | |||
| node_types = {"R":6, "N":12, "NPN":6} | |||
There was a problem hiding this comment.
Sure, however you think is best to roll it out, I assume it is relatively easy to update and grow the network as needed
test/testdata/deployednettemplates/recipes/fastnet/gen_topology.py
Outdated
Show resolved
Hide resolved
egieseke
dismissed stale reviews from cce and onetechnical
made suggested changes
There was a problem hiding this comment.
thanks for the update to the gen_topology.py, approving that that change, but should probably wait until @onetechnical and/or @cce take a look again.
… create-fastnet-recipe
|
Used this recipe to start alphanet network: alphanet.algodev.network |
Codecov Report
@@ Coverage Diff @@
## master #4066 +/- ##
==========================================
- Coverage 54.55% 54.47% -0.08%
==========================================
Files 391 391
Lines 48662 48662
==========================================
- Hits 26546 26511 -35
- Misses 19890 19926 +36
+ Partials 2226 2225 -1
Continue to review full report at Codecov.
|
Summary
Define the Algonet recipe to support Alphanet testing.
Test Plan
Deploy and test the network.