Skip to content
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

Extend HUB CI timeout from 90 min to 240 min #287

Merged
merged 2 commits into from
Jun 1, 2023

Conversation

glenn-jocher
Copy link
Member

@glenn-jocher glenn-jocher commented May 31, 2023

@kalenmike @sergiuwaxmann @AyushExel I saw that since the CI timed-out after 90 min we did not get a Slack error message, so I've extended the timeout to 240 min, which should be long enough to fail all export formats and then submit an error message to Slack.

copilot:all

🛠️ PR Summary

Made with ❤️ by Ultralytics Actions

🌟 Summary

Reduced CI timeout and expanded failure notifications for the Ultralytics repository.

📊 Key Changes

  • 🕒 CI timeout has been decreased from 90 minutes to 60 minutes.
  • 🔔 Failure notification condition updated; now includes both failure() and cancelled() events for certain triggers.

🎯 Purpose & Impact

  • 🚀 Decreasing the timeout is likely to make the CI pipeline more efficient by freeing up resources sooner for cases where jobs may not need the full 90 minutes.
  • 📢 Updating the notification condition to include cancelled() events ensures that maintainers are informed not just about process failures, but also about any cancellations which could highlight issues with the CI workflow or manual interventions.
  • 👁️‍🗨️ Improving automation and monitoring can potentially lead to quicker identification and resolution of issues, streamlining the development process for all contributors.

@glenn-jocher glenn-jocher merged commit 76eb2d0 into master Jun 1, 2023
@glenn-jocher glenn-jocher deleted the glenn-jocher-patch-1 branch June 1, 2023 08: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.

1 participant