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.

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

[chore] start and stop otlp exporter once #32809

Merged
merged 2 commits into from
May 3, 2024

Conversation

atoulme
Copy link
Contributor

@atoulme atoulme commented May 1, 2024

The testbed starts and stops the exporter once per signal, but we know it's the same exporter underneath.

This PR changes the behavior of testbed to start and stop the exporter just once. This change is needed to support open-telemetry/opentelemetry-collector#10059

testbed/testbed/receivers.go Outdated Show resolved Hide resolved
Co-authored-by: Yang Song <songy23@users.noreply.github.com>
if err = bor.metricsReceiver.Start(context.Background(), componenttest.NewNopHost()); err != nil {
return err
}
// we reuse the receiver across signals. Starting the log receiver starts the metrics and traces receiver.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is confusing, do we need the three factory calls above then?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you need them to set the logs, metrics and traces consumer.

@codeboten codeboten merged commit 3bec577 into open-telemetry:main May 3, 2024
160 checks passed
@github-actions github-actions bot added this to the next release milestone May 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants