-
Notifications
You must be signed in to change notification settings - Fork 132
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Error getting vulnerability data for msbuild repo #4634
Comments
I couldn't figure out the best area label to add to this issue. If you have write-permissions please help me learn by adding exactly one area label. |
I couldn't figure out the best area label to add to this issue. If you have write-permissions please help me learn by adding exactly one area label. |
/cc @zivkan as well |
Hard to tell on a first look. Seems like the source wasn't accessible. Did the SDK version change recently? |
Just as a test, can you run However, I don't see the Azure Pipelines |
It looks like this was a transient error. Builds are passing now. |
Reopening as the latest builds failed again due to the same issue. Now, it is affecting the 9.0 and 8.0 repos. |
Yes, it also showed up in dotnet/sdk#44174. |
/cc @rainersigwald |
In the 8.0 builds, it gets the following error which provides the extra context of 401 (Unauthorized):
This feed is referenced here: https://github.com/dotnet/dotnet/blob/77ace8473cd818b681c48046464d1cca112b945c/src/msbuild/NuGet.config#L21 @rainersigwald - Do you have context on this feed and know what's going on here? |
This also affects main. It showed up in the unified build run of this PR: dotnet/sdk#44192 |
Did an auth token expire or something? The feed https://pkgs.dev.azure.com/ms/BuildXL/_packaging/BuildXL/nuget/v3/index.json requires authentication, what is configuring authentication for feeds? |
That feed no longer exists and shouldn't be referenced in current branches. But the flow of that removal into SDK has been held up on unrelated problems (dotnet/sdk#44011). |
@rainersigwald - That flow is for main branch but this also applies to both 8.0 and 9.0. Has a similar feed removal been made for those branches? Because no such removal has reached the VMR yet. |
It may not have but I was just talking to @MichalPavlik about how we needed it. |
VMR patches created to workaround this: |
All the PRs mentioned in #4634 (comment) have been merged so this should be resolved now. |
The dotnet-source-build-lite pipeline is failing with the following error:
Example build (internal link)
The text was updated successfully, but these errors were encountered: