Skip to content

fix netfx test not running issue #4862

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

Merged
merged 1 commit into from
Feb 20, 2020

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
@@ -23,7 +23,7 @@
<VSTestLogger>trx</VSTestLogger>
<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