Skip to content

Increase build timeout from 10 to 15 minutes - #106

Merged
pshnghng0318 merged 3 commits into
devfrom
pshnghng0318-patch-1
Oct 5, 2026
Merged

pshnghng0318 merged 3 commits into
devfrom
pshnghng0318-patch-1

Conversation

@pshnghng0318

Copy link
Copy Markdown
Contributor

Description

Increasing build timeout limit from 10 to 15 min to allow virtual machine to build the backend with fewer resources

Checklist

For the pull request:

  • no documentation changes are needed

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟡 Changes recommended

The parallel ICD workflows still retain the old 10-minute timeout.

Review effort: Lite
Findings: 1 Medium severity

Open (1)
What changed in this PR

Raises the backend build timeout from 10 to 15 minutes in the primary ICD test workflow to support lower-resource virtual machines.

Changes:

  • Updates the Build job timeout in .github/workflows/icd_tests.yml.
File Summary
.github/​workflows/​icd_tests.yml Raises the build timeout to 15 minutes. Parallel ICD workflows still use 10 minutes and should be updated or excluded from the PR scope.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

name: Build ${{ matrix.os_version }}
runs-on: ${{ matrix.runner }}
timeout-minutes: 10
timeout-minutes: 15

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Increasing timeout in all-stages and single-stage tests

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot review overview

🟢 Approval recommended

All reviewed changes consistently update the intended build timeouts with no unresolved issues.

Review effort: Lite
Findings: 1 Medium severity

Open (1)

@pshnghng0318
pshnghng0318 merged commit 33027e1 into dev Oct 5, 2026
1 check passed
@pshnghng0318
pshnghng0318 deleted the pshnghng0318-patch-1 branch October 5, 2026 14:00
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