Skip to content

Update dotnet sdk5 docker image to avoid nuget download error in C# .NET 5 distribtest#25420

Merged
jtattermusch merged 1 commit intogrpc:masterfrom
jtattermusch:dotnet_sdk5_nuget_distrib_fix
Feb 11, 2021
Merged

Update dotnet sdk5 docker image to avoid nuget download error in C# .NET 5 distribtest#25420
jtattermusch merged 1 commit intogrpc:masterfrom
jtattermusch:dotnet_sdk5_nuget_distrib_fix

Conversation

@jtattermusch
Copy link
Copy Markdown
Contributor

@jtattermusch jtattermusch commented Feb 11, 2021

Fixes internal b/178903415

The C# .NET5 distribtests are currently broken with

  Determining projects to restore...
/var/local/git/grpc/test/distrib/csharp/DistribTest/DistribTestDotNet.csproj : error NU3028: Package 'Microsoft.AspNetCore.App.Runtime.linux-x64 5.0.0-rc.2.20475.17' from source 'https://api.nuget.org/v3/index.json': The author primary signature's timestamp found a chain building issue: UntrustedRoot: self signed certificate in certificate chain
/var/local/git/grpc/test/distrib/csharp/DistribTest/DistribTestDotNet.csproj : error NU3037: Package 'Microsoft.AspNetCore.App.Runtime.linux-x64 5.0.0-rc.2.20475.17' from source 'https://api.nuget.org/v3/index.json': The author primary signature validity period has expired.
/var/local/git/grpc/test/distrib/csharp/DistribTest/DistribTestDotNet.csproj : error NU3028: Package 'Microsoft.AspNetCore.App.Runtime.linux-x64 5.0.0-rc.2.20475.17' from source 'https://api.nuget.org/v3/index.json': The repository countersignature's timestamp found a chain building issue: UntrustedRoot: self signed certificate in certificate chain
/var/local/git/grpc/test/distrib/csharp/DistribTest/DistribTestDotNet.csproj : error NU3028: Package 'Microsoft.NETCore.App.Runtime.linux-x64 5.0.0-rc.2.20475.5' from source 'https://api.nuget.org/v3/index.json': The author primary signature's timestamp found a chain building issue: UntrustedRoot: self signed certificate in certificate chain
/var/local/git/grpc/test/distrib/csharp/DistribTest/DistribTestDotNet.csproj : error NU3037: Package 'Microsoft.NETCore.App.Runtime.linux-x64 5.0.0-rc.2.20475.5' from source 'https://api.nuget.org/v3/index.json': The author primary signature validity period has expired.
/var/local/git/grpc/test/distrib/csharp/DistribTest/DistribTestDotNet.csproj : error NU3028: Package 'Microsoft.NETCore.App.Runtime.linux-x64 5.0.0-rc.2.20475.5' from source 'https://api.nuget.org/v3/index.json': The repository countersignature's timestamp found a chain building issue: UntrustedRoot: self signed certificate in certificate chain
  Failed to restore /var/local/git/grpc/test/distrib/csharp/DistribTest/DistribTestDotNet.csproj (in 2.15 sec).
+ FAILED=true

The root cause seems to be NuGet/Announcements#49
Since the distribtest is using the official dotnet SDK image, it is enough to just fetch a fresh version of it to get this fixed.

@jtattermusch jtattermusch added the release notes: no Indicates if PR should not be in release notes label Feb 11, 2021
@jtattermusch jtattermusch merged commit ab0a441 into grpc:master Feb 11, 2021
@stanley-cheung stanley-cheung mentioned this pull request Feb 12, 2021
Closed
@lidizheng
Copy link
Copy Markdown
Contributor

lidizheng commented Feb 12, 2021 •
edited
Loading

@jtattermusch Can you please also backport this change to v1.36.x branch?

@jtattermusch jtattermusch mentioned this pull request Feb 12, 2021
Merged
@jtattermusch
Copy link
Copy Markdown
Contributor Author

jtattermusch commented Feb 12, 2021

@jtattermusch Can you please also backport this change to v1.36.x branch?

Sure thing. Backport is here: #25433

lidizheng added a commit that referenced this pull request Feb 12, 2021
@lidizheng
Merge pull request #25433 from jtattermusch/backport_25420
3cae29d 
Backport #25420 into v1.36.x
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@apolcyn apolcyn apolcyn approved these changes

@nicolasnoble nicolasnoble Awaiting requested review from nicolasnoble

Assignees

@apolcyn apolcyn

Labels

release notes: no Indicates if PR should not be in release notes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@jtattermusch @lidizheng @apolcyn