Skip to content

Remove obsolete Windows MSI onboarding telemetry - #57927

Open
clarkb7 wants to merge 3 commits into
mainfrom
branden.clark/remove-old-installer-telemetry
Open

clarkb7 wants to merge 3 commits into
mainfrom
branden.clark/remove-old-installer-telemetry

Conversation

@clarkb7

@clarkb7 clarkb7 commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

What does this PR do?

Remove the Windows Agent MSI success and rollback callbacks that sent apm-onboarding-event telemetry, along with their exported entry points, dedicated sender, and obsolete unit tests.

Motivation

The intake was deactivated over a year ago (WINA-2832, backend removal), so these installer callbacks no longer serve their original purpose.

Describe how you validated your changes

existing E2E tests validate MSI behavior

Additional Notes

Removal from agent-linux-install-script remains separate follow-up work. The PowerShell installer and its build/signing/deployment jobs were already removed in commit 5d48a22a650.

@clarkb7
clarkb7 requested a review from a team as a code owner October 9, 2026 16:25
@clarkb7
clarkb7 requested a review from mrafi97 October 9, 2026 16:25
@dd-octo-sts dd-octo-sts Bot added internal Identify a non-fork PR team/windows-products labels Oct 9, 2026
@github-actions github-actions Bot added the medium review PR review might take time label Oct 9, 2026

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AI review by Codex (OpenAI) - workflow run

The patch is correct. The telemetry sender, exported callbacks, WiX actions, and obsolete tests are removed consistently, with no remaining references found. Shared HTTP support and installation actions remain intact. Windows MSI build and runtime tests were not independently run during this review.

@datadog-prod-us1-4

datadog-prod-us1-4 Bot commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

🎯 Code Coverage (details)
• Patch Coverage: 100.00%
• Overall Coverage: 62.80% (+4.20%)

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 69820b0 | Docs | Give us feedback!

@clarkb7 clarkb7 added the qa/done QA done before merge and regressions are covered by tests label Oct 9, 2026
@dd-octo-sts

dd-octo-sts Bot commented Oct 9, 2026

Copy link
Copy Markdown
Contributor

Files inventory check summary

File checks results against ancestor c2a701c6:

Results for datadog-agent_7.86.0~devel.git.326.69820b0.pipeline.143783696-1_amd64.deb:

No change detected

Results for datadog-iot-agent_7.86.0~devel.git.326.69820b0.pipeline.143783696-1_amd64.deb:

No change detected

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal Identify a non-fork PR medium review PR review might take time qa/done QA done before merge and regressions are covered by tests team/windows-products

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant