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.

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

Jump to bottom

Test flake due to connection reset in setup actions #10997

Closed
AaronFriel opened this issue Oct 11, 2022 · 3 comments · Fixed by #11001
Closed

Test flake due to connection reset in setup actions #10997

AaronFriel opened this issue Oct 11, 2022 · 3 comments · Fixed by #11001
Assignees
@AaronFriel
Labels
area/build CI/CD for pulumi/pulumi kind/bug Some behavior is incorrect or out of spec p1 Bugs severe enough to be the next item assigned to an engineer resolution/fixed This issue was fixed
Milestone
0.79

Comments

@AaronFriel
Copy link
Member

AaronFriel commented Oct 11, 2022 •
edited

Test such as this run fail in Set up DotNet 6.0.x:

Run actions/setup-dotnet@v2
Error: read ECONNRESET

This is a tracking issue for us to correlate these errors and make upstream issue(s) as needed.

This problem appears to be localized to the actions/setup-dotnet@v2 action.

  • 14 out of 20 of the most recent merge jobs failed.
  • Ten failures were in the Set up DotNet 6.0.x job, and this occurred on all 3 platforms (windows, ubuntu, macos).
  • One failure was due to a different network issue (pulumictl ETIMEDOUT)
  • Three due to an issue with the PR.

Upstream issue: actions/setup-dotnet#333
@AaronFriel AaronFriel self-assigned this Oct 11, 2022
@AaronFriel AaronFriel added kind/bug Some behavior is incorrect or out of spec area/build CI/CD for pulumi/pulumi p1 Bugs severe enough to be the next item assigned to an engineer labels Oct 11, 2022
@AaronFriel AaronFriel added this to the 0.79 milestone Oct 11, 2022
@kpitzen
Copy link
Contributor

kpitzen commented Oct 11, 2022

https://github.com/pulumi/pulumi/actions/runs/3230628127/jobs/5289317304

This was referenced Oct 11, 2022
Closed
Merged
@kpitzen
Copy link
Contributor

kpitzen commented Oct 12, 2022

https://github.com/pulumi/pulumi/actions/runs/3231178910/jobs/5290629115

@kpitzen
Copy link
Contributor

kpitzen commented Oct 12, 2022

https://github.com/pulumi/pulumi/actions/runs/3231404553/jobs/5290948152

@bors bors bot closed this as completed in 7978eed Oct 12, 2022
@pulumi-bot pulumi-bot added the resolution/fixed This issue was fixed label Oct 12, 2022
@AaronFriel AaronFriel mentioned this issue Nov 11, 2022
Merged
bors bot added a commit that referenced this issue Nov 11, 2022
@bors @AaronFriel
Merge #11337
5c1c462 
11337: ci: Re-enable version sets for .NET, test GA version of 6 and 7 r=AaronFriel a=AaronFriel

The `actions/setup-dotnet` action published v3.0.3 two weeks ago addressing the ECONNRESET issues discovered in #10997.

Reverts #11001

Resolves #11000

Co-authored-by: Aaron Friel <mayreply@aaronfriel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AaronFriel AaronFriel

Labels
area/build CI/CD for pulumi/pulumi kind/bug Some behavior is incorrect or out of spec p1 Bugs severe enough to be the next item assigned to an engineer resolution/fixed This issue was fixed
Projects
None yet
Milestone
0.79
Development

Successfully merging a pull request may close this issue.

fix(ci): Disable setup-dotnet action, rely on globally installed dotnet pulumi/pulumi
3 participants
@AaronFriel @kpitzen @pulumi-bot