Skip to content

[ci-scan] Skip ActivityTracking tests on Android Mono (refs #129088)#129094

Merged
BrzVlad merged 3 commits into
mainfrom
ci-scan/disable-activitytracking-android-729b972033d42be5
Jun 10, 2026
Merged

[ci-scan] Skip ActivityTracking tests on Android Mono (refs #129088)#129094
BrzVlad merged 3 commits into
mainfrom
ci-scan/disable-activitytracking-android-729b972033d42be5

Conversation

@github-actions

@github-actions github-actions Bot commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

Reasoning

The BasicEventSourceTests.ActivityTracking tests consistently fail on Android when running on the Mono runtime because the Activity ID remains as all-zeros Guid after EventSource Start/Stop operations. On Android with Mono, the threading/activity context does not propagate as expected, causing Assert.NotEqual(Guid.Empty, ...) to fail.

The fix narrows the [ActiveIssue] condition to Android + Mono only by using PlatformDetection.IsAndroid together with PlatformDetection.IsMonoRuntime. All four affected tests share the same root cause, and the updated skip scope now matches the failing legs more precisely than the original Android-wide condition.

Linked KBE: #129088

Match verification (from Step 4.8):

  1. Same test/family: yes — BasicEventSourceTests.ActivityTracking.StartStopCreatesActivity and 3 sibling tests
  2. Same failure signature: yes — Assert.NotEqual() Failure: Values are equal with Expected Not 00000000-0000-0000-0000-000000000000
  3. Same OS/runtime combination: yes — Android on Mono
  4. Same architecture: yes — all affected Android Mono architectures

Impact on platforms

  • runtime-extra-platforms (def 154) / android-arm Release AllSubsets_Mono / Windows.11.Amd64.Android-armel.Open / exit code 1
  • runtime-extra-platforms (def 154) / android-x64 Release AllSubsets_Mono / Windows.11.Amd64.Android-x64.Open / exit code 1
  • runtime-extra-platforms (def 154) / android-arm64 Release AllSubsets_Mono / exit code 1
  • runtime-extra-platforms (def 154) / android-x86 Release AllSubsets_Mono / exit code 1

Errors log

[FAIL] BasicEventSourceTests.ActivityTracking.StartStopCreatesActivity
Assert.NotEqual() Failure: Values are equal
Expected: Not 00000000-0000-0000-0000-000000000000
Actual:       00000000-0000-0000-0000-000000000000
   at BasicEventSourceTests.ActivityTracking.StartStopCreatesActivity()

[FAIL] BasicEventSourceTests.ActivityTracking.ActivityFlowsAsync
[FAIL] BasicEventSourceTests.ActivityTracking.SetCurrentActivityIdBeforeEventFlowsAsync
[FAIL] BasicEventSourceTests.ActivityTracking.SetCurrentActivityIdAfterEventDoesNotFlowAsync

First build it occurred

Linked issue

#129088


Filed by ci-failure-scan, which scans dnceng-public outer-loop pipelines on main and converts stable failures into KBEs and test-disable PRs. Comment here or on the workflow file to suggest changes; ci-failure-scan-feedback reads in-scope feedback daily and opens (or updates) a PR with prompt edits.

> [!NOTE]
>

> Generated by CI Outer-Loop Failure Scanner · ● 38.1M ·

The BasicEventSourceTests.ActivityTracking tests consistently fail on
Android because the Activity ID remains zero after EventSource Start/Stop
operations. This disables the four affected tests on Android with a reference
to the tracking KBE issue.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Tagging subscribers to 'arch-android': @vitek-karas, @simonrozsival, @steveisok, @akoeplinger
See info in area-owners.md if you want to be subscribed.

@github-actions github-actions Bot mentioned this pull request Jun 8, 2026
Co-authored-by: BrzVlad <4720621+BrzVlad@users.noreply.github.com>
Copilot AI changed the title [ci-scan] Skip ActivityTracking tests on Android (refs #129088) [ci-scan] Skip ActivityTracking tests on Android Mono (refs #129088) Jun 9, 2026
Copilot AI requested a review from BrzVlad June 9, 2026 07:16
@BrzVlad BrzVlad marked this pull request as ready for review June 9, 2026 09:26
Copilot AI review requested due to automatic review settings June 9, 2026 09:26

Copilot AI 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.

Pull request overview

This PR updates System.Diagnostics.Tracing ActivityTracking unit tests to be skipped only on Android when running on the Mono runtime, by adding a new [ActiveIssue] condition that requires both PlatformDetection.IsAndroid and PlatformDetection.IsMonoRuntime.

Changes:

  • Add a new [ActiveIssue] attribute to the four failing BasicEventSourceTests.ActivityTracking tests.
  • Gate the skip specifically to the IsAndroid && IsMonoRuntime combination (multiple conditions within one [ActiveIssue] are combined with && in the test wrapper generator).

@BrzVlad BrzVlad merged commit 9a975e6 into main Jun 10, 2026
88 checks passed
@BrzVlad BrzVlad deleted the ci-scan/disable-activitytracking-android-729b972033d42be5 branch June 10, 2026 04:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants