Skip to content

Retry PublishArtifacts download on all exceptions - #7750

Merged
garath merged 3 commits into
dotnet:mainfrom
garath:13952-retry-on-ioexception-2
Aug 14, 2021
Merged

Retry PublishArtifacts download on all exceptions#7750
garath merged 3 commits into
dotnet:mainfrom
garath:13952-retry-on-ioexception-2

Conversation

@garath

@garath garath commented Aug 13, 2021

Copy link
Copy Markdown
Member

Follow-on of #7736, stronger resolution to the periodic download failure reported in dotnet/core-eng#13952.

@garath
garath requested review from a user and epananth August 13, 2021 21:57
@garath garath self-assigned this Aug 13, 2021
Comment thread src/Microsoft.DotNet.Build.Tasks.Feed/src/PublishArtifactsInManifestBase.cs Outdated
@garath
garath requested a review from ChadNedzlek August 13, 2021 22:20
@ChadNedzlek

Copy link
Copy Markdown
Contributor

:giant thumbs up: I'm a big fan of being really permissive with retries with things are supposed to work, and the cost of retry is low.

@garath
garath enabled auto-merge (squash) August 13, 2021 23:30
@garath
garath merged commit 9b7027b into dotnet:main Aug 14, 2021
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.

3 participants