fix(llmobs): do not emit integration metrics in agentless mode

[Yun-Kim]

In #9912, we changed DD_LLMOBS_AGENTLESS_ENABLED to be stored in the global config setting config._llmobs_agentless_enabled, but forgot to also reference the config setting in the OpenAIIntegration.metrics_enabled() check. Instead, currently the openai integration only checks the env var meaning metrics will still be emitted in agentless mode if a user configures using config._llmobs_agentless_enabled or LLMObs.enable(agentless_enabled=True).

This PR fixes that to make metrics emitting check the central global config setting.

Checklist

• PR author has checked that all the criteria below are met
• The PR description includes an overview of the change
• The PR description articulates the motivation for the change
• The change includes tests OR the PR description describes a testing strategy
• The PR description notes risks associated with the change, if any
• Newly-added code is easy to change
• The change follows the library release note guidelines
• The change includes or references documentation updates if necessary
• Backport labels are set (if applicable)

Reviewer Checklist

• Reviewer has checked that all the criteria below are met
• Title is accurate
• All changes are related to the pull request's stated goal
• Avoids breaking API changes
• Testing strategy adequately addresses listed risks
• Newly-added code is easy to change
• Release note makes sense to a user of the library
• If necessary, author has acknowledged and discussed the performance implications of this PR as reported in the benchmarks PR comment
• Backport labels are set in a manner that is consistent with the release branch maintenance policy

[github-actions]

CODEOWNERS have been resolved as:

releasenotes/notes/fix-llmobs-agentless-openai-metrics-cfeb4458555d51be.yaml  @DataDog/apm-python
ddtrace/llmobs/_integrations/base.py                                    @DataDog/ml-observability
tests/contrib/openai/conftest.py                                        @DataDog/ml-observability
tests/contrib/openai/test_openai_llmobs.py                              @DataDog/ml-observability

[datadog-dd-trace-py-rkomorn]

Datadog Report

Branch report: yunkim/llmobs-int-metrics
Commit report: 2f90c56
Test service: dd-trace-py

✅ 0 Failed, 11363 Passed, 29599 Skipped, 36m 52.13s Total duration (35m 11.13s time saved)

[pr-commenter]

Benchmarks

Benchmark execution time: 2024-08-05 20:53:42

Comparing candidate commit 72b383c in PR branch yunkim/llmobs-int-metrics with baseline commit a23da22 in branch main.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 214 metrics, 2 unstable metrics.

[github-actions]

The backport to 2.11 failed:

The process '/usr/bin/git' failed with exit code 128

To backport manually, run these commands in your terminal:

# Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add .worktrees/backport-2.11 2.11
# Navigate to the new working tree
cd .worktrees/backport-2.11
# Create a new branch
git switch --create backport-10084-to-2.11
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x --mainline 1 0f87f164e8e4071beb4699e09b7a9b0f0890f9da
# Push it to GitHub
git push --set-upstream origin backport-10084-to-2.11
# Go back to the original working tree
cd ../..
# Delete the working tree
git worktree remove .worktrees/backport-2.11

Then, create a pull request where the base branch is 2.11 and the compare/head branch is backport-10084-to-2.11.