Skip to content

Conversation

@anjalitrace2-nhs
Copy link
Contributor

@anjalitrace2-nhs anjalitrace2-nhs commented Jan 22, 2026

  • Make perf tests give a more detailed output report including a breakdown per scenario
  • The csv generated by the seed script now does not include any unprocessed items that failed to be upserted to dynamo
    • Still not an accurate representation of pointer count increase - items which were updated rather than created will count as processed and therefore be included in this csv too
  • When regenerating the input csv, cap it by default at 1.5m rows. As long as there are more rows than test iterations, this is sufficient as input for the perf tests. Generally we have 9k-14k iterations. Trying to load a 15m row csv into the test VUs was very slow and ate up >350% of my machine's CPU before the tests had even begun!
  • Point perftest env at 15m table for 15m testing
    • Updated the readme's instructions on how to do this
  • Ramped the TPS down to 1 per scenario. Scenarios run in parallel so this gives roughly producer = 5 TPS, consumer = 7 TPS.
  • Redid the baseline testing with these new TPS so that we could accurately compare to higher volumes

@github-actions
Copy link

🚀 PR environment successfully deployed.
Commit Hash: 907c67c6c8cf262e5bf4a64a3e592d00f2ab0b85
URL: https://nrl1875-01ba47.api.record-locator.dev.national.nhs.uk/

@github-actions
Copy link

🚀 PR environment successfully deployed.
Commit Hash: 907c67c6c8cf262e5bf4a64a3e592d00f2ab0b85
URL: https://nrl1875-01ba47.api.record-locator.dev.national.nhs.uk/

…not load a 15m row csv into the test runner VUs - they are unhappy
@github-actions
Copy link

🚀 PR environment successfully deployed.
Commit Hash: fcb24fdf99ced108bf6200e0b90583e37d364bc2
URL: https://nrl1875-01ba47.api.record-locator.dev.national.nhs.uk/

@sonarqubecloud
Copy link

@github-actions
Copy link

🚀 PR environment successfully deployed.
Commit Hash: 62021d6c34bcbc129d093596f225a664642c8d8e
URL: https://nrl1875-01ba47.api.record-locator.dev.national.nhs.uk/

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.

1 participant