mirror of
https://github.com/kevinveenbirkenbach/computer-playbook.git
synced 2026-11-06 10:59:59 +00:00
fix(ci-trigger): name the inputs a retrigger actually reproduces
CONFIG_INPUTS lost modes, sequencing and mode_fail_fast when the deploy modes collapsed into one chunked sweep, but two mirrors of it did not follow. The --run help text enumerates what a retrigger carries over, so it was telling the operator about three inputs the dispatch no longer has. The job-log fixture described a run recording mode_fail_fast, a state the dispatch can no longer produce, which left the log half of config_from_run asserting on an input nobody writes. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This commit is contained in:
1 parent
4077480c17
commit
ce2db79ee5
2 files changed
+3
-3
No files matched your search
@@ -63,7 +63,7 @@ def main(argv: list[str] | None = None) -> int:
|
||||
metavar="URL|ID",
|
||||
help=(
|
||||
"Run URL or bare run id to reproduce: its configuration (distros, "
|
||||
"modes, lifecycles, filesystem, sequencing, mode_fail_fast, "
|
||||
"mode, lifecycles, filesystem, chunk_gate, "
|
||||
"workspace, instructions) is carried over, and with --failed its "
|
||||
"results also pick the apps. A bare id resolves against the current "
|
||||
"branch's repo. With --failed and no --run: the latest deploy run "
|
||||
|
||||
@@ -230,7 +230,7 @@ class TestInputsFromJobs(unittest.TestCase):
|
||||
"2026-08-08T10:22:59.5Z whitelist: ",
|
||||
"2026-08-08T10:22:59.5Z priority: "
|
||||
+ " ".join(f"web-app-{n}" for n in range(69)),
|
||||
"2026-08-08T10:22:59.5Z mode_fail_fast: true",
|
||||
"2026-08-08T10:22:59.5Z chunk_gate: true",
|
||||
"2026-08-08T10:22:59.5Z ##[endgroup]",
|
||||
"2026-08-08T10:22:59.6Z ##[group]Run echo hi",
|
||||
"2026-08-08T10:22:59.6Z priority: not-an-input",
|
||||
@@ -259,7 +259,7 @@ class TestInputsFromJobs(unittest.TestCase):
|
||||
self.assertEqual(len(inputs["priority"].split()), 69)
|
||||
self.assertEqual(inputs["distros"], "arch")
|
||||
self.assertEqual(inputs["whitelist"], "")
|
||||
self.assertEqual(inputs["mode_fail_fast"], "true")
|
||||
self.assertEqual(inputs["chunk_gate"], "true")
|
||||
|
||||
def test_a_run_without_a_called_job_reads_nothing(self) -> None:
|
||||
self.assertEqual(runs.inputs_from_jobs([{"name": "🎲 Pick"}], "o/r"), {})
|
||||
|
||||
Reference in new issue
Block a user