pokt-network / poktroll

The official Shannon upgrade implementation of the Pocket Network Protocol implemented using Rollkit.dev
MIT License
15 stars 6 forks source link

[Testing] Stress / load test ephemeral & persistent chains #540

Closed red-0ne closed 1 month ago

red-0ne commented 1 month ago

Summary

Make the necessary adjustments to the load test framework to be able to run on persistent chains.

Issue

The load testing framework was not able to run without scaling Gateways and Suppliers. Which is not the goal for testing persistent chains like DevNet and TestNet.

Type of change

Select one or more:

Testing

Documentation changes (only if making doc changes)

Local Testing (only if making code changes)

PR Testing (only if making code changes)

Sanity Checklist

Olshansk commented 1 month ago

@red-0ne Please re-request review and update PR title when this is ready.

red-0ne commented 1 month ago

Possible improvement:

Be able to specify the test plan (feature files) to execute from the manifest file

red-0ne commented 1 month ago

@Olshansk , I think the address/keyName issue is creating a lot of ambiguity. Will take some time to get rid it by only relying on addresses.

Olshansk commented 1 month ago

@red-0ne Love the simplification, so much clearer now!

Only big remaining question is whether we need both address and accAddress in accountInfo.

I know I can pull the PR and figure it out myself but just asking to expedite things.

red-0ne commented 1 month ago

@Olshansk , accAddress removed successfully

github-actions[bot] commented 1 month ago

The CI will now also run the e2e tests on devnet, which increases the time it takes to complete all CI checks. If you just created a pull request, you might need to push another commit to produce a container image DevNet can utilize to spin up infrastructure. You can use make trigger_ci to push an empty commit.