Skip to content

[release/10.0.1xx] Complete Maven feed backport - #12469

Merged
jonathanpeppers merged 5 commits into
release/10.0.1xxfrom
jonathanpeppers-backport-public-maven-feeds
Aug 24, 2026
Merged

jonathanpeppers merged 5 commits into
release/10.0.1xxfrom
jonathanpeppers-backport-public-maven-feeds

Conversation

@jonathanpeppers

Copy link
Copy Markdown
Member

Summary

  • backport 1c4d41c09eec so direct Maven mirroring fails when any requested coordinate produces no payloads
  • update external/Java.Interop from 33992194b9373e9322244612c94ed9941b9bc2fd to [release/10.0.1xx] Backport Maven hardening and feed routing java-interop#1498's single release backport commit 7c110bfad3ee0a87f303609426167722ccb4ba77
  • include the release-applicable Java.Interop Maven test and java-source-utils routing that could not be carried by the main-branch subtree patch
  • seed the release JavaParser 3.18.0 dependency graph into dotnet-public-maven for anonymous CI resolution

Backport mapping

Main change Release equivalent
ce636392a229 / #11711, 3c62389651c2 / #12055, Android portions of 466da4a84dc2 / #12199, and adffb38fd731 / #12368 659cd98e7fb9 / #12397
8df1e87799f1 / #12198, 7ace3137a4fa / #12336, c09c89aa1f2e / #12338, 13727e665929 / #12335, and d57b8c42f131 / #12367 352831edcba6 / #12396
approved .NET tool feed portion of fa00357649bb / #11701 dc9a8524138e / #12420
1c4d41c09eec / #12412 24252012aaf6 in this PR
Java.Interop hardening plus release-applicable routing from 466da4a84dc2 dotnet/java-interop#1498, pinned here at 7c110bfad3ee0a87f303609426167722ccb4ba77

The later Dependabot-only Google Maven exposure and unrelated Gradle/dependency version churn are intentionally excluded. The final Java.Interop pin supersedes the earlier hardening-only candidate without amending or force-pushing commits.

Validation

  • Java.Interop.Tools.Maven tests: 106 passed, 0 failed
  • CI-mode java-source-utils jar build resolved through dotnet-public-maven and succeeded after the mirror helper seeded the full transitive graph
  • mirror-dependencies.ps1 parsed successfully with the PowerShell parser
  • Java.Interop final SHA is exactly one commit on top of the release pin
  • git diff --check origin/release/10.0.1xx..HEAD

jonathanpeppers and others added 3 commits August 20, 2026 15:57
## Summary

- return explicit success and failure counts from `Invoke-Mirror`
- evaluate direct Maven mirroring per requested coordinate
- fail `-MavenArtifact` when any coordinate mirrors no payloads

Addresses the `eng/gradle/mirror-dependencies.ps1` feedback from #12397 without modifying the release backport.

## Validation

- parsed `mirror-dependencies.ps1` with the PowerShell parser
- verified a coordinate succeeds when at least one payload returns HTTP 200
- verified all-failed payloads exit with code 1
- verified multiple coordinates exit with code 1 when any coordinate has no successful payload
- `git diff --check`

Co-authored-by: Copilot App <[email protected]>
(cherry picked from commit 1c4d41c)
Pin dotnet/java-interop#1495's single release backport commit, which validates Maven coordinates and keeps resolved cache paths contained.

Co-authored-by: Copilot App <[email protected]>
Replace the superseded hardening-only pin with dotnet/java-interop#1498's single release commit, adding approved-feed routing for Java.Interop Maven tests and java-source-utils.

Co-authored-by: Copilot App <[email protected]>
Copilot AI lite review requested due to automatic review settings August 20, 2026 21:12
@jonathanpeppers

Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
1 pipeline(s) were filtered out due to trigger conditions.

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

Completes the release-branch Maven feed backport, including mirror completeness validation and related Java.Interop updates.

Changes:

  • Tracks per-coordinate mirror success and failure results.
  • Fails direct mirroring when requested coordinates produce no payloads.
  • Preserves Gradle retry handling with structured results.
  • Updates the Java.Interop release pin and Maven dependency graph.

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

Backport the Android-side parent POM coordinate fallback from 7da7db7 and cover POMs that inherit groupId and version.

Co-authored-by: Copilot App <[email protected]>

Copilot-Session: 30d6319e-30d5-4311-831c-58c4a0571403
@jonathanpeppers

Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
1 pipeline(s) were filtered out due to trigger conditions.

Update the gitlink to the current dotnet/java-interop release/10.0.1xx tip containing merged PR #1498.

Co-authored-by: Copilot App <[email protected]>

Copilot-Session: 30d6319e-30d5-4311-831c-58c4a0571403
@jonathanpeppers
jonathanpeppers marked this pull request as ready for review August 21, 2026 17:08
@jonathanpeppers

Copy link
Copy Markdown
Member Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
1 pipeline(s) were filtered out due to trigger conditions.

@jonathanpeppers jonathanpeppers added the ready-to-review This PR is ready to review/merge, I think any CI failures are just flaky (ignorable). label Aug 21, 2026
@jonathanpeppers

Copy link
Copy Markdown
Member Author

Failing tests unrelated, likely fixed in main already, merging.

System.AggregateException : AggregateException_ctor_DefaultMessage (Connection failure)
----> System.Net.Http.HttpRequestException : Connection failure
----> Java.Net.ConnectException : Failed to connect to httpbin.org/44.223.250.126:443

@jonathanpeppers
jonathanpeppers merged commit b115e81 into release/10.0.1xx Aug 24, 2026
1 of 2 checks passed
@jonathanpeppers
jonathanpeppers deleted the jonathanpeppers-backport-public-maven-feeds branch August 24, 2026 13:30
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 24, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

ready-to-review This PR is ready to review/merge, I think any CI failures are just flaky (ignorable).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants