Skip to content

Commit 97b2a1a

Browse files
committed
ci: fix smoke test timeout flag
`harbor run` (jobs.py) only exposes `--agent-setup-timeout-multiplier`; the absolute `--agent-setup-timeout` flag only exists on `harbor trials`. Use the multiplier (2.0 = 720s from the 360s default) instead.
1 parent 6af8e53 commit 97b2a1a

1 file changed

Lines changed: 8 additions & 4 deletions

File tree

.github/workflows/smoke-test.yml

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -101,8 +101,12 @@ jobs:
101101
# curl|bash + apt install nodejs before the CLI even downloads, and on
102102
# the Blacksmith runners that occasionally pushes past 6 min when an apt
103103
# mirror or NodeSource cdn is slow, killing the run with
104-
# AgentSetupTimeoutError. 600s gives enough headroom for transient
105-
# mirror/cdn slowness while still finishing well under timeout-minutes.
104+
# AgentSetupTimeoutError. Doubling the multiplier to 2 (= 720s) gives
105+
# enough headroom for transient mirror/cdn slowness while still
106+
# finishing well under timeout-minutes.
107+
#
108+
# Note: `harbor run` only exposes --agent-setup-timeout-multiplier; the
109+
# absolute --agent-setup-timeout flag only exists on `harbor trials`.
106110
- name: Run smoke test — hello-world
107111
env:
108112
KILO_CLI_URL: ${{ steps.cli.outputs.cli_url }}
@@ -112,7 +116,7 @@ jobs:
112116
-m kilo/anthropic/claude-sonnet-4.6 \
113117
-d hello-world \
114118
--job-name smoke-test-hello-world \
115-
--agent-setup-timeout 600
119+
--agent-setup-timeout-multiplier 2
116120
117121
- name: Run smoke test — log-summary-date-ranges
118122
env:
@@ -124,7 +128,7 @@ jobs:
124128
-d terminal-bench-sample \
125129
-t "log-summary-date-ranges" \
126130
--job-name smoke-test-log-summary \
127-
--agent-setup-timeout 600
131+
--agent-setup-timeout-multiplier 2
128132
129133
- name: Validate results
130134
run: python3 scripts/validate_smoke_test.py jobs/smoke-test-*/

0 commit comments

Comments
 (0)