-
Notifications
You must be signed in to change notification settings - Fork 2.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Load test failing on dedicated runner #27429
Comments
An error appears to be the load tests to fail, but it's not being checked which makes it hard to debug. Helps with open-telemetry#27429 Signed-off-by: Alex Boten <[email protected]>
An error appears to be the load tests to fail, but it's not being checked which makes it hard to debug. Helps with #27429 Signed-off-by: Alex Boten <[email protected]> Co-authored-by: Bogdan Drutu <[email protected]>
The numbers reported by the dedicated runner pretty different from what we're seeing on github shared runners. A sample below shows the results on shared runners:
And on the dedicated runner:
The dedicated hardware runs with 16 cores, compared to the 2 cores for shared runners. The question I have is should the testbed be modified for set the number of CPUs used by the tests? Or should the expected values of the tests match the new hardware? |
One way to set number of CPU in the childProcessCollector's
|
Setting the number of cores seems like a good idea. Can a runner run multiple jobs simultaneously? Assuming so, cores may end up being restricted inconsistently based on the number of jobs running. |
As far as I can tell, the runners currently only run one job at a time now, but you're right that it could change in the future. |
This will result in more consistent benchmarks across different environments. Fixes #27429 Signed-off-by: Alex Boten <[email protected]>
An error appears to be the load tests to fail, but it's not being checked which makes it hard to debug. Helps with open-telemetry#27429 Signed-off-by: Alex Boten <[email protected]> Co-authored-by: Bogdan Drutu <[email protected]>
This will result in more consistent benchmarks across different environments. Fixes open-telemetry#27429 Signed-off-by: Alex Boten <[email protected]>
**Description:** Adding a feature - These changes add a new `WithEnvVar` `ChildProcessOption` to be able to influence the child process environment without acting on the current environment. They also move the `GOMAXPROCS=2` setting added in dd8e010 to each invoking test because though being helpful to address #27429, the constraint doesn't seem applicable to the helper directly. Limiting the utility as a whole instead of specifying in the test context cannot be easily worked around and is interfering w/ some load testing efforts.
…0491) **Description:** Adding a feature - These changes add a new `WithEnvVar` `ChildProcessOption` to be able to influence the child process environment without acting on the current environment. They also move the `GOMAXPROCS=2` setting added in open-telemetry@dd8e010 to each invoking test because though being helpful to address open-telemetry#27429, the constraint doesn't seem applicable to the helper directly. Limiting the utility as a whole instead of specifying in the test context cannot be easily worked around and is interfering w/ some load testing efforts.
Component(s)
No response
Describe the issue you're reporting
The following failure occurs when running the load test on dedicated runners:
https://github.com/open-telemetry/opentelemetry-collector-contrib/actions/runs/6408839554/job/17401287975
The text was updated successfully, but these errors were encountered: