ci: migrate release workflows to ci-infra runners#2467
ci: migrate release workflows to ci-infra runners#2467yzh119 merged 1 commit intoflashinfer-ai:mainfrom
Conversation
|
Note Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported. |
📝 WalkthroughWalkthroughGitHub Actions workflow files are updated to standardize runner selectors across three workflows. Runner specifications are expanded to include explicit labels (linux, cpu, on-demand) and replace platform identifiers with consistent architecture mapping (arm64 for aarch64, x64 otherwise) for improved self-hosted runner targeting and scheduling. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches
🧪 Generate unit tests (beta)
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. Comment |
|
Hi @yongwww can you explain how we trigger CI after this change? |
The CI triggering mechanism remains the same, no changes needed on the workflow trigger side. When any workflow is triggered (push, PR, schedule, or manual dispatch), GitHub automatically sends a workflow_job webhook event when jobs are queued. The ci-infra system receives this webhook via API Gateway → Lambda, which then launches an ephemeral EC2 runner on-demand to execute the job. |
|
@yongwww thanks for explaination. |
|
@flashinfer-bot run |
<!-- .github/pull_request_template.md --> ## 📌 Description <!-- What does this PR do? Briefly describe the changes and why they’re needed. --> Migrate self-hosted runner labels from the old always-on EC2 instances to the new ephemeral runners managed by ci-infra. ## 🔍 Related Issues <!-- Link any related issues here --> ## 🚀 Pull Request Checklist Thank you for contributing to FlashInfer! Before we review your pull request, please make sure the following items are complete. ### ✅ Pre-commit Checks - [x] I have installed `pre-commit` by running `pip install pre-commit` (or used your preferred method). - [x] I have installed the hooks with `pre-commit install`. - [x] I have run the hooks manually with `pre-commit run --all-files` and fixed any reported issues. > If you are unsure about how to set up `pre-commit`, see [the pre-commit documentation](https://pre-commit.com/). ## 🧪 Tests - [x] Tests have been added or updated as needed. - [x] All tests are passing (`unittest`, etc.). ## Reviewer Notes <!-- Optional: anything you'd like reviewers to focus on, concerns, etc. --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Chores** * Updated CI/CD pipeline configurations to optimize resource allocation and execution environments across multiple release workflows, improving build and test scheduling efficiency. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
📌 Description
Migrate self-hosted runner labels from the old always-on EC2 instances to the new ephemeral runners managed by ci-infra.
🔍 Related Issues
🚀 Pull Request Checklist
Thank you for contributing to FlashInfer! Before we review your pull request, please make sure the following items are complete.
✅ Pre-commit Checks
pre-commitby runningpip install pre-commit(or used your preferred method).pre-commit install.pre-commit run --all-filesand fixed any reported issues.🧪 Tests
unittest, etc.).Reviewer Notes
Summary by CodeRabbit