Skip to content

Increased rvc2 test workflow's timeout#1823

Merged
moratom merged 1 commit into
developfrom
fix/workflow_rvc2_timeout
Jun 1, 2026
Merged

Increased rvc2 test workflow's timeout#1823
moratom merged 1 commit into
developfrom
fix/workflow_rvc2_timeout

Conversation

@aljazdu

@aljazdu aljazdu commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

Purpose

Increase the timeout length for the self-hosted rvc2 testing job.

Specification

None / not applicable

Dependencies & Potential Impact

None / not applicable

Deployment Plan

None / not applicable

Testing & Validation

None / not applicable

AI Usage

Assisted-by: AGENT_NAME:MODEL_VERSION [TOOL1] [TOOL2]

Submitted code was reviewed by a human: YES/NO

The author is taking the responsibility for the contribution: YES/NO

Summary by CodeRabbit

  • Chores
    • Updated timeout configuration to allow for extended processing time in automated workflows.

@coderabbitai

coderabbitai Bot commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 56d5f8f3-d894-4d1a-9a52-fdc1f354850d

📥 Commits

Reviewing files that changed from the base of the PR and between 399607a and e8339df.

📒 Files selected for processing (1)
  • .github/workflows/test_child.yml
📜 Recent review details
🔇 Additional comments (1)
.github/workflows/test_child.yml (1)

19-19: LGTM!


📝 Walkthrough

Walkthrough

The CI workflow configuration in .github/workflows/test_child.yml updates the default timeout for the RVC2 (recompiled vertex chain) processing step from 360 minutes (6 hours) to 720 minutes (12 hours).

Changes

RVC2 timeout configuration

Layer / File(s) Summary
RVC2 timeout default update
.github/workflows/test_child.yml
The workflow_call input rvc2_timeout_minutes default value is doubled from 360 to 720 minutes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 Six hours seemed quite snug,
But twelve's the new hug,
For vertices compiled long,
A patient job, brave and strong!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title directly and clearly summarizes the main change: increasing the timeout for the rvc2 test workflow from 360 to 720 minutes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/workflow_rvc2_timeout

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@aljazdu aljazdu requested a review from moratom June 1, 2026 07:37
@aljazdu aljazdu self-assigned this Jun 1, 2026
@moratom moratom merged commit 314a2c4 into develop Jun 1, 2026
32 checks passed
@moratom moratom deleted the fix/workflow_rvc2_timeout branch June 1, 2026 12:11
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