-
Notifications
You must be signed in to change notification settings - Fork 148
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
Update documents #786
Update documents #786
Conversation
toshipp
commented
Nov 20, 2023
- fix Update staled documents #296
- related: Implement Volume Snapshotting #124
2fc7f0f
to
19d9dee
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.
This is based on deploy/README.md
and user-manual.md
docs/faq.md
Outdated
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.
The contents are staled and moved to other appropriate places.
docs/node-maintenance.md
Outdated
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 is based on user-manual.md
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.
Comments so far.
I will continue my review. (Currently about 40% done.)
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 will continue my review. (Currently about 40% done.)
I finished my initial review.
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.
Almost LGTM. I can approve it after you fix my last comment and rebase the branch.
(I approved by mistake...)
@pluser |
Update documents to reflect latest TopoLVM behavior. Also reduce redundant text and re-organize these to make it easier to read them by users. Signed-off-by: Toshikuni Fukaya <toshikuni-fukaya@cybozu.co.jp>