Skip to content

Increase artifact build timeout for windows-C##26179

Merged
drfloob merged 1 commit intogrpc:masterfrom
drfloob:increase-csharp-artifact-build-timeout
May 6, 2021
Merged

Increase artifact build timeout for windows-C##26179
drfloob merged 1 commit intogrpc:masterfrom
drfloob:increase-csharp-artifact-build-timeout

Conversation

@drfloob
Copy link
Copy Markdown
Member

@drfloob drfloob commented May 5, 2021

The 30min timeout has caused the multiarch artifact build to fail twice
in the past week. See
http://sponge2/55f7f6f8-d3c0-4b6e-91c8-c0c316f4717b
@drfloob drfloob added lang/Python area/build release notes: no Indicates if PR should not be in release notes labels May 5, 2021
@drfloob drfloob requested a review from jtattermusch May 5, 2021 19:03
Copy link
Copy Markdown
Contributor

@jtattermusch jtattermusch left a comment

Choose a reason for hiding this comment

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

LGTM. It's sad the the artifact builds are so slow.

@drfloob
Copy link
Copy Markdown
Member Author

drfloob commented May 6, 2021

It is a shame. In my ~6 months on this team I think I've had to increase the build timeouts 3 times now for various reasons. If it continues, we may want to prioritize making some improvements here.

Thanks for the review.

@drfloob drfloob merged commit 24e9a69 into grpc:master May 6, 2021
paulosjca pushed a commit to paulosjca/grpc that referenced this pull request May 7, 2021
The 30min timeout has caused the multiarch artifact build to fail twice
in the past week. See
http://sponge2/55f7f6f8-d3c0-4b6e-91c8-c0c316f4717b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/build lang/Python release notes: no Indicates if PR should not be in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants