Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[CI] Fix load-tests #29310

Closed
TylerHelmuth opened this issue Nov 16, 2023 · 1 comment · Fixed by #29403
Closed

[CI] Fix load-tests #29310

TylerHelmuth opened this issue Nov 16, 2023 · 1 comment · Fixed by #29403
Assignees
@codeboten
Labels
bug Something isn't working ci-cd CI, CD, testing, build issues flaky test a test is flaky good first issue Good for newcomers help wanted Extra attention is needed

Comments

@TylerHelmuth
Copy link
Member

TylerHelmuth commented Nov 16, 2023 •
edited
Loading

Description

The load-test ci job hasn't passed on main in a looooong time: https://github.com/open-telemetry/opentelemetry-collector-contrib/actions/workflows/load-tests.yml?query=branch%3Amain. I believe the last one to pass was https://github.com/open-telemetry/opentelemetry-collector-contrib/actions/runs/6536440212. We should fix this or at least disable whatever test is broke.
@TylerHelmuth TylerHelmuth added bug Something isn't working needs triage New item requiring triage help wanted Extra attention is needed good first issue Good for newcomers ci-cd CI, CD, testing, build issues flaky test a test is flaky and removed needs triage New item requiring triage labels Nov 16, 2023
@codeboten codeboten self-assigned this Nov 16, 2023
@codeboten
Copy link
Contributor

This looks related to the JaegerGRPC changes I submitted a while ago. Will take a look

@codeboten codeboten mentioned this issue Nov 20, 2023
Merged
codeboten pushed a commit that referenced this issue Nov 20, 2023
[testbed] update jaeger test (#29403)
c176eaf 
Since the removal of the jaeger exporter, the TestTrace10kSPS/JaegerGRPC
has been failing. This was caused by the testbed expecting to be able to
export data using the jaeger exporter. This changes the test to use OTLP
and adds a new scenario method that supports specifying a different
backend for this type of use-case.

Fixes #29310

Signed-off-by: Alex Boten <aboten@lightstep.com>
@github-actions github-actions bot mentioned this issue Nov 21, 2023
Closed
RoryCrispin pushed a commit to ClickHouse/opentelemetry-collector-contrib that referenced this issue Nov 24, 2023
@RoryCrispin
[testbed] update jaeger test (open-telemetry#29403)
b46824d 
Since the removal of the jaeger exporter, the TestTrace10kSPS/JaegerGRPC
has been failing. This was caused by the testbed expecting to be able to
export data using the jaeger exporter. This changes the test to use OTLP
and adds a new scenario method that supports specifying a different
backend for this type of use-case.

Fixes open-telemetry#29310

Signed-off-by: Alex Boten <aboten@lightstep.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@codeboten codeboten

Labels
bug Something isn't working ci-cd CI, CD, testing, build issues flaky test a test is flaky good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[testbed] update jaeger test codeboten/opentelemetry-collector-contrib
2 participants
@codeboten @TylerHelmuth