-
Notifications
You must be signed in to change notification settings - Fork 341
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ci: optimize workflow timeouts based on timing analysis #3481
Conversation
- playwright.yml: reduce timeout from 16min to 13min (85% buffer) - test_cli.yaml: - build_wheel: reduce from 10min to 3min (200% buffer) - test_cli: reduce from 15min to 10min (122% buffer) - test_examples: reduce from 15min to 3min (500% buffer) Changes based on analysis of recent workflow runs to prevent consuming unnecessary minutes while maintaining sufficient buffers. Co-Authored-By: Myles Scolnick <myles@marimo.io>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
⚙️ Control Options:
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Update comment format to show average and max run times with date stamp instead of showing what the timeout was reduced from. Co-Authored-By: Myles Scolnick <myles@marimo.io>
CI Failure InvestigationI've investigated the failing test and determined it's unrelated to the workflow timeout changes in this PR. Failed Test Details
Why This Is Unrelated
Test Results Context
This appears to be a pre-existing flaky test issue where the stderr output occasionally doesn't appear within the expected timeframe. Our workflow timeout changes don't affect these internal test timeouts. |
🚀 Development release published. You may be able to view the changes at https://marimo.app?v=0.10.14-dev30 |
Optimize GitHub Actions Workflow Timeouts
This PR optimizes workflow timeouts based on analysis of recent workflow runs to prevent consuming unnecessary minutes while maintaining sufficient buffers.
Changes
playwright.yml
test_cli.yaml
build_wheel job:
test_cli job:
test_examples job:
Testing & Analysis
Changes are based on analysis of recent successful workflow runs using the GitHub API. All new timeouts maintain more than the requested 50-75% buffer over maximum observed runtimes to ensure reliability while preventing unnecessary resource consumption.
Link to Devin run: https://app.devin.ai/sessions/e653e561a2e648a581431a0a457d2ec6