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

[Bug]: otlp example does not work without configuration update #1998

Open
krlohnes opened this issue Aug 7, 2024 · 2 comments
Open

[Bug]: otlp example does not work without configuration update #1998

krlohnes opened this issue Aug 7, 2024 · 2 comments
Labels
documentation/examples Improvements or additions to documentation or examples

Comments

@krlohnes
Copy link

krlohnes commented Aug 7, 2024 •
edited
Loading

What happened?

I tried to run basic-otlp example and it simply didn't work.

I update the first section of otel-collector-config.yml to the following

receivers:
  otlp:
    protocols:
      grpc:
        endpoint: 0.0.0.0:4317

and it started working.

API Version

0.24.0

SDK Version

0.24.1

What Exporter(s) are you seeing the problem on?

OTLP

Relevant log output

OpenTelemetry log error occurred. Exporter otlp encountered the following errors: the grpc server returns error (The operation was cancelled): , detailed error message: operation was canceled
@krlohnes krlohnes added bug Something isn't working triage:todo Needs to be traiged. labels Aug 7, 2024
@cijothomas
Copy link
Member

Need a similar fix like https://github.com/open-telemetry/opentelemetry-rust/pull/1982/files to this example too. @krlohnes Would you mind sending a PR to fix?

@cijothomas cijothomas added documentation/examples Improvements or additions to documentation or examples and removed bug Something isn't working triage:todo Needs to be traiged. labels Aug 8, 2024
@lalitb
Copy link
Member

lalitb commented Aug 8, 2024

Also, we need similar fix in integration tests - https://github.com/open-telemetry/opentelemetry-rust/blob/main/opentelemetry-otlp/tests/integration_test/otel-collector-config.yaml

krlohnes added a commit to krlohnes/opentelemetry-rust that referenced this issue Aug 8, 2024
@krlohnes
Fix config for otlp example and integration test
b06afb3 
Fixes open-telemetry#1998

With updates to the otel collector 0.104+, the collector now binds to
localhost by default. The config now needs to explicitly allow access
from outside the docker container.
@krlohnes krlohnes mentioned this issue Aug 8, 2024
Closed
4 tasks
lalitb added a commit to krlohnes/opentelemetry-rust that referenced this issue Aug 14, 2024
@lalitb
Merge branch 'main' into open-telemetryGH-1998
07e26ba
TommyCpp added a commit to krlohnes/opentelemetry-rust that referenced this issue Aug 14, 2024
@TommyCpp
Merge branch 'main' into open-telemetryGH-1998
1a37726
cijothomas added a commit to krlohnes/opentelemetry-rust that referenced this issue Oct 5, 2024
@cijothomas
Merge branch 'main' into open-telemetryGH-1998
afb3f38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
documentation/examples Improvements or additions to documentation or examples
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix config for otlp example and integration test krlohnes/opentelemetry-rust
3 participants
@lalitb @cijothomas @krlohnes