Skip to content

[pkg/datadog] add hostname_detection_timeout and set default #40377

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

Conversation

jackgopack4
Copy link
Contributor

@jackgopack4 jackgopack4 commented May 29, 2025

Description

Adds hostname_detection_timeout config option and sets a reasonable default (25 seconds)

Link to tracking issue

Fixes #40373

Testing

manually run make kind-build and make kind-install-daemonset and verify collector does not crash after 30 seconds

Documentation

.chloggen and definition in datadogexporter examples folder

@jackgopack4 jackgopack4 force-pushed the jackgopack4/OTEL-2613-add-sourceprovider-timeout branch from 5c2fcf5 to de13287 Compare May 30, 2025 18:11
@jackgopack4 jackgopack4 requested review from songy23 and mx-psi May 30, 2025 18:13
@jackgopack4 jackgopack4 changed the title [pkg/datadog] add source_timeout and set default to 25s [pkg/datadog] add hostname_detection_timeout and set default May 30, 2025
@mx-psi mx-psi merged commit 9893d5d into open-telemetry:main Jun 2, 2025
178 checks passed
@mx-psi mx-psi deleted the jackgopack4/OTEL-2613-add-sourceprovider-timeout branch June 2, 2025 07:57
@github-actions github-actions bot added this to the next release milestone Jun 2, 2025
dd-jasminesun pushed a commit to DataDog/opentelemetry-collector-contrib that referenced this pull request Jun 23, 2025
…lemetry#40377)

<!--Ex. Fixing a bug - Describe the bug and how this fixes the issue.
Ex. Adding a feature - Explain what this achieves.-->
#### Description
Adds `hostname_detection_timeout` config option and sets a reasonable
default (25 seconds)
<!-- Issue number (e.g. #1234) or full URL to issue, if applicable. -->
#### Link to tracking issue
Fixes open-telemetry#40373 

<!--Describe what testing was performed and which tests were added.-->
#### Testing
manually run `make kind-build` and `make kind-install-daemonset` and
verify collector does not crash after 30 seconds
<!--Describe the documentation added.-->
#### Documentation
.chloggen and definition in datadogexporter examples folder
<!--Please delete paragraphs that you did not use before submitting.-->
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.

[exporter/datadog] pod restarts when deployed as daemonset and hostname not set in config
4 participants