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

[main] Update dependencies from 7 repositories #85913

Merged
merged 24 commits into from
May 22, 2023

Conversation

dotnet-maestro[bot]
Copy link
Contributor

@dotnet-maestro dotnet-maestro bot commented May 8, 2023

This pull request updates the following dependencies

From https://github.com/dotnet/runtime

  • Subscription: 3db4b8c8-0fae-4f82-086c-08dad31ae87d
  • Build: 20230515.1
  • Date Produced: May 15, 2023 11:50:50 AM UTC
  • Commit: 4c0f2e7
  • Branch: refs/heads/main

From https://github.com/dotnet/llvm-project

  • Subscription: a7d541fc-4d59-4f09-2997-08d96284e872
  • Build: 20230519.1
  • Date Produced: May 19, 2023 9:04:27 PM UTC
  • Commit: ed3337915237ff11c062579df60331efc501ede7
  • Branch: refs/heads/dotnet/main

From https://github.com/dotnet/xharness

  • Subscription: be30ac4f-4b72-4287-1eb6-08d8d8fef0ea
  • Build: 20230509.1
  • Date Produced: May 9, 2023 8:18:44 AM UTC
  • Commit: 86ee2687cca0a1a28adf67eb3451ca06f0e989b4
  • Branch: refs/heads/main

From https://github.com/dotnet/emsdk

  • Subscription: c22d5069-447c-4252-29fd-08d90a7bb4bc
  • Build: 20230515.1
  • Date Produced: May 15, 2023 3:33:03 PM UTC
  • Commit: 7a90ec9edc7bad6f54ee57a4ab10787c1b989a54
  • Branch: refs/heads/main

From https://github.com/dotnet/hotreload-utils

  • Subscription: bfe6dacf-8231-4ea1-e2fe-08d962847885
  • Build: 20230515.2
  • Date Produced: May 15, 2023 4:20:03 PM UTC
  • Commit: f97865edbf5ca2bc2cf617440d43a2e645550e53
  • Branch: refs/heads/main

From https://github.com/dotnet/cecil

  • Subscription: bb5d2106-9fd3-425f-0abc-08daad65778c
  • Build: 20230515.2
  • Date Produced: May 15, 2023 2:07:26 PM UTC
  • Commit: f283c6229347f2dc3a25b04f6611014d8871d0b0
  • Branch: refs/heads/main

From https://github.com/dotnet/runtime-assets

  • Subscription: 0c5a34f5-504e-413b-9376-08d8d8ff2d75
  • Build: 20230515.1
  • Date Produced: May 15, 2023 3:21:25 PM UTC
  • Commit: 75fa6c5dfb4e247ec2846381ac784ed5b6e98827
  • Branch: refs/heads/main

…0507.3

Microsoft.NET.ILLink.Tasks , Microsoft.NET.Sdk.IL , Microsoft.NETCore.App.Runtime.win-x64 , Microsoft.NETCore.ILAsm , runtime.native.System.IO.Ports , System.Text.Json
 From Version 8.0.0-preview.5.23251.1 -> To Version 8.0.0-preview.5.23257.3
@dotnet-issue-labeler dotnet-issue-labeler bot added the area-codeflow for labeling automated codeflow label May 8, 2023
… 20230508.2

runtime.linux-arm64.Microsoft.NETCore.Runtime.JIT.Tools , runtime.linux-arm64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk , runtime.linux-arm64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools , runtime.linux-arm64.Microsoft.NETCore.Runtime.ObjWriter , runtime.linux-musl-arm64.Microsoft.NETCore.Runtime.JIT.Tools , runtime.linux-musl-arm64.Microsoft.NETCore.Runtime.ObjWriter , runtime.linux-musl-x64.Microsoft.NETCore.Runtime.JIT.Tools , runtime.linux-musl-x64.Microsoft.NETCore.Runtime.ObjWriter , runtime.linux-x64.Microsoft.NETCore.Runtime.JIT.Tools , runtime.linux-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk , runtime.linux-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools , runtime.linux-x64.Microsoft.NETCore.Runtime.ObjWriter , runtime.osx-arm64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk , runtime.osx-arm64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools , runtime.osx-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk , runtime.osx-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools , runtime.win-arm64.Microsoft.NETCore.Runtime.JIT.Tools , runtime.win-arm64.Microsoft.NETCore.Runtime.ObjWriter , runtime.win-x64.Microsoft.NETCore.Runtime.JIT.Tools , runtime.win-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk , runtime.win-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools , runtime.win-x64.Microsoft.NETCore.Runtime.ObjWriter
 From Version 14.0.0-alpha.1.23228.1 -> To Version 14.0.0-alpha.1.23258.2
