ci: fix buildkite analytics for e2e
Created by: jhchabran
There was a bug in how the script was written for the E2E test, which prevented the API key to be properly assigned.
There was also another bug that slipped in that messed the exit code handling code in the QA test suite.
I have taken the opportunity of also improving the output, issuing a warning message when the test suite isn't working.
It's probably a good candidate for sentry reporting as well, I'll add that in a lta
Test plan
- CI Run and observing that:
- E2E test suite properly submits its report
- QA test suite properly submits its report
- Go test suite properly submits its report