[Draft] 3006.x: rename nightly-stress-test.yml to run-stress-test.yml, add branch input - #70089
Closed
charzl wants to merge 1 commit into
Closed
[Draft] 3006.x: rename nightly-stress-test.yml to run-stress-test.yml, add branch input#70089charzl wants to merge 1 commit into
charzl wants to merge 1 commit into
Conversation
Drops the schedule: trigger (dead on this non-default branch anyway -- GitHub only honors schedule: off the repo's default branch, so it never fired on this branch anyway) and adds a required branch input (default 3006.x) so this can be invoked with --ref 3006.x from the dispatcher in saltstack/salt's nightly-stress-test.yml (VCOPS-100029's nightly coverage) as well as manually against any branch/PR diff. Branch-qualifies the Docker cache key, uploaded artifact name, and stress-snapshots run dir with inputs.branch, matching master's and 3008.x's copies of this file.
charzl
changed the title
Collaborator
Author
|
Closing — went with Option A (single source of truth on master, dispatched via --ref master + inputs.branch) instead of per-branch copies. This branch doesn't need its own copy of run-stress-test.yml; see #70087. |
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
1 participant
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
Part of VCOPS-100029 (run a stress test against a specific PR/branch on demand). Companion to #70087 (master) and #70088 (3008.x) -- the master dispatcher fires this branch's own copy of the (renamed) workflow nightly via
workflow_dispatch --ref 3006.x, matching the--ref matrix.branchpattern already used byrun-nightly.yml'strigger-branch-nightly-buildsjob.nightly-stress-test.yml->run-stress-test.yml(drops the deadschedule:trigger -- GitHub only honorsschedule:off the repo's default branch, so it never fired on this branch anyway).branchinput (default3006.x), used tocheckoutthe actual code under test -- lets this be invoked against an arbitrary branch/PR, not just this one.stress-snapshotsrun dir withinputs.branch, matching master's and 3008.x's copies of this file.Test plan
pre-commithooks pass.workflow_dispatch(manually, and via the master dispatcher once [DRAFT]Dispatch nightly-stress-test.yml per-branch (master/3006.x/3008.x), each against its own --ref #70087 merges) runs successfully against this branch.