Skip to content

Drain pending telemetry before CLI shutdown - #55983

Draft
nagilson with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-build-telemetry-timeout
Draft

Drain pending telemetry before CLI shutdown#55983
nagilson with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-build-telemetry-timeout

Conversation

Copilot AI commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

TelemetryHostMatrixTests.BuildTelemetryIsExportedForEachHost intermittently timed out because command/finish could remain queued while shutdown stopped and exported the main activity.

  • Fix shutdown ordering
    • Wait for queued telemetry events before stopping the activity and flushing providers.
    • Prevent command/finish from being dropped during process teardown.
TelemetryClient.Instance?.WaitForPendingEvents();
s_mainActivity?.Stop();
TelemetryClient.FlushProviders();

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
3 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

Co-authored-by: nagilson <23152278+nagilson@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix timeout in telemetry host matrix tests Drain pending telemetry before CLI shutdown Aug 28, 2026
Copilot AI requested a review from nagilson August 28, 2026 06:39
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.

[AI discovered CI] TelemetryHostMatrixTests.BuildTelemetryIsExportedForEachHost times out waiting for build telemetry

2 participants