7.0 source-built SDK packs are missing missing xml doc

[MichaelSimons]

The XmlDocTests caught a regression in 7.0 where it appears that all xml doc for the packs is missing.

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.

[MichaelSimons]

In speaking w/@crummel, he thinks this may be caused by not picking up the SBRP versions of the packs which are the only ones with the XML doc.

[oleksandr-didyk]

General overview:
Certain target packs that should contain .xml docs in the produced source-build do not contain any in 7.0.

The origin of the issue should be identified and, if plausible, fixed. The at-the-time-of-writing XmlDocTests disabled test should be updated and re-enabled to allow for catching of any similar issues in the future.

Actions to take:

• verify that source-build does indeed differ in .xml doc content compared to MS build. If it does not - the issue should be pushed to other product teams up the chain
• identify the reason for the issue and restore the .xml docs to the build
• update the baseline for XmlDocTests and enable it for CI and local test runs

[oleksandr-didyk]

I verified the available target pack .xml docs between the source-built and the default built product and found no differences.

I also compared the existing missing docs baseline that the test in question uses with the contents of the target packs and found 3 missing .xml doc files:

• Microsoft.NETCore.App.Ref\ref\netx.y\System.Net.Quic.xml
• Microsoft.NETCore.App.Ref\ref\netx.y\System.Runtime.InteropServices.JavaScript.xml
• Microsoft.NETCore.App.Ref\analyzers\dotnet\cs\Microsoft.Interop.JavaScript.JSImportGenerator.xml

@MichaelSimons could you please advise on the person I should refer this issue to? Thanks

Note: the dotnet/installer commit I used to build the product is 549b7d641e2f5e90ae284c3154b04cf48aa1b68f

[MichaelSimons]

@oleksandr-didyk, Hmmm, that doesn't match what I see with a quick inspection. Let me clarify what steps I think should be taken and let me know how/if that differs from what you did. These are the steps the test is doing.

1. Grab a source-built SDK from CI or build one yourself.
2. Install the MSFT built SDK for the same commit sha or one that is close.
3. Retrieve a list of the XML doc files included with the Microsoft.NETCore.App.Ref and Microsoft.AspNetCore.App.Ref packs directory of both the MSFT and SB builds
4. Diff the two lists - there should be no differences. I expect that you will find that most of the Microsoft.NETCore.App.Ref xml doc is missing.

The baseline file for the test represents the list of assemblies that do not container xml doc. The reason the test stores this as a baseline is so that it can avoid having to retrieve the MSFT build to compare against. This makes the test more efficient as well as supporting the ability to run in an offline environment.

[oleksandr-didyk]

I think I probably described the findings in a non-clear way, resulting in us miss-understanding each other. What I meant is that I got the same result as in point 4 -> the MSFT and source-built SDK do not differ in targeting packs doc content. All of the docs form the baseline are missing as you mentioned + I found additional 3 .dlls that are not stated in the baseline but also don't have a doc (the ones listed in my original comment)