[chore] Fix loadtest setup script #33860
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description:
This PR fixes the load test pipeline which was broken for a while.
The array of tests was not iterated over correctly in https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/.github/workflows/scripts/setup_e2e_tests.sh and the first iteration of the loop resulted in the whole array being added to the matrix instead of just the first entry of the array.
I introduced a different way to read in the array of tests at the start of the setup bash script, which works now.
source for the solution
Link to tracking Issue: Fixes #33577
Testing: Tested successfully on my personal fork. This is a link to a load test run where the matrix is correctly parsed by github actions.
Documentation: nothing to document