feat: add how-to for deploying Slurm using both the CLI and a bundle #5
Conversation
This file contains 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
- Starter pack recommends using .custom_wordlist for per-documentation modifications Signed-off-by: Jason C. Nucciarone <[email protected]>
Signed-off-by: Jason C. Nucciarone <[email protected]>
Signed-off-by: Jason C. Nucciarone <[email protected]>
Signed-off-by: Jason C. Nucciarone <[email protected]>
jedel1043
reviewed
Jul 31, 2024
jedel1043
reviewed
Jul 31, 2024
|
@jedel1043 thank you for the comments. General gist I'm getting is to omit the unnecessary. I'll wordsmith to make it more recipe-y like 🪨🔨 |
|
@jedel1043 R4R again. I simplified the sections that you highlighted and made things more dry, to-the-point. Let me know what you think of the revision! Page preview
|
- Remove unnecessary details from different sections. - Remove "Getting started" section. - Make important `virt-type` info before deployment commands. - Make blurb about Slurm being the only supported workload manager a note. - Various grammar fixes. Signed-off-by: Jason C. Nucciarone <[email protected]>
NucciTheBoss
force-pushed
the
howto-deploy-slurm
branch
from
691e19d to
481d8e5
Compare
|
I'm happy with this! I'll merge. At some point after we have a good base doc set we'll want to get one of the technical authors to formally review what we've wrote, but for now we be zooming. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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 PR adds the initial draft for how to deploy Slurm using both the Juju CLI and a bundle.
Misc.
.custom_wordlist.txtsince.wordlist.txtis directly from the documentation starter pack and should remain unmodified.Page preview