Skip to content

Conversation

@StevenDavid
Copy link
Contributor

@StevenDavid StevenDavid commented Aug 16, 2025

Load gen Update

Issue #, if available:
No logged issue

Description of changes:
Removed extra files and consolidated to a specific approach to using artillery to run load tests. This approach uses playwright to better simulate user interactions with the website. You can now run tests locally or in fargate on ECS.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

…ight with artillery for better simulation of user experience
@niallthomson
Copy link
Collaborator

niallthomson commented Aug 21, 2025

Hey @StevenDavid we'll probably need to discuss this PR more. It looks like it removes pretty much all the existing code which will break:

  1. Running it easily locally both by yarn and docker
  2. The CI tests
  3. Assets like workshops and blogs which rely on the packaged version of the load generator scenarios

Adding Playwright should probably just be another options alongside the existing "basic" HTTP version, not a replacement. And in my experience using Playwright really benefits from having a Dockerfile since installing the dependencies can be a pain.

Could we:

  1. Add this is a separate scenario beside the existing one
  2. Add Playwright as a dependency in existing package.json, especially so its pinned
  3. We'll need to update the Dockerfile.run but this might benefit from larger rework to just rebase it on an AL2023 image that installs node.js then we just yarn install to we get both artillery and playwright

@StevenDavid
Copy link
Contributor Author

ok i added back in the standard load tests and updated the read.me to reflect the two options. Let me know if this looks good.

@StevenDavid StevenDavid changed the title Load generator Update feat: added a playwright option to Load generator Sep 9, 2025
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 this pull request may close these issues.

2 participants