Shallow checkout used by Integration CI uses wrong commit

dotnet / roslyn

The Roslyn .NET compiler provides C# and Visual Basic languages with rich code analysis APIs.

https://docs.microsoft.com/dotnet/csharp/roslyn-sdk/

MIT License

19.06k stars 4.04k forks source link

Shallow checkout used by Integration CI uses wrong commit #52444

Open JoeRobich opened 3 years ago

JoeRobich commented 3 years ago

See https://dev.azure.com/dnceng/public/_build/results?buildId=1074927&view=logs&j=922f7fe3-0735-5839-4d76-f6d8a5e047bd&t=0d25a0ea-e524-58e7-6406-4c73ba346f51

From the Shallow Checkout log

Notice the line HEAD is now at 2f35dad9 Include mvids. Head should be pointing at a merge commit.

Tasks

[ ] Disable shallow checkout (https://github.com/dotnet/roslyn/pull/52445)
[ ] Reevaluate shallow checkout

RikkiGibson commented 3 years ago

Most PR runs are using the merge commit. Wondering if there's a possibility that /azp run is creating a run associated with the wrong commit ID--is this something that used to work properly in the past and is now broken?