Skip to content

[iris-cw-ci] Bump integration pipeline timeouts to 30m/15m#5125

Merged
rjpower merged 1 commit intomainfrom
claude/jolly-wilson-85bb09
Apr 23, 2026
Merged

[iris-cw-ci] Bump integration pipeline timeouts to 30m/15m#5125
rjpower merged 1 commit intomainfrom
claude/jolly-wilson-85bb09

Conversation

@rjpower
Copy link
Copy Markdown
Collaborator

@rjpower rjpower commented Apr 23, 2026

The full marin pipeline test runs 8 sequential Iris sub-jobs, each paying ~60-90s of pod startup + dep sync overhead. On the last CoreWeave CI run, train_lm completed just 34s after pytest-timeout fired at 600s. Bump the shell wrapper from 900s to 1800s and pytest-timeout from 600s to 900s.

The full marin pipeline test runs 8 sequential Iris sub-jobs, each
paying ~60-90s of pod startup + dep sync overhead. On the last run
train_lm completed just 34s after pytest-timeout fired. Bump the shell
wrapper from 900s to 1800s and pytest-timeout from 600s to 900s.
@rjpower rjpower added the agent-generated Created by automation/agent label Apr 23, 2026
@rjpower rjpower requested a review from yonromai April 23, 2026 16:41
@rjpower rjpower enabled auto-merge (squash) April 23, 2026 16:41
@claude
Copy link
Copy Markdown
Contributor

claude Bot commented Apr 23, 2026

Claude finished @rjpower's task in 1m 23s —— View job


Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

This is a trivial CI timeout bump in a single workflow file with clear justification in the PR description. The new values preserve a sensible ordering: timeout 1800 (shell wrapper) > --timeout=900 (pytest), so pytest hits its own timeout and emits diagnostic output before the shell sends SIGKILL. The buffer between the two grew from 300s → 900s, which is a strict improvement.

  • Check if PR needs review
  • Verify timeout ordering is sane
  • Post review summary
    Branch

@rjpower rjpower merged commit 8ef6073 into main Apr 23, 2026
35 checks passed
@rjpower rjpower deleted the claude/jolly-wilson-85bb09 branch April 23, 2026 17:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent-generated Created by automation/agent

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants