-
Notifications
You must be signed in to change notification settings - Fork 6
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
feat: add docs for how to deploy the filesystem client #17
feat: add docs for how to deploy the filesystem client #17
Conversation
6acd2c1
to
757f924
Compare
I think this is on a good enough state for review. |
757f924
to
7cd70b7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall content looks great! Provided a few grammar changes and some organization suggestions.
To add the charm that takes care of mounting a shared filesystem to the cluster nodes, | ||
deploy the `filesystem-client` charm: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To add the charm that takes care of mounting a shared filesystem to the cluster nodes, | |
deploy the `filesystem-client` charm: | |
To add the `filesystem-client` charm, which mounts a shared filesystem to the cluster nodes: |
7cd70b7
to
36faade
Compare
6a6a55c
to
ad4e220
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! The change away from the second set of tabs helps. One small grammar edit and note on the public cloud links.
- Amazon Web Services | ||
- See [Create an NFS file share using the default configuration][aws-nfs] for general set up information. | ||
- Azure | ||
- See [Tutorial: Create an NFS Azure file share and mount it on a Linux VM using the Azure portal][azure-nfs] | ||
for general set up information. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This could probably be formatted more cleanly but I'm not quite sure what that would look like - I'll think about it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Made some adjustments to use clickable cards instead of a bullet list. Let me know if this looks better!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like the cards! I might see if we can tweak the background color so they stand out a bit more (darn dark mode), but they're functional as is.
ad4e220
to
4cf536d
Compare
4cf536d
to
c11a41b
Compare
No description provided.