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

Does dd-trace-go/tracer support DD_REMOTE_CONFIGURATION_ENABLED? #3060

Closed
tetsuro-ohyama opened this issue Dec 26, 2024 · 3 comments · Fixed by #3063
Closed

Does dd-trace-go/tracer support DD_REMOTE_CONFIGURATION_ENABLED? #3060

tetsuro-ohyama opened this issue Dec 26, 2024 · 3 comments · Fixed by #3063
Labels
low-hanging Tasks that are quick and/or easy. remote-config

Comments

@tetsuro-ohyama
Copy link

Does dd-trace-go support DD_REMOTE_CONFIGURATION_ENABLED like https://docs.datadoghq.com/tracing/trace_collection/library_config/cpp/ ?

Without that setting, it will always have remote configuration enabled when tracer is run.
If the agent is running with DD_REMOTE_CONFIGURATION_ENABLED: false, the tracer requests that agents get a setting at DD_REMOTE_CONFIG_POLL_INTERVAL_SECONDS intervals, but it consistently fails.

Also, please let me know if you have any ideas to stop the redundant requests.

@github-actions github-actions bot added the needs-triage New issues that have not yet been triaged label Dec 26, 2024
@tetsuro-ohyama tetsuro-ohyama changed the title Does dd-trace-go support DD_REMOTE_CONFIGURATION_ENABLED? Does dd-trace-go/tracer support DD_REMOTE_CONFIGURATION_ENABLED? Dec 26, 2024
@eliottness eliottness added low-hanging Tasks that are quick and/or easy. remote-config and removed needs-triage New issues that have not yet been triaged labels Jan 2, 2025
@eliottness
Copy link
Contributor

👋 Hello, thanks for asking. I believe we don't support it for now but this looks like a very low hanging fruit.

@eliottness
Copy link
Contributor

@tetsuro-ohyama I did a little something in #3063. Let me know if this what you are looking for. This could takes some time to be merged for testing and to make sure the the env var name is official and can be reported in the docs.

@tetsuro-ohyama
Copy link
Author

@eliottness Thank you for your quick response! It's what I want.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
low-hanging Tasks that are quick and/or easy. remote-config
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants