Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Offline builds fail trying to access online feeds #4107

Closed
MichaelSimons opened this issue Feb 8, 2024 · 9 comments
Closed

Offline builds fail trying to access online feeds #4107

MichaelSimons opened this issue Feb 8, 2024 · 9 comments
Assignees
Labels
area-infra Source-build infrastructure and reporting ops-monitor Issues created/handled by the source build monitor role

Comments

@MichaelSimons
Copy link
Member

MichaelSimons commented Feb 8, 2024

  /vmr/artifacts/source-built-sdks/Microsoft.DotNet.Arcade.Sdk/tools/Tools.proj : error NU1301: Unable to load the service index for source https://pkgs.dev.azure.com/dnceng/public/_packaging/dotnet-public/nuget/v3/index.json.

Failing build (internal microsoft link) - https://dev.azure.com/dnceng/internal/_build/results?buildId=2373582&view=logs&j=a361eaf5-91c0-5b8d-1f36-c97d0865c360&t=8198a24c-af12-5a92-d480-67a88a3b425d

@ellahathaway ellahathaway added the ops-monitor Issues created/handled by the source build monitor role label Feb 8, 2024
@mthalman
Copy link
Member

mthalman commented Feb 8, 2024

I think this is what was fixed by dotnet/installer#18565. The full VMR build hasn't run since that came in. The lite pipeline is passing.

@ellahathaway
Copy link
Member

For source build monitoring - the latest run of source-build-sdk-diff-tests is failing in all legs due to this issue; latest run cannot find any associated artifacts from the full build.

@MichaelSimons
Copy link
Member Author

I think this is what was fixed by dotnet/installer#18565. The full VMR build hasn't run since that came in. The lite pipeline is passing.

I triggered a new run this morning after those changes. - I updated the linked build.

@MichaelSimons
Copy link
Member Author

@jkoritzinsky, can you take a look?

@MichaelSimons MichaelSimons moved this from Backlog to 9.0 Preview 2 in .NET Source Build Feb 8, 2024
@MichaelSimons MichaelSimons added area-infra Source-build infrastructure and reporting and removed untriaged area-ci-testing labels Feb 8, 2024
@jkoritzinsky
Copy link
Member

Looks like it's failing to find the Microsoft.Dotnet.SourceBuild.Tasks library locally. I'll try to figure out how this was working beforehand.

@jkoritzinsky
Copy link
Member

@MichaelSimons looks like the RestoreConfigFile isn't being passed down. The fix for this is in dotnet/runtime#97778, which is newer than the dotnet/runtime currently in the VMR (https://github.com/dotnet/runtime/commits/d40c654c274fe4f4afe66328f0599130f3eb2ea6).

Once this change flows through (dotnet/installer#18457), then this should be fixed.

@MichaelSimons
Copy link
Member Author

Interesting, I thought we were already carrying the fix for dotnet/runtime#97778 as a patch - https://github.com/dotnet/installer/blob/main/src/SourceBuild/patches/runtime/0001-Enable-use-of-RestoreConfigFile-property-in-source-b.patch

When I look at the tip of the VMR, This doesn't appear to be applied.

@MichaelSimons
Copy link
Member Author

MichaelSimons commented Feb 8, 2024

That patch somehow didn't get re-applied in dotnet/dotnet@02e3703 and was dropped. @premun, Can you explain what happened here?

@MichaelSimons
Copy link
Member Author

This issue was resolved with the latest sdk->installer dependency flow which removed all runtime patches.

The underlying issue with the sync not applying patches is tracked with dotnet/arcade-services#3286

@github-project-automation github-project-automation bot moved this from 9.0 Preview 2 to Done in .NET Source Build Feb 13, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
area-infra Source-build infrastructure and reporting ops-monitor Issues created/handled by the source build monitor role
Projects
Archived in project
Development

No branches or pull requests

4 participants