Skip to content
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

Rename "NetSdkManaged" types to "FileBased" #23955

Merged
merged 3 commits into from
Feb 18, 2022
Merged

Conversation

gkulin
Copy link
Member

@gkulin gkulin commented Feb 15, 2022

Fixes #23758

@gkulin gkulin requested a review from a team February 15, 2022 20:18
@dnfadmin
Copy link

dnfadmin commented Feb 15, 2022

CLA assistant check
All CLA requirements met.

@dsplaisted
Copy link
Member

@dotnet/nuget-team We had a test fail with the following error:

/private/tmp/helix/working/B17F0976/p/d/sdk/6.0.300-ci/NuGet.targets(564,5): error : The process cannot access the file '/private/tmp/helix/working/B17F0976/w/A8740934/e/testExecutionDirectory/.dotnet/.nuget/NuGet/nugetorgadd.trk' because it is being used by another process. [/private/tmp/helix/working/B17F0976/w/A8740934/e/testExecutionDirectory/It_publishes_---B5C78A02/Library/Library.csproj]

Do you know what the nugetoradd.trk file is? Is there possibly a bug in NuGet with concurrent access to that file?

@joeloff
Copy link
Member

joeloff commented Feb 15, 2022

@dotnet/nuget-team We had a test fail with the following error:

/private/tmp/helix/working/B17F0976/p/d/sdk/6.0.300-ci/NuGet.targets(564,5): error : The process cannot access the file '/private/tmp/helix/working/B17F0976/w/A8740934/e/testExecutionDirectory/.dotnet/.nuget/NuGet/nugetorgadd.trk' because it is being used by another process. [/private/tmp/helix/working/B17F0976/w/A8740934/e/testExecutionDirectory/It_publishes_---B5C78A02/Library/Library.csproj]

Do you know what the nugetoradd.trk file is? Is there possibly a bug in NuGet with concurrent access to that file?

MSBuild had a similar/related issue: dotnet/msbuild#3925 (comment)

@zivkan
Copy link
Member

zivkan commented Feb 15, 2022

nugetoradd.trk is to mitigate Chocolately and Windows PowerShell's PowerShellGallery module creating a user-profile nuget.config without nuget.org as a package source, which confused many customers using these two tools before using dotnet or Visual Studio for the first time on a clean devbox. For more information: NuGet/Home#11387

Perhaps there's an issue with the implementation. By any chance, on the machines where it's failing, are they running tests in parallel, without ever having run dotnet build or dotnet restore once before running the tests? My guess is that the machine doesn't yet have the user-profile nuget.config or nugetorgadd.trk, and then running multiple processes at the same time that use NuGet.

@marcpopMSFT
Copy link
Member

Should we consider adding a dotnet build call to the RunTestsOnHelix.* scripts that runs before our helix tests get launched? That could potentially work around this if @zivkan's assumption is accurate.

@zivkan
Copy link
Member

zivkan commented Feb 16, 2022

dotnet nuget list source is sufficient for NuGet to create the default config (and nuget.org tracking file).

build/RunTestsOnHelix.sh Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants