feat(llmobs): only patch llm integrations if enabled

[Yun-Kim]

This PR does two things:

• Changes the LLMObs.enable() argument integrations list of integrations to enable to --> integrations_enabled a boolean flag whether or not the LLM integrations should be patched.
• Restructures the openai/langchain/bedrock test suites to separate out LLMObs tests into a separate file, i.e. test_bedrock_llmobs.py. This will help maintainability. No functionality is changed in the tests other than to reflect the above change.

Previously we could use LLMObs.enable(integrations=[...]) to enable which LLMObs integrations to patch, but this is unnecessary functionality that is already supported by ddtrace. Users should just use ddtrace.patch_all()/ddtrace.patch() instead.

Checklist

• Change(s) are motivated and described in the PR description
• Testing strategy is described if automated tests are not included in the PR
• Risks are described (performance impact, potential for breakage, maintainability)
• Change is maintainable (easy to change, telemetry, documentation)
• Library release note guidelines are followed or label changelog/no-changelog is set
• Documentation is included (in-code, generated user docs, public corp docs)
• Backport labels are set (if applicable)
• If this PR changes the public interface, I've notified @DataDog/apm-tees.

Reviewer Checklist

• Title is accurate
• All changes are related to the pull request's stated goal
• Description motivates each change
• Avoids breaking API changes
• Testing strategy adequately addresses listed risks
• Change is maintainable (easy to change, telemetry, documentation)
• Release note makes sense to a user of the library
• 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

[pr-commenter]

Benchmarks

Benchmark execution time: 2024-05-27 21:04:58

Comparing candidate commit 040bf38 in PR branch yunkim/llmobs-patch-all with baseline commit 3de0cf5 in branch main.

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

[datadog-dd-trace-py-rkomorn]

Datadog Report

Branch report: yunkim/llmobs-patch-all
Commit report: dd34406
Test service: dd-trace-py

❌ 62 Failed (0 Known Flaky), 1876 Passed, 1290 Skipped, 33m 8.73s Total duration (20m 52.53s time saved)

❌ Failed Tests (62)

This report shows up to 5 failed tests.

• test_llmobs_ai21_invoke - test_bedrock_llmobs.py - Details

  Expand for error

   ('bedrock_client', <FixtureRequest for <Function test_llmobs_ai21_invoke>>)
  

• test_llmobs_ai21_invoke - test_bedrock_llmobs.py - Details

  Expand for error

   ('bedrock_client', <FixtureRequest for <Function test_llmobs_ai21_invoke>>)
  

• test_llmobs_ai21_invoke - test_bedrock_llmobs.py - Details

  Expand for error

   ('bedrock_client', <FixtureRequest for <Function test_llmobs_ai21_invoke>>)
  

• test_llmobs_ai21_invoke - test_bedrock_llmobs.py - Details

  Expand for error

   ('bedrock_client', <FixtureRequest for <Function test_llmobs_ai21_invoke>>)
  

• test_llmobs_amazon_invoke - test_bedrock_llmobs.py

[Yun-Kim]

Closing in favor of splitting this into smaller PRs (#9398, #9397)