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

debug transfer test with 16 parallel wallets #8983

Open
2 tasks
Tracked by #8920
ludamad opened this issue Oct 2, 2024 · 1 comment
Open
2 tasks
Tracked by #8920

debug transfer test with 16 parallel wallets #8983

ludamad opened this issue Oct 2, 2024 · 1 comment
Labels
C-pxe Component: PXE (Private eXecution Envrionment) T-bug Type: Bug. Something is broken.

Comments

@ludamad
Copy link
Collaborator

ludamad commented Oct 2, 2024

  • checkout master

Observe:
If end-to-end/src/spartan/transfer.test.ts has WALLET_COUNT=1 then test passes, otherwise see error below

Two options:
Automated workflow, slow iteration, use:

  • earthly ./yarn-project/+export-e2e-test-images

  • cd yarn-project/end-to-end

  • NAMESPACE=transfer FRESH_INSTALL=true VALUES_FILE=default.yaml ./scripts/network_test.sh ./src/spartan/transfer.test.ts

  • Better workflow, use

  • Call network_test.sh but comment out the docker run call to set up but run no tests in docker then, having repo locally built, use:
    SCENARIO=default PXE_URL=http://localhost:9080 DEBUG="aztec:*" LOG_LEVEL=debug LOG_JSON=1 yarn test ./src/spartan/transfer.test.ts

This will allow for quicker iteration (don't even need to compile the tests if they are just part of end-to-end)

Action items:

  • Ensure that this error is improved in this case
  • Investigate why parallelism here fails, doesn't seem to come up in the kubernetes logs
@ludamad ludamad changed the title is parallelism the issue with transfer test? debug transfer test with 16 parallel wallets Oct 2, 2024
@ludamad ludamad added T-bug Type: Bug. Something is broken. C-pxe Component: PXE (Private eXecution Envrionment) labels Oct 2, 2024
@ludamad
Copy link
Collaborator Author

ludamad commented Oct 2, 2024

Currently, unless WALLET_COUNT=1 in ./src/spartan/transfer.test.ts, somehow fails due to parallelism with this error

 aztec:js:contract_interaction [DEBUG] Sent deployment tx of SchnorrAccount contract with deployment address 0x2b2eb4f8b8fe865f12eb03c38806c1a82bb8d4f09eef412a9c9f6f8c4ed522b2 +0ms
  aztec:js:contract_interaction [DEBUG] Sent deployment tx of SchnorrAccount contract with deployment address 0x133349c69b3b9e5a4e4418d8243e2c4d140086921689c31168912c2cb2e9f2c1 +0ms
  aztec:js:contract_interaction [DEBUG] Sent deployment tx of SchnorrAccount contract with deployment address 0x08822d7355bc089eaa45c394b987b17e0c15d309fa48dc469d081b1987137f08 +0ms
FAIL src/spartan/transfer.test.ts (316.831 s)
  token transfer test
    ✕ can get info (4 ms)
    ✕ can transfer 1 token privately and publicly

  ● token transfer test › can get info

    (JSON-RPC PROPAGATED) fetch failed

      49 |     return resp.data;
      50 |   } else if (resp.status >= 400 && resp.status < 500) {
    > 51 |     throw new NoRetryError('(JSON-RPC PROPAGATED) ' + resp.data.error.message);
         |           ^
      52 |   } else {
      53 |     throw new Error('(JSON-RPC PROPAGATED) ' + resp.data.error.message);
      54 |   }

      at axiosFetch (../../aztec.js/src/rpc_clients/node/index.ts:51:11)
      at retry (../../foundation/src/retry/index.ts:56:14)
      at fetch (../../aztec.js/src/rpc_clients/node/index.ts:67:12)
      at request (../../foundation/src/json-rpc/client/json_rpc_client.ts:116:17)
      at ../../aztec.js/src/contract/sent_tx.ts:130:27
      at retryUntil (../../foundation/src/retry/index.ts:88:20)
      at DeployAccountSentTx.waitForReceipt (../../aztec.js/src/contract/sent_tx.ts:128:12)
      at DeployAccountSentTx.wait (../../aztec.js/src/contract/sent_tx.ts:80:21)
      at DeployAccountSentTx.wait (../../aztec.js/src/account_manager/deploy_account_sent_tx.ts:38:21)
          at async Promise.all (index 2)
      at fixtures/snapshot_manager.ts:507:5
      at Object.<anonymous> (spartan/transfer.test.ts:66:29)

  ● token transfer test › can transfer 1 token privately and publicly

    (JSON-RPC PROPAGATED) fetch failed

      49 |     return resp.data;
      50 |   } else if (resp.status >= 400 && resp.status < 500) {
    > 51 |     throw new NoRetryError('(JSON-RPC PROPAGATED) ' + resp.data.error.message);
         |           ^
      52 |   } else {
      53 |     throw new Error('(JSON-RPC PROPAGATED) ' + resp.data.error.message);
      54 |   }

      at axiosFetch (../../aztec.js/src/rpc_clients/node/index.ts:51:11)
      at retry (../../foundation/src/retry/index.ts:56:14)
      at fetch (../../aztec.js/src/rpc_clients/node/index.ts:67:12)
      at request (../../foundation/src/json-rpc/client/json_rpc_client.ts:116:17)
      at ../../aztec.js/src/contract/sent_tx.ts:130:27
      at retryUntil (../../foundation/src/retry/index.ts:88:20)
      at DeployAccountSentTx.waitForReceipt (../../aztec.js/src/contract/sent_tx.ts:128:12)
      at DeployAccountSentTx.wait (../../aztec.js/src/contract/sent_tx.ts:80:21)
      at DeployAccountSentTx.wait (../../aztec.js/src/account_manager/deploy_account_sent_tx.ts:38:21)
          at async Promise.all (index 2)
      at fixtures/snapshot_manager.ts:507:5
      at Object.<anonymous> (spartan/transfer.test.ts:66:29) 

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-pxe Component: PXE (Private eXecution Envrionment) T-bug Type: Bug. Something is broken.
Projects
Status: Todo
Development

No branches or pull requests

1 participant