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

Update dependencies with darc #6714

Merged
merged 26 commits into from
Jan 17, 2019
Merged

Conversation

natemcmaster
Copy link
Contributor

Manually ran darc update-dependencies -c '.NET Core 3 Dev'.

It missed:

  • Updating DotNetEfPackageVersion, even though versions.details.xml was updated (cc @mmitche )
  • Updating dependencies from aspnetcore-tooling (cc @pranavkm @rynowak )

pranavkm and others added 2 commits January 15, 2019 10:44
* Use Razor SDK version with workaround for dotnet#6500
* Remove package reference to Razor SDK in template
@natemcmaster natemcmaster requested a review from dougbu January 15, 2019 18:55
@benaadams
Copy link
Member

Middleware\HealthChecks.EntityFrameworkCore\src\
 DependencyInjection\EntityFrameworkCoreHealthChecksBuilderExtensions.cs(44,24): 

error CS1574: XML comment has cref attribute 'IDatabaseCreatorWithCanConnect'
 that could not be resolved 

[F:\vsagent\11\s\src\Middleware\HealthChecks.EntityFrameworkCore\src\Microsoft.Extensions.Diagnostics.HealthChecks.EntityFrameworkCore.csproj]

@natemcmaster
Copy link
Contributor Author

Sent email to figure out where that API went.

@benaadams
Copy link
Member

benaadams commented Jan 16, 2019

System.Net.Http.HttpRequestException : Error while copying content to a stream.
---- System.IO.IOException : 
-------- System.TypeLoadException : Could not load type 'System.Buffers.OperationStatus'
 from assembly 'System.Memory, Version=4.2.0.0, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51'.

Guessing you need a new runtime? dotnet/coreclr#21912

/cc @GrabYourPitchforks

@natemcmaster natemcmaster added the blocked The work on this issue is blocked due to some dependency label Jan 16, 2019
@natemcmaster natemcmaster changed the title wip: Update dependencies with darc Update dependencies with darc Jan 16, 2019
@natemcmaster
Copy link
Contributor Author

Progress:

@natemcmaster
Copy link
Contributor Author

@benaadams I ran update again to get the latest netcore.app. If the problem persists, then this will likely need to be sent to corefx/coreclr as a partner-blocking issues.

@benaadams
Copy link
Member

Different one this time

 error : Missing Version metadata for the following external dependencies:
 error :  - Microsoft.JSInterop. 

2019-01-16T02:21:11.1735260Z F:\vsagent\8\s\build\repo.targets(270,5):
  [F:\vsagent\8\.dotnet\buildtools\korebuild\3.0.0-build-20190110.4\KoreBuild.proj]
2019-01-16T02:21:11.1735472Z F:\vsagent\8\s\build\repo.targets(270,5): 
 [F:\vsagent\8\.dotnet\buildtools\korebuild\3.0.0-build-20190110.4\KoreBuild.proj]

@benaadams
Copy link
Member

Build and test: ANCM IIS Job

2019-01-16T02:40:46.4592825Z Current agent version: '2.144.0'
2019-01-16T02:40:46.4837499Z Prepare build directory.
2019-01-16T02:40:47.1261315Z ##[error]One or more errors occurred.
 (One or more errors occurred.
 (Access to the path 'F:\vsagent\2\s\
  src\Servers\Kestrel\test\Interop.FunctionalTests\bin\Release
  \netcoreapp3.0\h2spec\windows\h2spec.exe' is denied.)) 
  (Access to the path 'F:\vsagent\2\s\src\Servers\Kestrel\test\Interop.FunctionalTests\bin\Release
  \netcoreapp3.0\h2spec\windows\h2spec.exe' is denied.)
2019-01-16T02:40:47.1292725Z ##[section]Finishing: Initialize job

@benaadams
Copy link
Member

  [xUnit.net 00:11:21.21]     E2ETests.SmokeTests.Smoke_Tests(variant: Server: IISExpress, TFM: netcoreapp3.0, Type: Standalone, Arch: x64, ANCM: V2, Host: InProcess) [FAIL]
Failed   E2ETests.SmokeTests.Smoke_Tests(variant: Server: IISExpress, TFM: netcoreapp3.0, Type: Standalone, Arch: x64, ANCM: V2, Host: InProcess)
Error Message:
 Assert.Contains() Failure
Not found: <h2>Checkout Complete</h2>
In value:  <!DOCTYPE html>
<html>
<head>
    <meta charset="utf-8" />
    <meta name="viewport" content="width=device-width, initial-scale=1.0">
    <title>Address And Payment - ASP.NET MVC Music Store</title>

    
Stack Trace:
   at E2ETests.Validator.CheckOutCartItems() in /_/src/MusicStore/test/MusicStore.E2ETests/Implementation/Validator.cs:line 509
   at E2ETests.SmokeTests.RunTestsAsync(DeploymentResult deploymentResult, ILogger logger) in /_/src/MusicStore/test/MusicStore.E2ETests/SmokeTests.cs:line 142
   at E2ETests.SmokeTests.Smoke_Tests(TestVariant variant) in /_/src/MusicStore/test/MusicStore.E2ETests/SmokeTests.cs:line 56
--- End of stack trace from previous location where exception was thrown ---
019-01-16T03:25:00.1466173Z F:\vsagent\7\.dotnet\buildtools\korebuild\
 3.0.0-build-20190110.4\modules\vstest\module.targets(136,5):

 error : Test group MusicStore.E2ETests/netcoreapp3.0 failed with exit code '1'. [F:\vsagent\7\.dotnet\buildtools\korebuild\3.0.0-build-20190110.4\KoreBuild.proj]

@natemcmaster natemcmaster removed the blocked The work on this issue is blocked due to some dependency label Jan 16, 2019
@natemcmaster
Copy link
Contributor Author

The test failures are consistent, not just flaky. @ajcvickers were there any recent changes in EF Core which might lead to this error?

Microsoft.EntityFrameworkCore.Database.Command[20102]
      Failed executing DbCommand (19ms) [Parameters=[@p0='?' (DbType = Int32), @p1='?' (DbType = Int32), @p2='?' (DbType = Int32), @p3='?' (DbType = Decimal)], CommandType='Text', CommandTimeout='30']
      SET NOCOUNT ON;
      INSERT INTO [OrderDetails] ([AlbumId], [OrderId], [Quantity], [UnitPrice])
      VALUES (@p0, @p1, @p2, @p3);
      SELECT [OrderDetailId]
      FROM [OrderDetails]
      WHERE @@ROWCOUNT = 1 AND [OrderDetailId] = scope_identity();
System.Data.SqlClient.SqlException (0x80131904): The INSERT statement conflicted with the FOREIGN KEY constraint "FK_OrderDetails_Orders_OrderId". The conflict occurred in database "MusicStore", table "dbo.Orders", column 'OrderId'.
The statement has been terminated.
   at System.Data.SqlClient.SqlCommand.<>c.<ExecuteDbDataReaderAsync>b__126_0(Task`1 result)
   at System.Threading.Tasks.ContinuationResultTaskFromResultTask`2.InnerInvoke()
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location where exception was thrown ---
   at System.Threading.Tasks.Task.ExecuteWithThreadLocal(Task& currentTaskSlot, Thread threadPoolThread)
--- End of stack trace from previous location where exception was thrown ---
   at Microsoft.EntityFrameworkCore.Storage.Internal.RelationalCommand.ExecuteAsync(IRelationalConnection connection, DbCommandMethod executeMethod, IReadOnlyDictionary`2 parameterValues, CancellationToken cancellationToken) in /_/src/EFCore.Relational/Storage/Internal/RelationalCommand.cs:line 340

@natemcmaster
Copy link
Contributor Author

Build failures this time due to dotnet/arcade#1677. The version of core-setup hasn't caught up with the version of corefx packages.

@benaadams
Copy link
Member

Failure is unexpected assemblies

Starting test execution, please wait...
[xUnit.net 00:00:00.33]     Microsoft.AspNetCore.SharedFxTests.PlatformManifestListsAllFiles [FAIL]
Failed   Microsoft.AspNetCore.SharedFxTests.PlatformManifestListsAllFiles
Error Message:
 Assert.Empty() Failure
Collection: ["Microsoft.Win32.Registry", "System.Diagnostics.DiagnosticSource", "System.Reflection.Metadata", "System.Security.AccessControl", "System.Security.Cryptography.Cng", ...]
Stack Trace:
   at Microsoft.AspNetCore.SharedFxTests.PlatformManifestListsAllFiles() in /_/src/Framework/test/SharedFxTests.cs:line 72
Standard Output Messages:
 ==== file contents ====
...
*snip*

 ==== unexpected assemblies in the manifest ====
 Microsoft.Win32.Registry
System.Diagnostics.DiagnosticSource
System.Reflection.Metadata
System.Security.AccessControl
System.Security.Cryptography.Cng
System.Security.Principal.Windows


Results File: F:\vsagent\2\s\artifacts\logs\SharedFx-UnitTests-3.0.0-preview-19066-79.trx

Total tests: 4. Passed: 3. Failed: 1. Skipped: 0.
Test Run Failed.
Test execution time: 1.2700 Seconds
\vsagent\2\s\build\SharedFx.targets(104,5):
 error : SharedFx.UnitTests failed with exit code '1'. [F:\vsagent\2\.dotnet\buildtools\korebuild\3.0.0-build-20190110.4\KoreBuild.proj]

Build FAILED.

@natemcmaster
Copy link
Contributor Author

@benaadams - yes, this is the 'diamond dependency' problem I described in dotnet/arcade#1677. We don't have a good mechanism yet for telling Arcade that we can't take a version of corefx higher than what core-setup used. Until we have a fix for this, our dependency flow is going to take a lot of trial and error to get the right combination of versions.

@natemcmaster
Copy link
Contributor Author

Waiting on dotnet/core-setup#4996. cc @dagood

@natemcmaster natemcmaster added the blocked The work on this issue is blocked due to some dependency label Jan 17, 2019
@natemcmaster
Copy link
Contributor Author

Test failures in Linux now.

2019-01-17T20:30:24.4873203Z   System.InvalidOperationException: No process is associated with this object.
2019-01-17T20:30:24.4873972Z      at System.Diagnostics.Process.EnsureState(State state)
2019-01-17T20:30:24.4874675Z      at System.Diagnostics.Process.EnsureState(State state)
2019-01-17T20:30:24.4875635Z      at System.Diagnostics.Process.GetWaitState()
2019-01-17T20:30:24.4876288Z      at System.Diagnostics.Process.WaitForExitCore(Int32 milliseconds)
2019-01-17T20:30:24.4876529Z      at System.Diagnostics.Process.WaitForExit()
2019-01-17T20:30:24.4876780Z      at Microsoft.AspNetCore.SignalR.StackExchangeRedis.Tests.Docker.RunProcessAndWait(String fileName, String arguments, String prefix, ILogger logger, TimeSpan timeout, String& output) in /_/src/SignalR/server/StackExchangeRedis/test/Docker.cs:line 163

https://dev.azure.com/dnceng/public/_build/results?buildId=75206&view=logs&jobId=7536d2cd-87d4-54fe-4891-bfbbf2741d83&taskId=125326b4-b8df-5c91-4ebb-8c86fdfefd4c&lineStart=10001&lineEnd=10011&colStart=1&colEnd=174

@muratg @BrennanConroy @mikaelm12 - does this problem look familiar?

@natemcmaster natemcmaster removed the blocked The work on this issue is blocked due to some dependency label Jan 17, 2019
@mikaelm12
Copy link
Contributor

Hmm, took a look at the build failure and I haven't seen that before

@muratg
Copy link
Contributor

muratg commented Jan 17, 2019

@mikaelm12 can you investigate the signalr issue please?

@mikaelm12
Copy link
Contributor

Yeah, will take a closer look

@mikaelm12
Copy link
Contributor

Discussed with Nate. We think this is a possible regression in the new .Net Core runtime. Reaching out to the appropriate people.

@benaadams
Copy link
Member

Might want to check corefx for issues on WaitForExit, see if there is anything related e.g. https://github.com/dotnet/corefx/issues/31713

@natemcmaster
Copy link
Contributor Author

@BrennanConroy
Copy link
Member

BrennanConroy commented Jan 17, 2019 via email

@natemcmaster
Copy link
Contributor Author

@muratg @Eilon one test failure in what appears to be a flaky test. Sounds sufficiently good to me to override the PR check. Thoughts?

@muratg
Copy link
Contributor

muratg commented Jan 17, 2019

@natemcmaster, agreed. :shipit:

@natemcmaster natemcmaster merged commit 090c53c into dotnet:master Jan 17, 2019
@natemcmaster natemcmaster deleted the update-deps branch January 17, 2019 21:31
@tmds
Copy link
Member

tmds commented Feb 21, 2019

2019-01-17T20:30:24.4873203Z   System.InvalidOperationException: No process is associated with this object.
2019-01-17T20:30:24.4873972Z      at System.Diagnostics.Process.EnsureState(State state)
2019-01-17T20:30:24.4874675Z      at System.Diagnostics.Process.EnsureState(State state)
2019-01-17T20:30:24.4875635Z      at System.Diagnostics.Process.GetWaitState()
2019-01-17T20:30:24.4876288Z      at System.Diagnostics.Process.WaitForExitCore(Int32 milliseconds)
2019-01-17T20:30:24.4876529Z      at System.Diagnostics.Process.WaitForExit()

@natemcmaster @benaadams this may help: you get this when you call WaitForExit after calling Stop/Dispose on a Process.

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.