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

Merging internal commits for release/7.0.1xx #14931

Conversation

vseanreesermsft
Copy link

No description provided.

@mmitche
Copy link
Member

mmitche commented Nov 9, 2022

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@MichaelSimons MichaelSimons enabled auto-merge (squash) November 9, 2022 20:07
@MichaelSimons MichaelSimons enabled auto-merge (squash) November 9, 2022 20:10
@crummel
Copy link

crummel commented Nov 9, 2022

Looking at this failure now.

@crummel
Copy link

crummel commented Nov 10, 2022

Error message:

Emitting PerfMap file: /__w/1/s/artifacts/source-build/self/src/artifacts/bin/Microsoft.AspNetCore.App.Runtime/Release/net7.0/linux-x64/Microsoft.Net.Http.Headers.ni.r2rmap
  EXEC : error : Multiple input files matching same simple name /__w/1/s/artifacts/source-build/self/src/artifacts/bin/Microsoft.AspNetCore.App.Runtime/Release/net7.0/linux-x64/System.Text.Json.dll /__w/1/s/artifacts/source-build/self/package-cache/microsoft.netcore.app.runtime.linux-x64/7.0.0/runtimes/linux-x64/lib/net7.0/System.Text.Json.dll [/__w/1/s/artifacts/source-build/self/src/src/Framework/App.Runtime/src/Microsoft.AspNetCore.App.Runtime.csproj]
##[error]EXEC(0,0): error : (NETCORE_ENGINEERING_TELEMETRY=Build) Multiple input files matching same simple name /__w/1/s/artifacts/source-build/self/src/artifacts/bin/Microsoft.AspNetCore.App.Runtime/Release/net7.0/linux-x64/System.Text.Json.dll /__w/1/s/artifacts/source-build/self/package-cache/microsoft.netcore.app.runtime.linux-x64/7.0.0/runtimes/linux-x64/lib/net7.0/System.Text.Json.dll
Internal.CommandLine.CommandLineException: Multiple input files matching same simple name /__w/1/s/artifacts/source-build/self/src/artifacts/bin/Microsoft.AspNetCore.App.Runtime/Release/net7.0/linux-x64/System.Text.Json.dll /__w/1/s/artifacts/source-build/self/package-cache/microsoft.netcore.app.runtime.linux-x64/7.0.0/runtimes/linux-x64/lib/net7.0/System.Text.Json.dll

The specific DLL is not always the same but it does seem to always be the MS.AspNetCore.App.Runtime package.

@MichaelSimons and I have been looking at this issue for quite a while now and so far we've discovered:

  • This repros consistently in the full tarball source-build.
  • It repros inconsistently in the aspnetcore source-build and non-source-build configurations.
  • It repros in both online and offline source-builds, so we think it's unlikely that it's the NuGet restore nondeterminism possibility that was raised.
    • The CopyLocal nondeterminism still seems possible but we don't know where that would be or how it would happen.
  • This is the first time we know of the issue showing up in source-build.
  • The changes from runtime and aspnetcore in this PR do not appear to be related to the issue.

We're now building a 7.0 tip full tarball source-build to compare against this PR, but we're kind of at a loss and could use some help diagnosing the issue. Any ideas? cc some source-build champs: @janvorli @rainersigwald @wtgodbe

@MichaelSimons
Copy link
Member

@MichaelSimons
Copy link
Member

The command line args for crossgen2 are

CommandLineArguments = "/repos/tarball-7.0.1xx/src/aspnetcore/artifacts/source-build/self/package-cache/microsoft.netcore.app.crossgen2.linux-x64/7.0.0/tools/crossgen2" --composite --targetarch:x64 --targetos:linux /repos/tarball-7.0.1xx/src/aspnetcore/artifacts/source-build/self/package-cache/microsoft.netcore.app.runtime.linux-x64/7.0.0/runtimes/linux-x64/lib/net7.0/*.dll /repos/tarball-7.0.1xx/src/aspnetcore/artifacts/source-build/self/src/artifacts/bin/Microsoft.AspNetCore.App.Runtime/Release/net7.0/linux-x64/*.dll --out:"/repos/tarball-7.0.1xx/src/aspnetcore/artifacts/source-build/self/src/artifacts/obj/Composite/Release/full-composite.r2r.dll"

Comparing the layout of 7.0.1xx tip and this PR I see a difference in the contents of /repos/tarball-7.0.1xx/src/aspnetcore/artifacts/source-build/self/src/artifacts/bin/Microsoft.AspNetCore.App.Runtime/Release/net7.0/linux-x64/*.dll

This PR is includes System.Formats.Asn1.dll, System.Text.Encodings.Web.dll and System.Text.Json.dll

@MichaelSimons
Copy link
Member

I have binlogs available for the two environments if needed.

@dougbu
Copy link
Member

dougbu commented Nov 10, 2022

/cc @wtgodbe

@crummel @MichaelSimons in dotnet/aspnetcore, we hit exactly the same issue prior to our equivalent of this PR going in. The problem turned out to be mixing a prior public stable-versioned feed for a dotnet/runtime 7.0.0 build w/ the dotnet-public feed containing same-versioned copies of packages. In the System.Text.Json and System.Text.Encodings.Web cases, there were differences in the contained assemblies that caused the problem.

@MichaelSimons
Copy link
Member

In the System.Text.Json and System.Text.Encodings.Web cases, there were differences in the contained assemblies that caused the problem.

@dougbu - what differences did you see?

@dougbu
Copy link
Member

dougbu commented Nov 10, 2022

Over to @wtgodbe for the details. In brief, one assembly exposed different APIs than expected and the other caused the Multiple input files matching same simple name errors in our release/7.0 builds (and PRs targeting that branch) after dotnet-public was updated and before we merged our merge-from-internal PR.
I suspect either source-build has something similar going on (perhaps still) or everything targeting release/7.0.1xx is failing and will continue to fail until you're using newer (7.0.1) runtime and aspnetcore bits here.

@MichaelSimons MichaelSimons force-pushed the internal-merge-7.0.1xx-2022-11-08-1117 branch 2 times, most recently from 651d90e to 22686dd Compare November 11, 2022 20:38
@crummel crummel self-requested a review November 11, 2022 20:38
@MichaelSimons MichaelSimons force-pushed the internal-merge-7.0.1xx-2022-11-08-1117 branch from 22686dd to e924291 Compare November 11, 2022 20:46
@MichaelSimons MichaelSimons force-pushed the internal-merge-7.0.1xx-2022-11-08-1117 branch from e924291 to 5926aeb Compare November 11, 2022 21:30
@crummel crummel self-requested a review November 11, 2022 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants