Adding Managed Lustre to Cluster Toolkit#3950
Merged
Merged
Conversation
c2b94b8 to
b4a9290
Compare
b4a9290 to
243e682
Compare
0ffbd2b to
8d1bf3c
Compare
f2f2b02 to
757a5a4
Compare
757a5a4 to
4113cae
Compare
2a0fcf2 to
2606cc3
Compare
samskillman
reviewed
Apr 18, 2025
harshthakkar01
previously approved these changes
Apr 18, 2025
3a67be1 to
59b5a61
Compare
harshthakkar01
previously approved these changes
Apr 18, 2025
Addressing PR comments. Co-authored-by: Sam Skillman <samskillman@gmail.com>
samskillman
approved these changes
Apr 18, 2025
harshthakkar01
approved these changes
Apr 18, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This creates a Managed Lustre module to the Cluster Toolkit. It follows a lot of the Parallelstore structure.
This builds on top of this PR
Tested on several vm-instance deployments
Tested on Slurm