Skip to content

Conversation

@frank-dong-ms-zz
Copy link
Contributor

related to PR: #4854

setting CopyLocalLockFileAssemblies to false for netfx cause test not running on CI like below:
https://dev.azure.com/dnceng/public/_build/results?buildId=528701&view=logs&j=32952595-30e7-56fa-9b86-c4579b87f5d1

restrict CopyLocalLockFileAssemblies to false only for netcore 3.0

@frank-dong-ms-zz frank-dong-ms-zz requested a review from a team as a code owner February 20, 2020 04:26
<VSTestResultsDirectory>$(OutputPath)</VSTestResultsDirectory>
<AssemblyOriginatorKeyFile>$(ToolsDir)Test.snk</AssemblyOriginatorKeyFile>
<CopyLocalLockFileAssemblies>false</CopyLocalLockFileAssemblies>
<CopyLocalLockFileAssemblies Condition="'$(TargetFramework)' == 'netcoreapp3.0'">false</CopyLocalLockFileAssemblies>
Copy link
Contributor

@harishsk harishsk Feb 20, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought we set this to save on disk space. If we set this only for netcoreapp3.0 what happens to the diskspace usage on other platforms? #Resolved

Copy link
Contributor Author

@frank-dong-ms-zz frank-dong-ms-zz Feb 20, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, we only met disk issue on dotnet core 3.0 because of below setting from build tools:

false

false

true

for dotnet core 3.0 packages will copy to output folder by default and use extra 6-7 GB. So I set here to not copy packages to output, but this setting to false somehow affect test running for netfx 461 as below:
https://dev.azure.com/dnceng/public/_build/results?buildId=528701&view=logs&j=32952595-30e7-56fa-9b86-c4579b87f5d1

I'm not fully understand why this setting will affect test running on CI for netfx 461 as xunit seems not able to find any test to run but restrict this setting only for dotnet core 3.0 fix the disk issue also allow netfx 461 test to run on CI


In reply to: 382258052 [](ancestors = 382258052)

@frank-dong-ms-zz frank-dong-ms-zz merged commit a1106d7 into dotnet:master Feb 20, 2020
@frank-dong-ms-zz frank-dong-ms-zz deleted the fix-netfx branch April 15, 2020 20:36
@ghost ghost locked as resolved and limited conversation to collaborators Mar 19, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants