feat(pymongo): add support for v4.9

[mabdinur]

Motivation

• Add and test support for pymongo v4.9.1

Reproduction

pip install ddtrace==2.11.6 pymongo==4.9.1
python -c "import ddtrace; ddtrace.patch_all(pymongo=True); import pymongo"

Output

  File "dd-trace-py/ddtrace/_monkey.py", line 165, in on_import
    imported_module = importlib.import_module(path)
                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/munirabdinur/.pyenv/versions/3.12.2/lib/python3.12/importlib/__init__.py", line 90, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "<frozen importlib._bootstrap>", line 1387, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1360, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1331, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 935, in _load_unlocked
  File "dd-trace-py/ddtrace/internal/module.py", line 309, in _exec_module
    self.loader.exec_module(module)
  File "<frozen importlib._bootstrap_external>", line 995, in exec_module
  File "<frozen importlib._bootstrap>", line 488, in _call_with_frames_removed
  File "dd-trace-py/ddtrace/contrib/pymongo/__init__.py", line 45, in <module>
    from .patch import get_version
  File "<frozen importlib._bootstrap>", line 1360, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1331, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 935, in _load_unlocked
  File "dd-trace-py/ddtrace/internal/module.py", line 309, in _exec_module
    self.loader.exec_module(module)
  File "dd-trace-py/ddtrace/contrib/pymongo/patch.py", line 51, in <module>
    _VERIFY_VERSION_CLASS = pymongo.pool.SocketInfo if _VERSION < (4, 5) else pymongo.pool.Connection

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/pymong-4-9-support-83f7c613e5e009e6.yaml             @DataDog/apm-python
ddtrace/contrib/internal/pymongo/patch.py                               @DataDog/apm-core-python @DataDog/apm-idm-python
riotfile.py                                                             @DataDog/apm-python
.riot/requirements/10a00e7.txt                                          @DataDog/apm-python
.riot/requirements/1424e42.txt                                          @DataDog/apm-python
.riot/requirements/14e85f3.txt                                          @DataDog/apm-python
.riot/requirements/16cae33.txt                                          @DataDog/apm-python
.riot/requirements/1ce9a99.txt                                          @DataDog/apm-python
.riot/requirements/1ed6ce0.txt                                          @DataDog/apm-python
.riot/requirements/328b28c.txt                                          @DataDog/apm-python
.riot/requirements/ad40916.txt                                          @DataDog/apm-python
.riot/requirements/b089663.txt                                          @DataDog/apm-python
.riot/requirements/b344fed.txt                                          @DataDog/apm-python
.riot/requirements/de53117.txt                                          @DataDog/apm-python
.riot/requirements/f9d0e8e.txt                                          @DataDog/apm-python

[pr-commenter]

Benchmarks

Benchmark execution time: 2024-09-20 21:33:11

Comparing candidate commit 4cfdc70 in PR branch munir/test-pymongo-4-9 with baseline commit 33daba9 in branch main.

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

[datadog-dd-trace-py-rkomorn]

Datadog Report

Branch report: munir/test-pymongo-4-9
Commit report: 4cfdc70
Test service: dd-trace-py

✅ 0 Failed, 592 Passed, 694 Skipped, 18m 30.4s Total duration (18m 47.28s time saved)

[mabdinur]

Since pymongo is a popular library it would be nice if we could sneak this change in to the v2.13.0 release @erikayasuda 🙏

[erikayasuda]

Since pymongo is a popular library it would be nice if we could sneak this change in to the v2.13.0 release @erikayasuda 🙏

I can get back to you on this 🫡

[erikayasuda]

Just a nit, otherwise LGTM