@dotnet-maestro dotnet-maestro bot changed the title [main] Update dependencies from dotnet/runtime [main] Update dependencies from dotnet/runtime dotnet/llvm-project May 9, 2023
…30509.1

Microsoft.DotNet.XHarness.CLI , Microsoft.DotNet.XHarness.TestRunners.Common , Microsoft.DotNet.XHarness.TestRunners.Xunit
 From Version 8.0.0-prerelease.23253.1 -> To Version 8.0.0-prerelease.23259.1
@dotnet-maestro dotnet-maestro bot changed the title [main] Update dependencies from dotnet/runtime dotnet/llvm-project [main] Update dependencies from dotnet/runtime dotnet/llvm-project dotnet/xharness May 9, 2023
…08.1

Microsoft.NET.Workload.Emscripten.Current.Manifest-8.0.100.Transport
 From Version 8.0.0-preview.5.23252.1 -> To Version 8.0.0-preview.5.23258.1
@dotnet-maestro dotnet-maestro bot changed the title [main] Update dependencies from dotnet/runtime dotnet/llvm-project dotnet/xharness [main] Update dependencies from dotnet/runtime dotnet/llvm-project dotnet/xharness dotnet/emsdk May 9, 2023
dotnet-maestro bot added 2 commits May 9, 2023 13:25
…ild 20230508.1

Microsoft.DotNet.HotReload.Utils.Generator.BuildTool
 From Version 8.0.0-alpha.0.23252.1 -> To Version 8.0.0-alpha.0.23258.1
…09.1

Microsoft.DotNet.Cecil
 From Version 0.11.4-alpha.23252.1 -> To Version 0.11.4-alpha.23259.1
@dotnet-maestro dotnet-maestro bot changed the title [main] Update dependencies from dotnet/runtime dotnet/llvm-project dotnet/xharness dotnet/emsdk [main] Update dependencies from dotnet/runtime dotnet/llvm-project dotnet/xharness dotnet/emsdk dotnet/hotreload-utils dotnet/cecil May 9, 2023
@dotnet-maestro dotnet-maestro bot changed the title [main] Update dependencies from dotnet/runtime dotnet/llvm-project dotnet/xharness dotnet/emsdk dotnet/hotreload-utils dotnet/cecil [main] Update dependencies from dotnet/runtime dotnet/llvm-project dotnet/emsdk dotnet/hotreload-utils dotnet/cecil dotnet/xharness May 10, 2023
…ild 20230509.2

Microsoft.DotNet.HotReload.Utils.Generator.BuildTool
 From Version 8.0.0-alpha.0.23252.1 -> To Version 8.0.0-alpha.0.23259.2
@dotnet-maestro dotnet-maestro bot changed the title [main] Update dependencies from dotnet/runtime dotnet/llvm-project dotnet/emsdk dotnet/hotreload-utils dotnet/cecil dotnet/xharness [main] Update dependencies from dotnet/runtime dotnet/llvm-project dotnet/emsdk dotnet/cecil dotnet/xharness dotnet/hotreload-utils May 10, 2023
@dotnet-maestro dotnet-maestro bot changed the title [main] Update dependencies from dotnet/runtime dotnet/llvm-project dotnet/emsdk dotnet/cecil dotnet/xharness dotnet/hotreload-utils [main] Update dependencies from dotnet/runtime dotnet/llvm-project dotnet/emsdk dotnet/xharness dotnet/hotreload-utils dotnet/cecil May 10, 2023
@dotnet-maestro dotnet-maestro bot changed the title [main] Update dependencies from dotnet/runtime dotnet/llvm-project dotnet/emsdk dotnet/xharness dotnet/hotreload-utils dotnet/cecil [main] Update dependencies from dotnet/runtime dotnet/llvm-project dotnet/emsdk dotnet/hotreload-utils dotnet/cecil dotnet/xharness May 11, 2023
@dotnet-maestro dotnet-maestro bot changed the title [main] Update dependencies from dotnet/runtime dotnet/llvm-project dotnet/emsdk dotnet/hotreload-utils dotnet/cecil dotnet/xharness [main] Update dependencies from dotnet/runtime dotnet/llvm-project dotnet/hotreload-utils dotnet/cecil dotnet/xharness dotnet/emsdk May 11, 2023
@dotnet-maestro dotnet-maestro bot changed the title [main] Update dependencies from dotnet/runtime dotnet/llvm-project dotnet/hotreload-utils dotnet/cecil dotnet/xharness dotnet/emsdk [main] Update dependencies from dotnet/runtime dotnet/llvm-project dotnet/cecil dotnet/xharness dotnet/emsdk dotnet/hotreload-utils May 11, 2023
@dotnet-maestro dotnet-maestro bot changed the title [main] Update dependencies from dotnet/runtime dotnet/llvm-project dotnet/cecil dotnet/xharness dotnet/emsdk dotnet/hotreload-utils [main] Update dependencies from dotnet/runtime dotnet/llvm-project dotnet/xharness dotnet/emsdk dotnet/hotreload-utils dotnet/cecil May 11, 2023
@lewing lewing closed this May 11, 2023
@lewing lewing reopened this May 11, 2023
@dotnet-maestro dotnet-maestro bot changed the title [main] Update dependencies from dotnet/runtime dotnet/llvm-project dotnet/xharness dotnet/emsdk dotnet/hotreload-utils dotnet/cecil [main] Update dependencies from dotnet/runtime dotnet/llvm-project dotnet/emsdk dotnet/hotreload-utils dotnet/cecil dotnet/xharness May 12, 2023
@dotnet-maestro dotnet-maestro bot changed the title [main] Update dependencies from dotnet/runtime dotnet/llvm-project dotnet/emsdk dotnet/hotreload-utils dotnet/cecil dotnet/xharness [main] Update dependencies from dotnet/runtime dotnet/llvm-project dotnet/hotreload-utils dotnet/cecil dotnet/xharness dotnet/emsdk May 12, 2023
@dotnet-maestro dotnet-maestro bot changed the title [main] Update dependencies from dotnet/runtime dotnet/llvm-project dotnet/hotreload-utils dotnet/cecil dotnet/xharness dotnet/emsdk [main] Update dependencies from dotnet/runtime dotnet/llvm-project dotnet/cecil dotnet/xharness dotnet/emsdk dotnet/hotreload-utils May 12, 2023
dotnet-maestro bot added 2 commits May 16, 2023 13:32
…ild 20230515.2

Microsoft.DotNet.HotReload.Utils.Generator.BuildTool
 From Version 8.0.0-alpha.0.23252.1 -> To Version 8.0.0-alpha.0.23265.2
…15.2

Microsoft.DotNet.Cecil
 From Version 0.11.4-alpha.23252.1 -> To Version 0.11.4-alpha.23265.2
@akoeplinger
Copy link
Member

The installer tests fail with this error now:

System.TypeLoadException : Could not load type 'System.Buffers.SearchValues`1' from assembly 'System.Runtime, Version=8.0.0.0, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'.

Stack trace
   at System.Text.Json.Utf8JsonReader.ConsumeString()
   at System.Text.Json.Utf8JsonReader.ConsumePropertyName()
   at System.Text.Json.Utf8JsonReader.ReadSingleSegment()
   at System.Text.Json.Utf8JsonReader.Read()
   at System.Text.Json.JsonDocument.TryParseValue(Utf8JsonReader& reader, JsonDocument& document, Boolean shouldThrow, Boolean useArrayPools)
   at Microsoft.DotNet.CoreSetup.Test.RuntimeConfig.FromFile(String path) in /_/src/installer/tests/TestUtils/RuntimeConfig.cs:line 126
   at Microsoft.DotNet.CoreSetup.Test.HostActivation.FrameworkResolution.DotNetCliExtensions.DotNetFramework.RuntimeConfig(Action`1 runtimeConfigCustomizer) in /_/src/installer/tests/HostActivation.Tests/FrameworkResolution/DotNetCliExtensions.cs:line 80
   at Microsoft.DotNet.CoreSetup.Test.HostActivation.FrameworkResolution.RollForwardOnNoCandidateFxMultipleFrameworks.<>c__DisplayClass17_0.<ReconcileFrameworkReferences_InnerToInnerFrameworkReference_ToLower>b__1(DotNetCliCustomizer dotnetCustomizer) in /_/src/installer/tests/HostActivation.Tests/FrameworkResolution/RollForwardOnNoCandidateFxMultipleFrameworks.cs:line 354
   at Microsoft.DotNet.CoreSetup.Test.HostActivation.FrameworkResolution.FrameworkResolutionBase.RunTest(DotNetCli dotnet, TestApp app, TestSettings settings, Action`1 resultAction, Nullable`1 multiLevelLookup) in /_/src/installer/tests/HostActivation.Tests/FrameworkResolution/FrameworkResolutionBase.cs:line 31
   at Microsoft.DotNet.CoreSetup.Test.HostActivation.FrameworkResolution.RollForwardOnNoCandidateFxMultipleFrameworks.RunTest(Func`2 runtimeConfig, Action`1 customizeDotNet) in /_/src/installer/tests/HostActivation.Tests/FrameworkResolution/RollForwardOnNoCandidateFxMultipleFrameworks.cs:line 488
   at Microsoft.DotNet.CoreSetup.Test.HostActivation.FrameworkResolution.RollForwardOnNoCandidateFxMultipleFrameworks.ReconcileFrameworkReferences_InnerToInnerFrameworkReference_ToLower(String versionReference, Nullable`1 rollForwardOnNoCandidateFx, Nullable`1 applyPatches, String resolvedFramework) in /_/src/installer/tests/HostActivation.Tests/FrameworkResolution/RollForwardOnNoCandidateFxMultipleFrameworks.cs:line 349
   at InvokeStub_RollForwardOnNoCandidateFxMultipleFrameworks.ReconcileFrameworkReferences_InnerToInnerFrameworkReference_ToLower(Object, Object, IntPtr*)
   at System.Reflection.MethodInvoker.Invoke(Object obj, IntPtr* args, BindingFlags invokeAttr)

@akoeplinger
Copy link
Member

akoeplinger commented May 17, 2023

Ah we hit a similar issue on the sdk dependency update: dotnet/sdk#32250 (comment)

I'm not sure why we hit this here though, it looks like the installer tests don't run with the just built dotnet runtime?

@vitek-karas
Copy link
Member

@elinor-fung for ideas.

It's possible that we run the test with the SDK (as the tests themselves are not running the tested bits in-proc anyway), but I would expect that to be internally consistent.

@elinor-fung
Copy link
Member

The installer tests themselves run with the SDK, not the live-built runtime. The tests launch separate processes - those processes are the live-built hosts using the live-built runtime.

@elinor-fung
Copy link
Member

elinor-fung commented May 18, 2023

System.Text.Json is being referenced here (which the tests use):

<PackageReference Include="System.Text.Json" Version="$(SystemTextJsonVersion)" />

This PR updates that version to one that uses System.Buffers.SearchValues.

I'm not sure what to do here.

The basic situation is an app pulling in a newer System.Text.Json package and then running on a version of the runtime without the breaking change. For example:

  • Configure an application to use the latest 8.0 System.Text.Json:
    <PackageReference Include="System.Text.Json" Version="8.0.0-*" />
  • Add the dotnet8 nuget feed to get at the packages with the breaking change (presumably not needed once Preview 5 ships):
    <add key="dotnet8" value="https://pkgs.dev.azure.com/dnceng/public/_packaging/dotnet8/nuget/v3/index.json" />
  • Read/parse some json:
    var reader = new Utf8JsonReader("{\"name\":\"value\"}"u8);
    JsonObject root = (JsonObject)JsonNode.Parse(ref reader);
  • Run on a version of .NET that doesn't have the breaking change (like .NET 8 preview 4)
    Unhandled exception. System.TypeLoadException: Could not load type 'System.Buffers.SearchValues`1' from assembly 'System.Runtime, Version=8.0.0.0, Culture=neutral, PublicKeyToken=b03f5f7f11d50a3a'.
       at System.Text.Json.Utf8JsonReader.ConsumeString()
       at System.Text.Json.Utf8JsonReader.ConsumePropertyName()
       at System.Text.Json.Utf8JsonReader.ReadSingleSegment()
       at System.Text.Json.Utf8JsonReader.Read()
       at System.Text.Json.Utf8JsonReader.TrySkip()
       at System.Text.Json.JsonDocument.TryParseValue(Utf8JsonReader& reader, JsonDocument& document, Boolean shouldThrow, Boolean useArrayPools)
       at System.Text.Json.JsonElement.ParseValue(Utf8JsonReader& reader)
       at System.Text.Json.Nodes.JsonNode.Parse(Utf8JsonReader& reader, Nullable`1 nodeOptions)
       at Program.Main(String[] args) in C:\repos\helloworld\Program.cs:line 19
    

@vitek-karas
Copy link
Member

We could change the package reference to point to the "old" version of S.T.Json, then it should work with the SDK in the repo. And then when we update the SDK - this will break again and we will revert it back to the live-built S.T.Json.

@elinor-fung
Copy link
Member

I'm seeing if we can have get that dependency stop propagating to the tests / test utils - hoping to avoid the pin version / break / revert (and any second-level effect of that from SDK consuming Microsoft.NET.HostModel). If not, we can go with pointing at an old version to unblock this flow.

cc @MihaZupan for the OOB package behaviour per #85913 (comment). Not sure if that is a scenario we think users will hit, but I didn't see anything in the breaking change issue that mentions it or indicates how to resolve it.

@stephentoub
Copy link
Member

stephentoub commented May 18, 2023

Not sure if that is a scenario we think users will hit, but I didn't see anything in the breaking change issue that mentions it or indicates how to resolve it.

This is all within .NET 8, taking advantage of new APIs added in .NET 8 from an asset targeting .NET 8. We don't create breaking change issues for such things. It's a key tenet of adding new APIs that we use them wherever it's relevant to do so. And there's an expectation that if someone is trying out .NET 8, they're doing so using consistent preview versions of the libraries and runtime.

Copy link
Member

@lewing lewing left a comment

Choose a reason for hiding this comment

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

The failures appear to be unrelated, thank you @elinor-fung

…30509.1

Microsoft.DotNet.XHarness.CLI , Microsoft.DotNet.XHarness.TestRunners.Common , Microsoft.DotNet.XHarness.TestRunners.Xunit
 From Version 8.0.0-prerelease.23253.1 -> To Version 8.0.0-prerelease.23259.1
… 20230519.1

runtime.linux-arm64.Microsoft.NETCore.Runtime.JIT.Tools , runtime.linux-arm64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk , runtime.linux-arm64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools , runtime.linux-arm64.Microsoft.NETCore.Runtime.ObjWriter , runtime.linux-musl-arm64.Microsoft.NETCore.Runtime.JIT.Tools , runtime.linux-musl-arm64.Microsoft.NETCore.Runtime.ObjWriter , runtime.linux-musl-x64.Microsoft.NETCore.Runtime.JIT.Tools , runtime.linux-musl-x64.Microsoft.NETCore.Runtime.ObjWriter , runtime.linux-x64.Microsoft.NETCore.Runtime.JIT.Tools , runtime.linux-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk , runtime.linux-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools , runtime.linux-x64.Microsoft.NETCore.Runtime.ObjWriter , runtime.osx-arm64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk , runtime.osx-arm64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools , runtime.osx-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk , runtime.osx-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools , runtime.win-arm64.Microsoft.NETCore.Runtime.JIT.Tools , runtime.win-arm64.Microsoft.NETCore.Runtime.ObjWriter , runtime.win-x64.Microsoft.NETCore.Runtime.JIT.Tools , runtime.win-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Sdk , runtime.win-x64.Microsoft.NETCore.Runtime.Mono.LLVM.Tools , runtime.win-x64.Microsoft.NETCore.Runtime.ObjWriter
 From Version 14.0.0-alpha.1.23228.1 -> To Version 14.0.0-alpha.1.23269.1
@akoeplinger
Copy link
Member

The iOS failure is unrelated, merging.

@akoeplinger akoeplinger merged commit 9e49620 into main May 22, 2023
@akoeplinger akoeplinger deleted the darc-main-66bef9d5-62ab-4b75-82e0-13678cd02e4b branch May 22, 2023 07:40
@ghost ghost locked as resolved and limited conversation to collaborators Jun 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-codeflow for labeling automated codeflow area-HostModel Microsoft.NET.HostModel issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants