-
Notifications
You must be signed in to change notification settings - Fork 696
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
[Docs] Add Slurm agent setup #6231
[Docs] Add Slurm agent setup #6231
Conversation
Signed-off-by: JiangJiaWei1103 <[email protected]>
Code Review Agent Run Status
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #6231 +/- ##
==========================================
+ Coverage 36.87% 36.88% +0.01%
==========================================
Files 1318 1318
Lines 134647 134829 +182
==========================================
+ Hits 49647 49728 +81
- Misses 80679 80767 +88
- Partials 4321 4334 +13
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Signed-off-by: JiangJiaWei1103 <[email protected]>
Code Review Agent Run Status
|
Signed-off-by: JiangJiaWei1103 <[email protected]>
Code Review Agent Run Status
|
Signed-off-by: JiangJiaWei1103 <[email protected]>
Signed-off-by: JiangJiaWei1103 <[email protected]>
Code Review Agent Run Status
|
Signed-off-by: JiangJiaWei1103 <[email protected]>
Code Review Agent Run Status
|
Signed-off-by: Future-Outlier <[email protected]>
Signed-off-by: Future-Outlier <[email protected]>
Signed-off-by: Future-Outlier <[email protected]>
Code Review Agent Run #9a6330Actionable Suggestions - 0Review Details
|
Changelist by BitoThis pull request implements the following key changes.
|
Tracking issue
Issue: #3936
Related to flyteorg/flytekit#3005
Why are the changes needed?
Provide users with a step-by-step guide to setting up the Slurm agent from scratch.
What changes were proposed in this pull request?
Add documentation on setting up the Slurm agent for both local testing and deployment. The documentation includes following sections:
How was this patch tested?
Labels
Setup process
Screenshots
Check all the applicable boxes
Related PRs
flyteorg/flytekit#3005
Docs link
https://flyte--6231.org.readthedocs.build/en/6231/deployment/agents/slurm.html
Summary by Bito
Documentation enhancement PR focused on Slurm agent setup and configuration in Flyte. Provides detailed instructions for cluster setup, SSH configuration, AWS S3 integration, and deployment scenarios. Includes both local testing and production deployment guidelines. Minor updates to Databricks agent documentation included.Unit tests added: False
Estimated effort to review (1-5, lower is better): 2