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

Allow changing otel endpoint setting without restart #62

Merged
merged 1 commit into from
Sep 18, 2024

Conversation

hlinnaka
Copy link
Contributor

Also the naptime and connection timeout settings. If no endpoint was set at server start, though, the background worker is not started, and changing the endpoint will have no effect. Similarly, if you set endpoint to empty while the server is running, the background worker will keep running, but won't do anything. Changing the registration of the background worker in response to config changes is difficult. But at least you can change the endpoint without a restart.

Also the naptime and connection timeout settings. If no endpoint was
set at server start, though, the background worker is not started, and
changing the endpoint will have no effect. Similarly, if you set
endpoint to empty while the server is running, the background worker
will keep running, but won't do anything. Changing the registration of
the background worker in response to config changes is difficult. But
at least you can change the endpoint without a restart.
Copy link
Collaborator

@bonnefoa bonnefoa left a comment

Choose a reason for hiding this comment

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

Change looks good and works as expected. I will add a tap test to cover this behaviour.

@bonnefoa bonnefoa merged commit 3763ede into DataDog:main Sep 18, 2024
4 checks passed
@hlinnaka
Copy link
Contributor Author

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants