Skip to content

[Android] Re-enable System.Net.Http.Functional.Tests#125481

Merged
simonrozsival merged 8 commits into
dotnet:mainfrom
simonrozsival:fix/124526-android-no-peer-certificates
May 15, 2026
Merged

[Android] Re-enable System.Net.Http.Functional.Tests#125481
simonrozsival merged 8 commits into
dotnet:mainfrom
simonrozsival:fix/124526-android-no-peer-certificates

Conversation

@simonrozsival

@simonrozsival simonrozsival commented Mar 12, 2026

Copy link
Copy Markdown
Member

Fixes #124526

Re-enables System.Net.Http.Functional.Tests on Android CoreCLR by removing the ActiveIssue attribute added in #125171.

The tests were disabled because they were timing out on Android Helix queues. After investigation, the tests now pass on both Android Helix queues (x64-linux and arm64-win) with 0 failures and within the default 30min timeout.

While the tests pass, they run slower than they used to. The root cause is not yet identified and should be investigated separately.

…meout

- Remove ActiveIssue disable for dotnet#124526 on Android CoreCLR
- Increase Android Helix work item timeout from 30min to 60min to
  accommodate the ~1.5-1.6x slowdown from emulator 36.4.9

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings March 12, 2026 09:10
@simonrozsival

Copy link
Copy Markdown
Member Author

/azp run runtime-extra-platforms

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@simonrozsival simonrozsival added NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) NO-REVIEW Experimental/testing PR, do NOT review it labels Mar 12, 2026

This comment was marked as outdated.

@simonrozsival

Copy link
Copy Markdown
Member Author

/azp run runtime-extra-platforms

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

…crease

The 2h timeout was set to diagnose dotnet#124526, but CI shows work items
complete in ~11 min. Reverting to the original 30min to confirm the
original timeout issue is no longer reproducible.

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

Copy link
Copy Markdown
Member Author

/azp run runtime-extra-platforms

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@simonrozsival simonrozsival removed NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) NO-REVIEW Experimental/testing PR, do NOT review it labels Mar 17, 2026
@simonrozsival simonrozsival marked this pull request as ready for review March 17, 2026 12:01
Copilot AI review requested due to automatic review settings March 17, 2026 12:01

This comment was marked as resolved.

@simonrozsival

This comment was marked as resolved.

@azure-pipelines

This comment was marked as resolved.

@vitek-karas vitek-karas left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The PR seems to be "clean" in that the relevant Android tests are passing. I'd go with merging on red, the iOS failures are all infrastructure problems - the tests has been disabled recently #125771.

@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.

Copilot AI review requested due to automatic review settings April 13, 2026 08:49
@simonrozsival

Copy link
Copy Markdown
Member Author

/azp run runtime-extra-platforms

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@simonrozsival simonrozsival enabled auto-merge (squash) May 13, 2026 09:31
@simonrozsival simonrozsival disabled auto-merge May 15, 2026 08:18
@simonrozsival simonrozsival merged commit 9b73bbe into dotnet:main May 15, 2026
91 of 93 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 14, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[android][clr] No peer certificates when executing System.Net.Http.Functional.Tests on Android emulator

5 participants