upload-cloud-storage icon indicating copy to clipboard operation
upload-cloud-storage copied to clipboard

Re-enable load test

Open sethvargo opened this issue 3 years ago • 0 comments

TL;DR

The current load test can only run 3x in a short period and then we hit quota limits. This makes iterative testing and debugging extremely painful, so it's disabled.

Detailed design

We need to re-enable the load test. Ideas:

  • Move to a separate action that are allowed to fail
  • Only run on pushes to main (not on PRs)
  • Run on a cron schedule instead of on PRs and pushes

There also seems to be something weird with the authentication. The error messages we get are actually 503s from the STS service. It seems like the NPM package for Storage is doing an auth handshake for each file upload. I don't have more advanced telemetry to prove this, but that feels like a bug. The token should be cached and reused for its TTL /cc @bcoe

Additional information

No response

sethvargo avatar Apr 08 '22 15:04 sethvargo