chore(profiling): also wrap faulthandler.disable

[KowalskiThomas]

Description

As reported by the Codex Security Scan, follow-up to #17998 to also support disabling Python's fault handler.

[cit-pr-commenter-54b7da]

Codeowners resolved as

ddtrace/profiling/_faulthandler.py                                      @DataDog/profiling-python
tests/profiling/test_faulthandler.py                                    @DataDog/profiling-python

[datadog-official]

✨ Fix all issues with BitsAI or with Cursor

⚠️ Warnings

❄️ 1 New flaky test detected

test_faulthandler_disable_concurrent_with_enable from test_faulthandler.py   (Fix with Cursor)

Expected status 0, got 1.
=== Captured STDOUT ===
=== End of captured STDOUT ===
=== Captured STDERR ===
Traceback (most recent call last):
  File "tests/profiling/test_faulthandler.py", line 584, in <module>
    assert not still_alive, f"{len(still_alive)} thread(s) deadlocked"
AssertionError: 1 thread(s) deadlocked
=== End of captured STDERR ===

New test introduced in this PR is flaky.

View in Flaky Test Management

ℹ️ Info

No other issues found (see more)

🧪 All tests passed

Useful? React with 👍 / 👎

_{This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 581e792 | Docs | Datadog PR Page | Give us feedback!}

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: d79b83b67b

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[vlad-scherbich]

@KowalskiThomas are the two Codex's comments valid?

[KowalskiThomas]

@vlad-scherbich Yeah, valid and I just pushed the necessary fixes!