-
Notifications
You must be signed in to change notification settings - Fork 524
DevOps: Added recipe for transaction sync testing. #3884
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
algobarb
left a comment
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.
Is this just for Phase0 testing and a Betanet replica? Or would this be the format used for the Betanet Deployment?
Betanet TxnSync deployement would need 4 betanet relays and 10 additional non-participation nodes using m5d.xlarge machines, geo-distributed. I see participation nodes and that all the nodes here are c5.4xlarge machines and in us-east-2. Can we clarify the purpose of the recipe?
@algobarb This recipe is used to create the betanet network extension for the txn sync testing. I used c5.4xlarge machines to match what was used for the current betanet relays and nodes based on "Create R6..R9 on different geo-zones. Use the same host configuration used for the rest of the betanet relays." and "Create 5 hosts, using the same host configuration as R5". We can easily change to md5.xlarge if those are required. I will update the recipe to spread out to different regions, good catch! |
Codecov Report
@@ Coverage Diff @@
## master #3884 +/- ##
==========================================
+ Coverage 49.97% 50.01% +0.03%
==========================================
Files 393 393
Lines 68211 68211
==========================================
+ Hits 34089 34114 +25
+ Misses 30417 30399 -18
+ Partials 3705 3698 -7
Continue to review full report at Codecov.
|
|
|
"Expected cost: Use m5d.xlarge for each of the relays/nodes." under BetaNet Deployment in the Deployment doc. Host configuration is still the same, but machine type was supposed to be different, but John may not mind if it's c5.4xlarge. When we reviewed the documentation we settled on that we need a total of 9 machines: 4 machines with 1 Relay each, and then 5 machines with 2 NPN each. So we'll have to adjust this and maybe have a review with John since Tsachi is out |
algobarb
left a comment
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.
Thanks for clarifying. I wasn't sure if there was a need for them to be NPN in the original doc.
Summary
Added "txnsync" recipe matching configuration requirements for txn sync testing on Betanet.
Test Plan
Tested recipe by using algonet to launch a network. Verified that the rounds were progressing.