Skip to content

ci: fix smoke test timeout flag#9983

Merged
lambertjosh merged 1 commit into
mainfrom
ci/smoke-test-setup-timeout
May 7, 2026
Merged

ci: fix smoke test timeout flag#9983
lambertjosh merged 1 commit into
mainfrom
ci/smoke-test-setup-timeout

Conversation

@lambertjosh

@lambertjosh lambertjosh commented May 6, 2026

Copy link
Copy Markdown
Contributor

Summary

Follow-up to #9941. Fixes timeout flag to use the one available in currently pinned version of harbor run

@kilo-code-bot

kilo-code-bot Bot commented May 6, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (1 files)
  • .github/workflows/smoke-test.yml

Reviewed by gpt-5.5-20260423 · 219,716 tokens

@lambertjosh lambertjosh force-pushed the ci/smoke-test-setup-timeout branch 2 times, most recently from 97b2a1a to d304e41 Compare May 6, 2026 22:02
`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.
@lambertjosh lambertjosh force-pushed the ci/smoke-test-setup-timeout branch from d304e41 to eb8b3de Compare May 6, 2026 22:07
@lambertjosh

Copy link
Copy Markdown
Contributor Author

@lambertjosh lambertjosh merged commit 0722158 into main May 7, 2026
12 checks passed
@lambertjosh lambertjosh deleted the ci/smoke-test-setup-timeout branch May 7, 2026 10:52
jliounis pushed a commit to jliounis/kilocode that referenced this pull request May 19, 2026
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