Skip to content

[ci-scan] Skip LocalhostSubdomain_RespectsAddressFamily tests on Android (refs #128253, #128371)#128425

Draft
github-actions[bot] wants to merge 2 commits into
mainfrom
ci-scan/skip-localhost-subdomain-android-256abab6da5f30b6
Draft

[ci-scan] Skip LocalhostSubdomain_RespectsAddressFamily tests on Android (refs #128253, #128371)#128425
github-actions[bot] wants to merge 2 commits into
mainfrom
ci-scan/skip-localhost-subdomain-android-256abab6da5f30b6

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot commented May 21, 2026

Reasoning

The tests DnsGetHostEntry_LocalhostSubdomain_RespectsAddressFamily and DnsGetHostAddresses_LocalhostSubdomain_RespectsAddressFamily fail on Android because the Android resolver returns IPv6 addresses when AddressFamily.InterNetwork (IPv4 only) is requested. The [ActiveIssue] attribute is the correct fix to skip these tests on Android only, matching the observed failure scope.

Linked KBE: #128253
Linked KBE: #128371

Match verification (from Step 4.8):

  1. Same test/family: yes — DnsGetHostEntry_LocalhostSubdomain_RespectsAddressFamily and DnsGetHostAddresses_LocalhostSubdomain_RespectsAddressFamily
  2. Same failure signature: yes — Assert.All() Failure with Expected: InterNetwork / Actual: InterNetworkV6
  3. Same OS: yes — Android
  4. Same architecture: yes — arm64 (android-arm64 Release NativeAOT on Windows.11.Amd64.Android.Open)

Impact on platforms

  • runtime-extra-platforms / android-arm64 Release NativeAOT / Windows.11.Amd64.Android.Open — exit code 1
  • runtime-extra-platforms / android-arm64 Release AllSubsets_CoreCLR / Windows.11.Amd64.Android.Open — exit code 1

Errors log

[FAIL] System.Net.NameResolution.Tests.GetHostEntryTest.DnsGetHostEntry_LocalhostSubdomain_RespectsAddressFamily(addressFamily: InterNetwork)
Assert.All() Failure: 10 out of 12 items in the collection did not pass.
[1]:  Item:  fe80::f051:a9ff:fe8c:708a%47
      Error: Assert.Equal() Failure: Values differ
             Expected: InterNetwork
             Actual:   InterNetworkV6

First build it occurred

Linked issue

#128253
#128371

Note

🔒 Integrity filter blocked 1 item

The following item was blocked because it doesn't meet the GitHub integrity level.

  • #125825 search_issues: has lower integrity than agent requires. The agent cannot read data with integrity below "approved".

To allow these resources, lower min-integrity in your GitHub frontmatter:

tools:
  github:
    min-integrity: approved  # merged | approved | unapproved | none

Generated by CI Outer-Loop Failure Scanner · ● 33.4M · ◷
@github-actions
Skip LocalhostSubdomain_RespectsAddressFamily tests on Android
4f2b313 
Android resolver returns IPv6 addresses when InterNetwork is requested,
causing Assert.All() failures in these tests.

Refs #128253, #128371

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

dotnet-policy-service Bot commented May 21, 2026

Tagging subscribers to this area: @karelz, @dotnet/ncl
See info in area-owners.md if you want to be subscribed.
This was referenced May 21, 2026
Open
Open
@kotlarmilos kotlarmilos mentioned this pull request May 21, 2026
Closed
kotlarmilos added a commit that referenced this pull request May 21, 2026
@kotlarmilos
ci-failure-scan: dedup stem fallback + reject weak array signatures
a52ac4f 
Two fixes driven by the first scanner run on this branch:

- Step 4.4 (existing test-disable PR dedup): also search the test-name
  stem after stripping verb prefixes and platform suffixes. PR titles
  often abbreviate (e.g. DnsGetHostEntry_X -> X). Without the fallback,
  the scanner filed #128442 as a duplicate of #128425.

- KBE check #6 and bad-vs-good table: reject array-form signatures whose
  second element is a generic xunit assertion stem ('Assert.Equal()
  Failure: Values differ', 'Assert.True() Failure', etc). Require the
  unique 'Expected:'/'Actual:' value lines or the actual exception type
  + message. #128444 (now closed) emitted such a weak signature; Build
  Analysis would have mismatched it against unrelated failures.

Lock file: re-applies the manual pat_pool patch.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment