Skip to content

Conversation

dotnet-maestro[bot]
Copy link
Contributor

@dotnet-maestro dotnet-maestro bot commented May 19, 2025

This pull request updates the following dependencies

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

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

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

…ild 20250515.3

Microsoft.DotNet.HotReload.Utils.Generator.BuildTool
 From Version 10.0.0-alpha.0.25255.1 -> To Version 10.0.0-alpha.0.25265.3
Microsoft.NETCore.Runtime.ICU.Transport
 From Version 10.0.0-preview.5.25261.1 -> To Version 10.0.0-preview.5.25268.1
@dotnet-maestro dotnet-maestro bot changed the title [main] Update dependencies from dotnet/hotreload-utils [main] Update dependencies from dotnet/hotreload-utils, dotnet/icu May 20, 2025
…ld 20250519.1

Microsoft.DotNet.CilStrip.Sources , Microsoft.NET.HostModel.TestData , System.ComponentModel.TypeConverter.TestData , System.Data.Common.TestData , System.Drawing.Common.TestData , System.Formats.Tar.TestData , System.IO.Compression.TestData , System.IO.Packaging.TestData , System.Net.TestData , System.Private.Runtime.UnicodeData , System.Runtime.Numerics.TestData , System.Runtime.TimeZoneData , System.Security.Cryptography.X509Certificates.TestData , System.Text.RegularExpressions.TestData , System.Windows.Extensions.TestData
 From Version 10.0.0-beta.25262.1 -> To Version 10.0.0-beta.25269.1
@dotnet-maestro dotnet-maestro bot changed the title [main] Update dependencies from dotnet/hotreload-utils, dotnet/icu [main] Update dependencies from dotnet/hotreload-utils, dotnet/icu, dotnet/runtime-assets May 20, 2025
Microsoft.NETCore.Runtime.ICU.Transport
 From Version 10.0.0-preview.5.25261.1 -> To Version 10.0.0-preview.6.25270.1
dotnet-maestro bot added 3 commits May 26, 2025 05:02
…ild 20250519.1

Microsoft.DotNet.HotReload.Utils.Generator.BuildTool
 From Version 10.0.0-alpha.0.25255.1 -> To Version 10.0.0-alpha.0.25269.1
…ld 20250526.1

Microsoft.DotNet.CilStrip.Sources , Microsoft.NET.HostModel.TestData , System.ComponentModel.TypeConverter.TestData , System.Data.Common.TestData , System.Drawing.Common.TestData , System.Formats.Tar.TestData , System.IO.Compression.TestData , System.IO.Packaging.TestData , System.Net.TestData , System.Private.Runtime.UnicodeData , System.Runtime.Numerics.TestData , System.Runtime.TimeZoneData , System.Security.Cryptography.X509Certificates.TestData , System.Text.RegularExpressions.TestData , System.Windows.Extensions.TestData
 From Version 10.0.0-beta.25262.1 -> To Version 10.0.0-beta.25276.1
…ild 20250526.1

Microsoft.DotNet.HotReload.Utils.Generator.BuildTool
 From Version 10.0.0-alpha.0.25255.1 -> To Version 10.0.0-alpha.0.25276.1
@akoeplinger
Copy link
Member

Test failure:

Applying metadata update for /datadisks/disk1/work/96C30856/w/B1820954/e/System.Reflection.Metadata.ApplyUpdate.Test.AddInstanceField.dll, revision 1
[ERROR] FATAL UNHANDLED EXCEPTION: System.ArgumentException: Field not large enough to fill array (Parameter 'field_handle')
   at System.Runtime.CompilerServices.RuntimeHelpers.InitializeArray(Array array, RuntimeFieldHandle fldHandle) in /_/src/mono/System.Private.CoreLib/src/System/Runtime/CompilerServices/RuntimeHelpers.Mono.cs:line 19
   at System.Reflection.Metadata.ApplyUpdate.Test.AddInstanceField..ctor() in /__w/1/s/src/libraries/System.Runtime.Loader/tests/ApplyUpdate/System.Reflection.Metadata.ApplyUpdate.Test.AddInstanceField/AddInstanceField.cs:line 42
   at System.Reflection.Metadata.ApplyUpdateTest.<>c.<TestAddInstanceField>b__11_0() in /_/src/libraries/System.Runtime.Loader/tests/ApplyUpdateTest.cs:line 389
   at System.Reflection.MethodBaseInvoker.InterpretedInvoke_Method(Object obj, IntPtr* args) in /_/src/mono/System.Private.CoreLib/src/System/Reflection/MethodBaseInvoker.Mono.cs:line 22
   at System.Reflection.MethodBaseInvoker.InvokeWithNoArgs(Object obj, BindingFlags invokeAttr) in /_/src/libraries/System.Private.CoreLib/src/System/Reflection/MethodBaseInvoker.cs:line 57
--- End of stack trace from previous location ---
   at Microsoft.DotNet.RemoteExecutor.Program.Main(String[] args) in /_/src/arcade/src/Microsoft.DotNet.RemoteExecutor/src/Program.cs:line 97
Running test using RemoteExecutor
    System.Reflection.Metadata.ApplyUpdateTest.TestAddInstanceField [FAIL]
      Microsoft.DotNet.RemoteExecutor.RemoteExecutionException : Remote process failed with an unhandled exception.
      Stack Trace:
        
        Child exception:
          System.ArgumentException: Field not large enough to fill array (Parameter 'field_handle')
        /_/src/mono/System.Private.CoreLib/src/System/Runtime/CompilerServices/RuntimeHelpers.Mono.cs(19,0): at System.Runtime.CompilerServices.RuntimeHelpers.InitializeArray(Array array, RuntimeFieldHandle fldHandle)
        /__w/1/s/src/libraries/System.Runtime.Loader/tests/ApplyUpdate/System.Reflection.Metadata.ApplyUpdate.Test.AddInstanceField/AddInstanceField.cs(42,0): at System.Reflection.Metadata.ApplyUpdate.Test.AddInstanceField..ctor()
        /_/src/libraries/System.Runtime.Loader/tests/ApplyUpdateTest.cs(389,0): at System.Reflection.Metadata.ApplyUpdateTest.<>c.<TestAddInstanceField>b__11_0()
        /_/src/mono/System.Private.CoreLib/src/System/Reflection/MethodBaseInvoker.Mono.cs(22,0): at System.Reflection.MethodBaseInvoker.InterpretedInvoke_Method(Object obj, IntPtr* args)
        /_/src/libraries/System.Private.CoreLib/src/System/Reflection/MethodBaseInvoker.cs(57,0): at System.Reflection.MethodBaseInvoker.InvokeWithNoArgs(Object obj, BindingFlags invokeAttr)
        
        Child process:
          System.Runtime.Loader.Tests, Version=10.0.0.0, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51 System.Reflection.Metadata.ApplyUpdateTest+<>c Void <TestAddInstanceField>b__11_0()

@jkoritzinsky you made some code changes in hotreload-utils, do you think those could be related?

dotnet-maestro bot added 2 commits June 3, 2025 02:02
Microsoft.NETCore.Runtime.ICU.Transport
 From Version 10.0.0-preview.5.25261.1 -> To Version 10.0.0-preview.6.25302.1
…ld 20250602.1

Microsoft.DotNet.CilStrip.Sources , Microsoft.NET.HostModel.TestData , System.ComponentModel.TypeConverter.TestData , System.Data.Common.TestData , System.Drawing.Common.TestData , System.Formats.Tar.TestData , System.IO.Compression.TestData , System.IO.Packaging.TestData , System.Net.TestData , System.Private.Runtime.UnicodeData , System.Runtime.Numerics.TestData , System.Runtime.TimeZoneData , System.Security.Cryptography.X509Certificates.TestData , System.Text.RegularExpressions.TestData , System.Windows.Extensions.TestData
 From Version 10.0.0-beta.25262.1 -> To Version 10.0.0-beta.25302.1
@jkoritzinsky
Copy link
Member

I updated the Roslyn we test with, so it's probably related to that.

…ild 20250602.2

Microsoft.DotNet.HotReload.Utils.Generator.BuildTool
 From Version 10.0.0-alpha.0.25255.1 -> To Version 10.0.0-alpha.0.25302.2
…ld 20250610.1

Microsoft.DotNet.CilStrip.Sources , Microsoft.NET.HostModel.TestData , System.ComponentModel.TypeConverter.TestData , System.Data.Common.TestData , System.Drawing.Common.TestData , System.Formats.Tar.TestData , System.IO.Compression.TestData , System.IO.Packaging.TestData , System.Net.TestData , System.Private.Runtime.UnicodeData , System.Runtime.Numerics.TestData , System.Runtime.TimeZoneData , System.Security.Cryptography.X509Certificates.TestData , System.Text.RegularExpressions.TestData , System.Windows.Extensions.TestData
 From Version 10.0.0-beta.25262.1 -> To Version 10.0.0-beta.25310.1
@akoeplinger
Copy link
Member

/ba-g failures are unrelated to the changes here. disabled the earlier failing test against #116624

@akoeplinger akoeplinger merged commit 83ffb89 into main Jun 13, 2025
150 of 160 checks passed
@dotnet-maestro dotnet-maestro bot deleted the darc-main-30cc607c-713e-40fc-8f7e-76432994de21 branch June 13, 2025 16:40
@github-actions github-actions bot locked and limited conversation to collaborators Jul 14, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-codeflow for labeling automated codeflow

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants