Skip to content

Update to SDK that understands NUGET_ENABLE_EXPERIMENTAL_HTTP_RETRY #7814

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.

Sign up for GitHub

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

Merged
MattGal merged 4 commits into from
Aug 31, 2021
Merged

Update to SDK that understands NUGET_ENABLE_EXPERIMENTAL_HTTP_RETRY #7814

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
6dce766
Update to an SDK that understands NUGET_ENABLE_EXPERIMENTAL_HTTP_RETR…
MattGal Aug 31, 2021
fd58003
Try to work around "The target framework 'netcoreapp2.1' is out of su…
MattGal Aug 31, 2021
563d664
Move logging back to netstandard2.0
MattGal Aug 31, 2021
d0a6890
PR feedbacks: Only set variables in tools.ps1/sh where we set other …
MattGal Aug 31, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 10 additions & 0 deletions eng/common/templates/job/job.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -62,6 +62,16 @@ jobs:
timeoutInMinutes: ${{ parameters.timeoutInMinutes }}

variables:

# These values enable longer delays, configurable number of retries, and special understanding of TCP hang-up
# See https://github.com/NuGet/Home/issues/11027 for details
- name: NUGET_ENABLE_EXPERIMENTAL_HTTP_RETRY
Comment thread
MattGal marked this conversation as resolved.
Outdated
value: true
- name: NUGET_EXPERIMENTAL_MAX_NETWORK_TRY_COUNT
value: 6
- name: NUGET_EXPERIMENTAL_NETWORK_RETRY_DELAY_MILLISECONDS
value: 1000

- ${{ if ne(parameters.enableTelemetry, 'false') }}:
- name: DOTNET_CLI_TELEMETRY_PROFILE
value: '$(Build.Repository.Uri)'
Expand Down
2 changes: 2 additions & 0 deletions eng/common/tools.ps1
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -717,6 +717,8 @@ function MSBuild() {
(Join-Path $basePath (Join-Path $buildTool.Framework 'Microsoft.DotNet.Arcade.Sdk.dll')),
(Join-Path $basePath (Join-Path netcoreapp2.1 'Microsoft.DotNet.ArcadeLogging.dll')),
(Join-Path $basePath (Join-Path netcoreapp2.1 'Microsoft.DotNet.Arcade.Sdk.dll'))
(Join-Path $basePath (Join-Path netcoreapp3.1 'Microsoft.DotNet.ArcadeLogging.dll')),
(Join-Path $basePath (Join-Path netcoreapp3.1 'Microsoft.DotNet.Arcade.Sdk.dll'))
)
$selectedPath = $null
foreach ($path in $possiblePaths) {
Expand Down
2 changes: 2 additions & 0 deletions eng/common/tools.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -427,6 +427,8 @@ function MSBuild {
possiblePaths+=( "$toolset_dir/$_InitializeBuildToolFramework/Microsoft.DotNet.Arcade.Sdk.dll" )
possiblePaths+=( "$toolset_dir/netcoreapp2.1/Microsoft.DotNet.ArcadeLogging.dll" )
possiblePaths+=( "$toolset_dir/netcoreapp2.1/Microsoft.DotNet.Arcade.Sdk.dll" )
possiblePaths+=( "$toolset_dir/netcoreapp3.1/Microsoft.DotNet.ArcadeLogging.dll" )
possiblePaths+=( "$toolset_dir/netcoreapp3.1/Microsoft.DotNet.Arcade.Sdk.dll" )
for path in "${possiblePaths[@]}"; do
if [[ -f $path ]]; then
selectedPath=$path
Expand Down
2 changes: 1 addition & 1 deletion global.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"tools": {
"dotnet": "6.0.100-rc.1.21379.2"
"dotnet": "6.0.100-rc.1.21430.12"
},
"msbuild-sdks": {
"Microsoft.DotNet.Arcade.Sdk": "6.0.0-beta.21427.6",
Expand Down
4 changes: 4 additions & 0 deletions src/Microsoft.DotNet.Build.Tasks.Packaging/src/build/Packaging.targets
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -929,6 +929,10 @@
<DefaultValidateFramework Include="netcoreapp3.0">
<RuntimeIDs>@(NETCoreApp30RIDs)</RuntimeIDs>
</DefaultValidateFramework>
<NETCoreApp31RIDs Condition="'@(NETCoreApp31RIDs)' == ''" Include="@(NETCoreApp30RIDs)" />
<DefaultValidateFramework Include="netcoreapp3.1">
<RuntimeIDs>@(NETCoreApp31RIDs)</RuntimeIDs>
</DefaultValidateFramework>
Comment on lines +932 to +935

@ViktorHofer ViktorHofer Sep 1, 2021

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Curious, why did you touch this as part of your PR? FYI we plan to delete this library with #7783.

@MattGal MattGal Sep 1, 2021

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Curious, why did you touch this as part of your PR? FYI we plan to delete this library with #7783.

Mostly it just came up in the grep, I asked @mmitche if it should have a 3.1 entry and he said it ought to. Will be happy to see this one and Tasks.Feed go :)

<NETCoreApp50RIDs Condition="'@(NETCoreApp50RIDs)' == ''" Include="@(NETCoreApp30RIDs)" />
<DefaultValidateFramework Include="net5.0">
<RuntimeIDs>@(NETCoreApp50RIDs)</RuntimeIDs>
Expand Down
2 changes: 1 addition & 1 deletion src/Microsoft.DotNet.RemoteExecutor/src/Microsoft.DotNet.RemoteExecutor.csproj
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
<!-- On .NET Framework and .NET Core we use this assembly as both a library and as an executable. -->
<OutputType>Exe</OutputType>
<!-- Since RemoteExecutor is a standalone library, we can target a lower version of .NET than the rest of arcade. -->
<TargetFrameworks>netcoreapp2.1;net461</TargetFrameworks>
<TargetFrameworks>netcoreapp3.1;net461</TargetFrameworks>
<Description>This package provides support for running tests out-of-process.</Description>
<ExcludeFromSourceBuild>true</ExcludeFromSourceBuild>
<IsPackable>true</IsPackable>
Expand Down