Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

[ci-scan] Skip DnsGetHost*_LocalhostSubdomain_RespectsAddressFamily on Android (refs #128253) - #128353

#128353
Closed
github-actions[bot] wants to merge 2 commits into
maindotnet/runtime:mainfrom
ci-scan/disable-dns-localhost-subdomain-android-128253-c5b4bb84f6bda72edotnet/runtime:ci-scan/disable-dns-localhost-subdomain-android-128253-c5b4bb84f6bda72eCopy head branch name to clipboard
Closed

[ci-scan] Skip DnsGetHost*_LocalhostSubdomain_RespectsAddressFamily on Android (refs #128253)#128353
github-actions[bot] wants to merge 2 commits into
maindotnet/runtime:mainfrom
ci-scan/disable-dns-localhost-subdomain-android-128253-c5b4bb84f6bda72edotnet/runtime:ci-scan/disable-dns-localhost-subdomain-android-128253-c5b4bb84f6bda72eCopy head branch name to clipboard

Conversation

@github-actions

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

Copy link
Copy Markdown
Contributor

Reasoning

The tests DnsGetHostEntry_LocalhostSubdomain_RespectsAddressFamily and DnsGetHostAddresses_LocalhostSubdomain_RespectsAddressFamily fail on Android because Dns.GetHostEntry/GetHostAddresses returns IPv6 addresses when only IPv4 (InterNetwork) is requested. The Android resolver does not filter by address family for localhost subdomains. The [ActiveIssue] attribute is the narrowest correct fix — Android-only, matching the observed failure scope.

Linked KBE: #128253

Match verification (from Step 4.8):

  1. Same test/family: yes — GetHostEntryTest.DnsGetHostEntry_LocalhostSubdomain_RespectsAddressFamily and sibling GetHostAddresses variant
  2. Same failure signature: yes — Assert.All() Failure with Expected: InterNetwork, Actual: InterNetworkV6
  3. Same OS: yes — Android
  4. Same architecture: yes — arm64

Impact on platforms

  • runtime-extra-platforms / android-arm64 Release AllSubsets_Mono / 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: 5 out of 7 items in the collection did not pass.
[1]: Item:  fe80::244c:96ff:fe41:5910%47
     Error: Assert.Equal() Failure: Values differ
            Expected: InterNetwork
            Actual:   InterNetworkV6

[FAIL] System.Net.NameResolution.Tests.GetHostAddressesTest.DnsGetHostAddresses_LocalhostSubdomain_RespectsAddressFamily(addressFamily: InterNetwork)
Assert.All() Failure: 5 out of 7 items in the collection did not pass.

First build it occurred

Linked issue

#128253

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 · ● 30M · ◷

@github-actions
Skip DnsGetHost*_LocalhostSubdomain_RespectsAddressFamily on Android …
5afa408 
…(refs #128253)

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

dotnet-policy-service Bot commented May 19, 2026

Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @karelz, @dotnet/ncl
See info in area-owners.md if you want to be subscribed.

This was referenced May 19, 2026
Open
Open
Open
Closed
@kotlarmilos
kotlarmilos marked this pull request as ready for review July 28, 2026 14:28
Copilot AI review requested due to automatic review settings July 28, 2026 14:28
@kotlarmilos

kotlarmilos commented Jul 28, 2026

Copy link
Copy Markdown
Member

@matouskozak Let's check if this PR is still needed

@kotlarmilos kotlarmilos added this to the 12.0.0 milestone Jul 28, 2026
@azure-pipelines

azure-pipelines Bot commented Jul 28, 2026

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

Copilot AI reviewed Jul 28, 2026
View reviewed changes

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 addresses persistent Android CI failures in System.Net.NameResolution functional tests by marking the affected localhost-subdomain “respects AddressFamily” tests as skipped on Android.

Changes:

  • Add an [ActiveIssue] skip on Android for DnsGetHostEntry_LocalhostSubdomain_RespectsAddressFamily.
  • Add an [ActiveIssue] skip on Android for DnsGetHostAddresses_LocalhostSubdomain_RespectsAddressFamily.
Show a summary per file
File Description
src/libraries/System.Net.NameResolution/tests/FunctionalTests/GetHostEntryTest.cs Adds an Android [ActiveIssue] skip attribute to the AddressFamily-respecting localhost-subdomain GetHostEntry test.
src/libraries/System.Net.NameResolution/tests/FunctionalTests/GetHostAddressesTest.cs Adds an Android [ActiveIssue] skip attribute to the AddressFamily-respecting localhost-subdomain GetHostAddresses test.

Copilot's findings

  • Files reviewed: 2/2 changed files
  • Comments generated: 2

Comment thread src/libraries/System.Net.NameResolution/tests/FunctionalTests/GetHostEntryTest.cs
Comment on lines 415 to 419
[Theory]
[InlineData(AddressFamily.InterNetwork)]
[InlineData(AddressFamily.InterNetworkV6)]
[ActiveIssue("https://github.com/dotnet/runtime/issues/128253", TestPlatforms.Android)]
public async Task DnsGetHostEntry_LocalhostSubdomain_RespectsAddressFamily(AddressFamily addressFamily)
Comment thread src/libraries/System.Net.NameResolution/tests/FunctionalTests/GetHostAddressesTest.cs
Comment on lines 250 to 254
[Theory]
[InlineData(AddressFamily.InterNetwork)]
[InlineData(AddressFamily.InterNetworkV6)]
[ActiveIssue("https://github.com/dotnet/runtime/issues/128253", TestPlatforms.Android)]
public async Task DnsGetHostAddresses_LocalhostSubdomain_RespectsAddressFamily(AddressFamily addressFamily)
@matouskozak

matouskozak commented Jul 28, 2026

Copy link
Copy Markdown
Member

@matouskozak Let's check if this PR is still needed

I don't see it failing on the latest extra-platforms run https://dev.azure.com/dnceng-public/public/_build/results?buildId=1529233&view=results

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

Reviewers

Copilot code review Copilot Copilot left review comments
@rzikm rzikm rzikm approved these changes

Assignees

No one assigned

Labels

agentic-workflows area-System.Net

Projects

None yet

Milestone

12.0.0

Development

Successfully merging this pull request may close these issues.

4 participants

@kotlarmilos @matouskozak @rzikm
Morty Proxy This is a proxified and sanitized view of the page, visit original site.