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
[enterprise-4.14] OCPBUGS#33113: Doc improvements related to volume cloning/snapshots #75330
[enterprise-4.14] OCPBUGS#33113: Doc improvements related to volume cloning/snapshots #75330
Conversation
a9be771
to
e92c11e
Compare
🤖 Wed May 08 12:08:44 - Prow CI generated the docs preview: https://75330--ocpdocs-pr.netlify.app/ |
@jakobmoellerdev @suleymanakbas91 |
/lgtm |
@radeore |
Hi @sr1kar99 , added few comments, other then that LGTM. Thanks! |
e92c11e
to
cff81b3
Compare
New changes are detected. LGTM label has been removed. |
/label peer-review-needed |
|
||
//Volume cloning | ||
include::modules/lvms-volume-clones-in-single-node-openshift.adoc[leveloffset=+1] | ||
include::modules/lvms-creating-volume-clones-in-single-node-openshift.adoc[leveloffset=+2] |
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.
FYI, there is one existing external link
in the openshift-docs repo to this file on the Customer Portal, from a Microshift assembly:
It won't 404 (it just dumps to the top of the page on the Portal), but you might want to update that link:
to the new module location / URL. Possibly cross-check with a member of the Microshift docs team.
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 a great catch! Thanks @adellape
I've informed a writer from the MicroShift team. This link will be updated upon merging this PR.
Thanks!
Hi @adellape Thanks for adding the 4.15+ PR link. :) |
cff81b3
to
5332e10
Compare
5332e10
to
3e0c724
Compare
@sr1kar99: all tests passed! Full PR test history. Your PR dashboard. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
/label merge-review-needed |
Version(s):
4.14
Issue:
OCPBUGS-33113
Link to docs preview:
Preview
QE review: