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

Repeated ECONNRESET errors in setup-dotnet action #333

Closed
4 of 5 tasks
AaronFriel opened this issue Oct 11, 2022 · 9 comments
Closed
4 of 5 tasks

Repeated ECONNRESET errors in setup-dotnet action #333

AaronFriel opened this issue Oct 11, 2022 · 9 comments
Assignees
Labels
bug Something isn't working

Comments

@AaronFriel
Copy link

Description:

Roughly 70% of merge jobs in the pulumi/pulumi repo are failing due to a high failure rate on actions/setup-dotnet@v2.

No additional logs are present. We are running with these arguments:

  with:
    dotnet-version: 6.0.x
    include-prerelease: false

Image showing "Run actions/setup-dotnet@v2" and "Error: read ECONNRESET"

Task version:

actions/setup-dotnet@v2

Platform:

Failures have occurred on all three platforms.

Runner type:

  • Hosted
  • Self-hosted

Repro steps:

Unsure of steps to reproduce - we run a large number of jobs to validate all merges (on the order of ~100), and in more than half of the workflows in the past 24 hours the workflow failed due to

Expected behavior:

Action should succeed.

Actual behavior:

Action fails catastrophically on a network issue.

Example failures:

  1. https://github.com/pulumi/pulumi/actions/runs/3222484557/jobs/5271676054
  2. https://github.com/pulumi/pulumi/actions/runs/3223023912/jobs/5272739307
  3. https://github.com/pulumi/pulumi/actions/runs/3223183376/jobs/5273031068
  4. https://github.com/pulumi/pulumi/actions/runs/3227143235/jobs/5281716689
  5. https://github.com/pulumi/pulumi/actions/runs/3228086473/jobs/5283821713
  6. https://github.com/pulumi/pulumi/actions/runs/3227884171/jobs/5283427013
  7. https://github.com/pulumi/pulumi/actions/runs/3230504739/jobs/5289102856
  8. https://github.com/pulumi/pulumi/actions/runs/3230628127/jobs/5289317304
  9. https://github.com/pulumi/pulumi/actions/runs/3230757326/jobs/5289634185
  10. https://github.com/pulumi/pulumi/actions/runs/3230757326/jobs/5289636076
@IvanZosimov
Copy link
Contributor

Hi, @AaronFriel 👋 I see the problem in your repo, but unfortunately, I can't reproduce it. It was probably a temporary problem on the side of the MS .NET SDKs dist, our action relies on it and it explains why the installation of SDK for all OS was blocked.

@AaronFriel
Copy link
Author

@IvanZosimov Would it make sense to engineer around this or mirror their distribution info?

Would it be possible to add some logging to this action so that when net requests fail, the URL requested is logged?

@IvanZosimov
Copy link
Contributor

@AaronFriel , Sure! We are working on this issue now.

@rcdailey
Copy link

@joelwkall
Copy link

I'm also seeing this. Not sure if it helps, but this is the error:

Sun, 16 Oct 2022 19:17:43 GMT
Run actions/setup-dotnet@v2
  with:
    dotnet-version: 5.0.x
    include-prerelease: false

Sun, 16 Oct 2022 19:17:44 GMT
Error: read ECONNRESET

This is my workflow yml:

name: .NET

on:
  push:
    branches: [ "main" ]
  pull_request:
    branches: [ "main" ]

jobs:
  build:

    runs-on: windows-latest

    steps:
    - uses: actions/checkout@v3
    - name: Setup .NET
      uses: actions/setup-dotnet@v2
      with:
        dotnet-version: 5.0.x
    - name: Restore dependencies
      run: dotnet restore
    - name: Build
      run: dotnet build --no-restore
    - name: Test
      run: dotnet test --no-build --verbosity normal

@IvanZosimov
Copy link
Contributor

Hi, @rcdailey and @joelwkall, thanks for your comments. We are still in the process of investigation. You can upgrade the version of the setup-dotnet action to v3 because it's less influenced by the error.

@diogokiss
Copy link

I'm also quite often facing this issue using this GitHub Action. 😢

@MaksimZhukov
Copy link
Contributor

Hello everyone!
We have released new action version with a fix for this issue. Feel free to use the v3 tag.
Thank you for the patience!

@dmitry-shibanov
Copy link
Contributor

For now I'm going to close the issue. If you have any concerns feel free to ping us.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

7 participants