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

The gRPC exporter does not respect timeout settings #1498

Closed
spadger opened this issue Jan 31, 2025 · 0 comments · Fixed by #1499
Closed

The gRPC exporter does not respect timeout settings #1498

spadger opened this issue Jan 31, 2025 · 0 comments · Fixed by #1499
Labels
bug Something isn't working

Comments

@spadger
Copy link
Contributor

spadger commented Jan 31, 2025

Describe your environment

PHP 8.4, linux (in docker), php-fpm model
grpc module version => 1.69.0
latest version of transport-grpc

Steps to reproduce
There are 2 scenarios:

Scenario 1 (long timeout)

  1. Set the following env vars:
OTEL_EXPORTER_OTLP_PROTOCOL: grpc
OTEL_EXPORTER_OTLP_ENDPOINT: http://google.com:4310
OTEL_EXPORTER_OTLP_TIMEOUT: 20000
  1. export a batch of spans
  2. observe the time for the request to complete

Expected: Request completes after 10 seconds
Actual: timeout completes after about 6 seconds

Scenario 2: short timeout (under 1 second)

  1. Set the following env vars:
OTEL_EXPORTER_OTLP_PROTOCOL: grpc
OTEL_EXPORTER_OTLP_ENDPOINT: http://google.com:4310
OTEL_EXPORTER_OTLP_TIMEOUT: 999
  1. export a batch of spans
  2. observe the time for the request to complete

Expected: Request completes after about a second
Actual: Request completes immediately because the gRPC call immediately fails in less than 1ms (this also happens when you specify a valid collector endpoint and a 999ms timeout)

@spadger spadger added the bug Something isn't working label Jan 31, 2025
spadger added a commit to spadger/opentelemetry-php that referenced this issue Jan 31, 2025
brettmc pushed a commit that referenced this issue Feb 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant