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

[FA] Fix default APM setup #608

Merged
merged 5 commits into from
Aug 30, 2024
Merged

[FA] Fix default APM setup #608

merged 5 commits into from
Aug 30, 2024

Conversation

coignetp
Copy link
Collaborator

Fix default APM setup
Fill the default list if one of the following is True:

  • datadog_apm_instrumentation_libraries is set as all
  • datadog_apm_instrumentation_libraries is empty AND datadog_apm_instrumentation_enabled was set

Else, we use what's given in datadog_apm_instrumentation_libraries

@coignetp coignetp marked this pull request as ready for review August 30, 2024 11:50
@coignetp coignetp requested a review from a team as a code owner August 30, 2024 11:50
@coignetp coignetp merged commit 3458572 into main Aug 30, 2024
155 checks passed
@coignetp coignetp deleted the paul/fa-fix-default-apm branch August 30, 2024 12:22
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.

3 participants