Skip to content

Have the release script warn if a workflow is queued for >15m #19084

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

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from
+11 −0
Open

Have the release script warn if a workflow is queued for >15m #19084

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
889ffd9
Have the release script warn if a workflow is queued for >15m
anoadragon453 Oct 21, 2025
7cd9678
newsfile
anoadragon453 Oct 21, 2025
b5c66de
Allow continuing on despite queued assets
anoadragon453 Oct 24, 2025
9cd8166
lint
anoadragon453 Oct 24, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions changelog.d/19084.misc
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Warn the developer when they are releasing Synapse if a release workflow has been queued for over 15 minutes.
10 changes: 10 additions & 0 deletions scripts-dev/release.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -596,6 +596,16 @@ def _wait_for_actions(gh_token: Optional[str]) -> None:
if len(resp["workflow_runs"]) == 0:
continue

# Warn the user if any workflows are still queued. They might need to fix something.
if any(workflow["status"] == "queued" for workflow in resp["workflow_runs"]):
_notify("Warning: at least one release workflow is still queued...")
Comment on lines +600 to +601
Copy link
Contributor

Choose a reason for hiding this comment

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

In the case of the macOS runners not working for that whole day, we'd see this warning every 5 minutes and asked to continue.

I assume that's fine as at that point you would be monitoring the actions and continue once you see them done and not pressing continue on this script over and over.

if not click.confirm("Continue waiting for queued assets?", default=True):
click.echo(
"Continuing on with the release. Note that you may need to upload missing assets manually later."
)
break
continue

if all(
workflow["status"] != "in_progress" for workflow in resp["workflow_runs"]
):
Expand Down