Skip to content
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

Loadgen test of PSM #6004

Closed
Tracked by #6041
mhofman opened this issue Aug 19, 2022 · 4 comments
Closed
Tracked by #6041

Loadgen test of PSM #6004

mhofman opened this issue Aug 19, 2022 · 4 comments
Assignees
Labels
enhancement New feature or request Inter-protocol Overarching Inter Protocol performance Performance related issues
Milestone

Comments

@mhofman
Copy link
Member

mhofman commented Aug 19, 2022

What is the Problem Being Solved?

We need to add support for PSM in the loadgen to get some performance numbers for the chain when only that contract is running

Description of the Design

Add PSM loadgen contract (based on agsolo deploy scripts)
Bootstrap chain with a config with PSM enabled and no other contracts (possibly PSM prod config)
Perform loadgen with just PSM load.

@mhofman mhofman added enhancement New feature or request performance Performance related issues MUST-HAVE labels Aug 19, 2022
@mhofman mhofman self-assigned this Aug 19, 2022
@dckc
Copy link
Member

dckc commented Aug 22, 2022

I got started on a psm-load.js script: #6015 (comment)

It's a 1-file script using zx. The PSM-only release might not support deploy scripts at all.

@dckc
Copy link
Member

dckc commented Aug 23, 2022

A Sample run of 10 trades

elapsed time: 42-60sec
2 didn't get into the list of wallet offers at all, somehow.
p.s. because a transaction had already been submitted from the account in the same account

instagoric/bases/shared/instagoric-server$ node psm-load.mjs 10 3 >,psm.log 2>&1 &
$ git log --oneline -1
a21a19f (HEAD -> dc-psm-load, origin/dc-psm-load) feat(psm-load): manyTrades

Agoric SDK version:

agoric-sdk/packages/cosmic-swingset$ git log --oneline -1
afabe7ff1 (HEAD -> dc-boot-psm, origin/dc-boot-psm) chore(vats): startWalletFactory without provisioning mailbox access

with psm-tool from fe6caab

,psm.log

@dckc
Copy link
Member

dckc commented Aug 23, 2022

87 trades in 2 hrs

I ran node psm-load.mjs 100 5 but it stopped at 87 because I was parsing node's console output as almost-JSON but I didn't account for ... abbreviation.

The logs are pretty messy. They should be all JSON-lines, and they should make it easier to correlate stuff.
I include the chain log for reference about block numbers and such.

,psm100.log

,chain.log

"not a live payment" distraction in logs

2022-08-23T07:43:29.012Z SwingSet: ls: v6: Error#184: Object [Alleged: IST payment] { getAllegedBrand: [Function: method] } was not a live payment for brand Object [Alleged: IST brand] { isMyIssuer: [Function: method], getAllegedName: [Function: method], getDisplayInfo: [Function: method], getAmountSchema: [Function: method] } . It could be a used-up payment, a payment for another brand, or it might not be a payment at all.

Block times: normal

  height blockTime
  634 2022-08-23 5:39:15
  2092 2022-08-23 7:41:04
     
blocks 1458  
elapsed   2:01:49
avg block time   0:00:05

TODO: messages from accounts without smart wallets

how much grief do they cause?

@dckc dckc removed their assignment Aug 25, 2022
@dckc
Copy link
Member

dckc commented Aug 25, 2022

@arirubinstein is on the frontier of this one now.

@Tartuffo Tartuffo added the Inter-protocol Overarching Inter Protocol label Aug 31, 2022
@Tartuffo Tartuffo added this to the Mainnet 1 RC0 milestone Aug 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Inter-protocol Overarching Inter Protocol performance Performance related issues
Projects
None yet
Development

No branches or pull requests

4 participants