Skip to content

test(e2e/windows): ignore TiWorker.exe crash dumps#51140

Open
clarkb7 wants to merge 1 commit into
mainfrom
branden.clark/ignore-tiworker-crash
Open

test(e2e/windows): ignore TiWorker.exe crash dumps#51140
clarkb7 wants to merge 1 commit into
mainfrom
branden.clark/ignore-tiworker-crash

Conversation

@clarkb7
Copy link
Copy Markdown
Contributor

@clarkb7 clarkb7 commented May 20, 2026

What does this PR do?

Adds TiWorker.exe to DefaultIgnoredCrashDumpImages so Windows Modules Installer Worker crashes are still collected as artifacts but do not fail Windows E2E tests.

Motivation

Ref: WINA-2712. TiWorker crashes were failing TestAgentMSIInstallsDotnetLibrary. Analysis of the dump from job 1698945354 with cdb:

  • Faulting instruction: MPCLIENT!tlgEnableCallback in an already-unloaded MPCLIENT.DLL (Microsoft Defender).
  • Stack is entirely ntdll!Etwp* notification thread frames.
  • !analyze -v root cause: "An ETW client DLL was unloaded but failed to unregister the ETW provider GUID prior to unload. This is a problem with the provider."

This is a Microsoft Defender ETW unload race, unrelated to the Datadog agent.

TiWorker.exe (Windows Modules Installer Worker) crashes have been observed
in dotnet library installer E2E tests. Analysis of the dump shows the AV
occurs in MPCLIENT!tlgEnableCallback after Microsoft Defender's MPCLIENT.DLL
was unloaded without unregistering its ETW provider — a Microsoft issue
unrelated to the Datadog agent.

Ref: WINA-2712
@clarkb7 clarkb7 added changelog/no-changelog No changelog entry needed qa/no-code-change No code change in Agent code requiring validation team/windows-products labels May 20, 2026
@github-actions
Copy link
Copy Markdown
Contributor

@codex review

@dd-octo-sts dd-octo-sts Bot added the internal Identify a non-fork PR label May 20, 2026
@github-actions github-actions Bot added the short review PR is simple enough to be reviewed quickly label May 20, 2026
@chatgpt-codex-connector
Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Bravo.

ℹ️ About Codex in GitHub

Codex has been enabled to automatically 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 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

@datadog-prod-us1-6
Copy link
Copy Markdown
Contributor

datadog-prod-us1-6 Bot commented May 20, 2026

Pipelines

Fix all issues with BitsAI

⚠️ Warnings

🚦 2 Pipeline jobs failed

DataDog/datadog-agent | oracle: [21.3.0-xe]   View in Datadog   GitLab

🔄 Retry job. This looks flaky and may succeed on retry. Failed to ping oracle instance: ORA-12514: TNS:listener does not currently know of service requested in connect descriptor.

DataDog/datadog-agent | unit_tests_notify   View in Datadog   GitLab

🛟 This job is unlikely to succeed on retry. Please review your pipeline configuration. Command 'ls junit-tests_*.tgz' failed: No such file or directory.

Useful? React with 👍 / 👎

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 09e54c9 | Docs | Datadog PR Page | Give us feedback!

@dd-octo-sts
Copy link
Copy Markdown
Contributor

dd-octo-sts Bot commented May 20, 2026

Files inventory check summary

File checks results against ancestor b661dc54:

Results for datadog-agent_7.81.0~devel.git.19.09e54c9.pipeline.114380653-1_amd64.deb:

No change detected

@clarkb7 clarkb7 marked this pull request as ready for review May 21, 2026 13:16
@clarkb7 clarkb7 requested a review from a team as a code owner May 21, 2026 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/no-changelog No changelog entry needed internal Identify a non-fork PR qa/no-code-change No code change in Agent code requiring validation short review PR is simple enough to be reviewed quickly team/windows-products

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant