Skip to content

[CI] Increase job timeouts to 20 minutes#18740

Merged
TheMilek merged 1 commit intoSylius:2.3from
Rafikooo:ci/increase-timeout-to-20-minutes
Jan 22, 2026
Merged

[CI] Increase job timeouts to 20 minutes#18740
TheMilek merged 1 commit intoSylius:2.3from
Rafikooo:ci/increase-timeout-to-20-minutes

Conversation

@Rafikooo
Copy link
Copy Markdown
Contributor

@Rafikooo Rafikooo commented Jan 20, 2026

Increases timeout-minutes from 15 to 20 minutes across all CI workflow files to reduce job cancellations.

Ref: #18693

Summary by CodeRabbit

  • Chores
    • Extended timeout limits for continuous integration and deployment jobs from 15 to 20 minutes across multiple workflow configurations to allow additional execution time.

✏️ Tip: You can customize this high-level summary in your review settings.

@Rafikooo Rafikooo requested review from a team as code owners January 20, 2026 10:43
@probot-autolabeler probot-autolabeler bot added the Maintenance CI configurations, READMEs, releases, etc. label Jan 20, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Jan 20, 2026

📝 Walkthrough

Walkthrough

This pull request increases the timeout-minutes configuration from 15 to 20 across multiple GitHub Actions workflow files. The changes apply consistently to various CI/CD jobs including full builds, end-to-end tests, JavaScript tests, package tests, code standards checks, and pull request merges.

Changes

Cohort / File(s) Summary
Full CI Workflows
.github/workflows/ci__full_1_14.yaml, ci__full_2_1.yaml, ci__full_2_2.yaml
Increased timeout-minutes from 15 to 20 for "Notify about build status" job in each workflow
E2E Database Workflows
.github/workflows/ci_e2e-mariadb.yaml, ci_e2e-mysql.yaml, ci_e2e-pgsql.yaml
Increased timeout-minutes from 15 to 20 for both phpunit-cli-api and behat-ui jobs in each database-specific workflow
E2E Unstable Workflow
.github/workflows/ci_e2e-unstable.yaml
Increased timeout-minutes from 15 to 20 for three unstable E2E test jobs
JavaScript Workflows
.github/workflows/ci_js.yaml
Increased timeout-minutes from 15 to 20 for behat-js and behat-panther jobs
Package Workflows
.github/workflows/ci_packages.yaml, ci_packages-unstable.yaml
Increased timeout-minutes from 15 to 20 for test and test_unstable jobs respectively
Code Standards Workflow
.github/workflows/refactor.yaml
Increased timeout-minutes from 15 to 20 for coding-standard job
Pull Request Merge Workflow
.github/workflows/upmerge_pr.yaml
Increased timeout-minutes from 15 to 20 for upmerge job

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • Sylius/Sylius#18693: Related changes to GitHub Actions workflow timeout configurations across the same set of workflow files.

Suggested labels

CI

Suggested reviewers

  • Wojdylak
  • TheMilek

Poem

🐰 A rabbit's ode to patient workflows

Time was short, but now it's grown,
From fifteen minutes, now full-blown,
Five more for tests to see the way,
Our CI builds won't rush away! 🏃‍♂️⏰

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title '[CI] Increase job timeouts to 20 minutes' clearly and concisely summarizes the main change: increasing timeout values across CI workflows from 15 to 20 minutes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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


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.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Jan 20, 2026

❌ Preview Environment deleted from Bunnyshell

Available commands:

  • 🚀 /bns:deploy to redeploy the environment

@TheMilek TheMilek merged commit a98c30f into Sylius:2.3 Jan 22, 2026
36 checks passed
@NoResponseMate
Copy link
Copy Markdown
Contributor

@TheMilek We should backport this to 2.2 as well

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Maintenance CI configurations, READMEs, releases, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants