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

Plugin upload workflow: Add 300s timeout to svn commit command #55621

Merged
merged 2 commits into from
Oct 25, 2023

Conversation

ockham
Copy link
Contributor

@ockham ockham commented Oct 25, 2023

What?

Add a 60s 300s timeout to the svn commit (and import) commands in the plugin upload workflow.

Fixes #55295.

Why?

The Gutenberg release workflow that uploads the plugin to the WordPress.org plugin repositoty frequently experiences a timeout that results in the relevant tag not being created (see here for a list of recent failures), requiring to be created manually instead.

How?

Some investigation seems to have revealed that this timeout might be due to some post-commit hooks running on the server side. To remediate this, this PR adds the http-timeout option to give the svn command more time.

Testing Instructions

Hard to test in isolation; I'd suggest we give it a try for the next GB release.

@ockham
Copy link
Contributor Author

ockham commented Oct 25, 2023

Looking at #55295 (comment), the recent svn commit commands all took up over 5 minutes (one of them even 10). I'm tempted to increase the timeout to 5 minutes to give a bit more leeway.

@ockham ockham changed the title Plugin upload workflow: Add 60s timeout to svn commit command Plugin upload workflow: Add 300s timeout to svn commit command Oct 25, 2023
@ockham ockham marked this pull request as ready for review October 25, 2023 12:54
@ockham
Copy link
Contributor Author

ockham commented Oct 25, 2023

I've increased the timeout to 5 minutes. I'd love to give this a shot for 16.9, which @justintadlock is going to release today. Anyone mind giving a quick rubber-stamp ✅ pls? 😊

@ockham ockham added the [Type] Build Tooling Issues or PRs related to build tooling label Oct 25, 2023
@ockham ockham added this to the Gutenberg 15.9 milestone Oct 25, 2023
@ockham ockham added the Backport to Gutenberg RC Pull request that needs to be backported to a Gutenberg release candidate (RC) label Oct 25, 2023
Copy link
Member

@mikachan mikachan left a comment

Choose a reason for hiding this comment

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

LGTM, based on my limited understanding of SVN and reading more about the http-timeout option here. I agree that it's worth trying this with the next release 😄

@ockham
Copy link
Contributor Author

ockham commented Oct 25, 2023

LGTM, based on my limited understanding of SVN and reading more about the http-timeout option here.

Thank you! Yeah, that's the docs I was perusing as well; I should've added a link!

I agree that it's worth trying this with the next release 😄

🤞
(If it fails, I'm around to help with manual tagging or whatever will be needed.)

@ockham ockham merged commit 0df2cc2 into trunk Oct 25, 2023
49 of 51 checks passed
@ockham ockham deleted the add/svn-upload-workflow-timeout branch October 25, 2023 13:08
@ockham
Copy link
Contributor Author

ockham commented Oct 25, 2023

Cherry-picked to the release/16.9 branch in 3a5a1a7.

@ockham ockham removed the Backport to Gutenberg RC Pull request that needs to be backported to a Gutenberg release candidate (RC) label Oct 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] Build Tooling Issues or PRs related to build tooling
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Tools: New tag for plugin release not created successfully on svn
2 participants