Skip to content

Move WP refresh concurrency group to workflow level#3380

Merged
brandonpayton merged 2 commits intotrunkfrom
fix/wp-refresh-concurrency-guard
Mar 12, 2026
Merged

Move WP refresh concurrency group to workflow level#3380
brandonpayton merged 2 commits intotrunkfrom
fix/wp-refresh-concurrency-guard

Conversation

@brandonpayton
Copy link
Copy Markdown
Member

@brandonpayton brandonpayton commented Mar 12, 2026

Summary

  • Moves the concurrency group from job-level to workflow-level in the "Refresh WordPress Major&Beta" workflow
  • Adds a timeout-minutes: 120 to prevent indefinitely stuck runs

Motivation

The job-level concurrency guard was not preventing concurrent workflow runs. Evidence:

By moving the concurrency group to the workflow level, GitHub Actions will queue pending runs before any jobs start, ensuring only one instance runs at a time.

Test plan

  • Verify the workflow YAML is valid
  • Confirm only one workflow run executes at a time after merging (observable via next scheduled trigger)

The job-level concurrency guard was not preventing concurrent runs.
Evidence: run 23010346814 started at 11:39am and ran for 40 minutes,
while run 23011564322 started 27 minutes later at 12:06pm and ran
concurrently.

Moving the concurrency group to the workflow level ensures GitHub
Actions queues workflow runs before any jobs start. Also adds a
2-hour timeout to prevent indefinitely stuck runs.
@brandonpayton brandonpayton requested review from a team, ashfame and Copilot March 12, 2026 20:26
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Moves GitHub Actions concurrency control for the “Refresh WordPress Major&Beta” workflow from a single job to the workflow level, and adds a job timeout to avoid stuck runs.

Changes:

  • Add workflow-level concurrency to serialize entire workflow runs
  • Remove job-level concurrency
  • Add timeout-minutes: 120 to the workflow job

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@brandonpayton brandonpayton merged commit d9ae3a5 into trunk Mar 12, 2026
42 checks passed
@brandonpayton brandonpayton deleted the fix/wp-refresh-concurrency-guard branch March 12, 2026 20:42
bgrgicak pushed a commit that referenced this pull request Mar 27, 2026
## Summary

- Moves the `concurrency` group from job-level to workflow-level in the
"Refresh WordPress Major&Beta" workflow
- Adds a `timeout-minutes: 120` to prevent indefinitely stuck runs

## Motivation

The job-level concurrency guard was not preventing concurrent workflow
runs. Evidence:

- [Run
23010346814](https://github.com/WordPress/wordpress-playground/actions/runs/23010346814)
started at 11:39am and ran for ~40 minutes
- [Run
23011564322](https://github.com/WordPress/wordpress-playground/actions/runs/23011564322)
started 27 minutes later at 12:06pm and ran concurrently

By moving the concurrency group to the workflow level, GitHub Actions
will queue pending runs before any jobs start, ensuring only one
instance runs at a time.

## Test plan

- [x] Verify the workflow YAML is valid
- [ ] Confirm only one workflow run executes at a time after merging
(observable via next scheduled trigger)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants