fix(llmobs): remove invocations of LLMObs.enable() from supported integrations

[sabrenner]

Changes Made

Removing all mentions of LLMObs.enable() from integration patching, as that patching logic was moved to LLMObs.enable() itself. This change is reflected as necessary in tests as well.

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

[datadog-dd-trace-py-rkomorn]

Datadog Report

Branch report: sabrenner/llmobs-apm-fix
Commit report: da0f71d
Test service: dd-trace-py

✅ 0 Failed, 2264 Passed, 722 Skipped, 1h 7m 44.72s Total duration (30m 48.47s time saved)

[pr-commenter]

Benchmarks

Benchmark execution time: 2024-05-14 14:36:23

Comparing candidate commit da0f71d in PR branch sabrenner/llmobs-apm-fix with baseline commit cee0c7b in branch main.

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

[Yun-Kim]

I think a better maintainable way to fix this is to remove any LLMObs.enable() mentions in integration patch code, since the LLMObs.enable() function will handle the actual openai/langchain/bedrock patching once #9172 is merged.

[sabrenner]

@Yun-Kim Good point. I can make the change and once those PRs are merged, check that the tests are still good.

[sabrenner]

Closing in favor of #9172, which will include this change and relevant test changes