Skip to content

[release/11.0] Update dependencies from dotnet/arcade-services - #17352

Open
dotnet-maestro[bot] wants to merge 1 commit into
release/11.0from
darc-release/11.0-e154da90-d95c-43d4-8976-757df2651806
Open

[release/11.0] Update dependencies from dotnet/arcade-services#17352
dotnet-maestro[bot] wants to merge 1 commit into
release/11.0from
darc-release/11.0-e154da90-d95c-43d4-8976-757df2651806

Conversation

@dotnet-maestro

Copy link
Copy Markdown
Contributor

This pull request updates the following dependencies

From https://github.com/dotnet/arcade-services

…ild 20260813.1

On relative base path root
Microsoft.DotNet.DarcLib , Microsoft.DotNet.ProductConstructionService.Client From Version 1.1.0-beta.26374.1 -> To Version 1.1.0-beta.26413.1
@github-actions

Copy link
Copy Markdown
Contributor

❌ Build Failure Analysis

Both build legs (Linux/Debug and Windows/Release) fail with the same two CS1503 compiler errors in PublishBuildToMaestro.cs.

Root Cause

This PR bumps Microsoft.DotNet.ProductConstructionService.Client from 1.1.0-beta.26374.11.1.0-beta.26413.1.
The new package version changed the signature of BuildsOperations.GetBuildAsync — the second positional parameter is no longer a CancellationToken.

Two call sites in PublishBuildToMaestro.cs pass cancellationToken as the second argument, which no longer matches any overload:

File Line Error
src/Microsoft.DotNet.Build.Tasks.Feed/src/PublishBuildToMaestro.cs 362 CS1503 — cannot convert CancellationTokenbool?
src/Microsoft.DotNet.Build.Tasks.Feed/src/PublishBuildToMaestro.cs 380 CS1503 — cannot convert CancellationTokenbool?

Suggested Fix

Remove cancellationToken from both GetBuildAsync call sites (the existing single-argument calls in PublishArtifactsInManifest.cs, PublishArtifactsInManifestV3.cs, and PublishArtifactsInManifestV4.cs confirm the method no longer accepts a CancellationToken positional parameter).

// Line 362 — before
producingBuild = await client.Builds.GetBuildAsync(asset.BuildId, cancellationToken);
// Line 362 — after
producingBuild = await client.Builds.GetBuildAsync(asset.BuildId);

// Line 380 — before
var build = await client.Builds.GetBuildAsync(buildId.Value, cancellationToken);
// Line 380 — after
var build = await client.Builds.GetBuildAsync(buildId.Value);

If the updated API exposes cancellation via a named parameter, pass cancellationToken there as well.

🤖 Automated content by GitHub Copilot. Generated by the Build Failure Analysis workflow.{ai_credits_suffix} · [◷]( · )

@github-actions github-actions Bot 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.

🤖 Automated content by GitHub Copilot. Generated by the Build Failure Analysis workflow.{ai_credits_suffix} ·

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants