[codex] Add workflow concurrency controls#17
Merged
Conversation
Review Summary by QodoAdd workflow concurrency controls to prevent redundant runs
WalkthroughsDescription• Add concurrency controls to update-data.yml workflow to cancel redundant runs • Remove codex/** branch from validate.yml to prevent duplicate validation • Implement concurrency groups keyed by ref/PR number for both workflows Diagramflowchart LR
A["update-data.yml"] -->|"Add concurrency group"| B["Cancel in-progress runs"]
C["validate.yml"] -->|"Remove codex/** branch"| D["Stop duplicate validation"]
C -->|"Add concurrency group"| B
File Changes1. .github/workflows/update-data.yml
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Update NIST CMVP Dataworkflow so newer runs for the same ref cancel older in-progress runsValidaterunning on PRs and main pushes, but stop running the same validation twice forcodex/**branch pushesWhy
After merging the API validation/provenance PR, multiple full data-update workflows were still running on
mainat the same time. The live scrape can take a long time, so stacking redundant runs wastes Actions time and can produce confusing publish races.Validation
python3 validate_api.pygit diff --checkThis is workflow-only; the existing PR validation workflow will run after PR creation.