Skip to content

Conversation

@JohnMcPMS
Copy link
Member

@JohnMcPMS JohnMcPMS commented Jan 13, 2026

Change

Add more data to try and get to the bottom of the test failures in the pipeline.

Most importantly adds an extensive tracing that wraps the failing task to hopefully get more details.

Also adds the PID to the startup logging and a message to indicate that the server is exiting due to its event being signaled. These won't actually be used in the pipeline right now, as the failure is happening the in GA PS/COM interaction. But they are helpful for diagnosing issues anyway.

Microsoft Reviewers: Open in CodeFlow

@JohnMcPMS JohnMcPMS requested a review from a team as a code owner January 13, 2026 19:37
@github-actions

This comment was marked as outdated.

florelis
florelis previously approved these changes Jan 13, 2026
@JohnMcPMS
Copy link
Member Author

Examining the trace shows that OS deployment is requesting that the package be terminated after we have started trying to use it. This ends up killing the COM server out from under the PowerShell usage. Added the provider for deployment to understand why it is requesting that since our Repair should no longer be running at that point.

@JohnMcPMS
Copy link
Member Author

The tracing shows that the issue is coming from AppReadiness registering the package due to our provisioning call. Moving to install only for the current user removes that asynchronous behavior (there does not appear to be a good reason for the provisioning in the original PR). I'm not currently aware of any method for us to wait for the AppReadiness registration to complete so that we can prevent the caller from proceeding to use the package.

Copy link
Member

@florelis florelis left a comment

Choose a reason for hiding this comment

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

Nit: Update title to say you're also fixing the failures?

@JohnMcPMS JohnMcPMS changed the title More data to diagnose pipeline failures Diagnostics and fix for pipeline test failures Jan 15, 2026
@JohnMcPMS JohnMcPMS merged commit f19d806 into microsoft:master Jan 15, 2026
9 checks passed
@JohnMcPMS JohnMcPMS deleted the pipeline-data branch January 15, 2026 19:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants