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

psm.test.js behaves flakey in z:acceptance #10728

Open
anilhelvaci opened this issue Dec 18, 2024 · 0 comments · May be fixed by #10747
Open

psm.test.js behaves flakey in z:acceptance #10728

anilhelvaci opened this issue Dec 18, 2024 · 0 comments · May be fixed by #10747
Assignees

Comments

@anilhelvaci
Copy link
Collaborator

anilhelvaci commented Dec 18, 2024

Problem Definition

In CI we observed some flakiness with psm.test.js.

https://github.com/Agoric/agoric-sdk/actions/runs/12363738833/job/34505666038?pr=10692#step:10:8349
https://github.com/Agoric/agoric-sdk/actions/runs/12363738833/job/34505666038?pr=10692#step:10:7692

The error message being;

  Error: psmSwap tx send failed: {"exitCode":2}
      at psmSwap (file:///usr/src/proposals/z:acceptance/test-lib/psm-lib.js:552:11)
      at async file:///usr/src/proposals/z:acceptance/psm.test.js:195:3
7:00PM ERR Error on broadcastTxCommit err="timed out waiting for tx to be included in a block" module=rpc
psmSwap tx send failed {
  command: 'agoric wallet --keyring-backend=test send --offer /tmp/agops.OlQ --from agoric1zlesuj60a3f59slushj65xx5asmzd8s7mn59qm --verbose',
  durationMs: 11091.834717,
  shortMessage: "Command failed with exit code 2: agoric wallet '--keyring-backend=test' send --offer /tmp/agops.OlQ --from agoric1zlesuj60a3f59slushj65xx5asmzd8s7mn59qm --verbose",
  cause: undefined,
  exitCode: 2,
…
    'Error: error in json rpc client, with http response metadata: (Status: 200 OK, Protocol HTTP/1.1). RPC error -32603 - Internal error: timed out waiting for tx to be included in a block\n' +

Cause

This issue seems to be related to -bblock flag that is used, used in;

Looking at cosmos-sdk repo, we can see that -bblock is deprecated as of v0.47. Here are some issues talking about this;

Here's the PR that deprecated -bblock: cosmos/cosmos-sdk#12659

This issue describes a possible reproducing scenario: cosmos/cosmos-sdk#8510

Why do we only see this error in psm.test.js?

I went over all tests under z:acceptance and observed that all tests other than psm.test.js use executeOffer which uses agops.perf that doesn't specify a broadcast option (sync, async, block).

The reason why we didn't use executeOffer in psm.test.js is; in order to test auto provisioning we needed to agoric wallet send as agops.perf needs a provisioned account to function.

Solution

Start using sendOfferAgd and remove -bblock option from it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant