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
Document volumes and disks #13
Conversation
835828f
to
53344f8
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 good in general, some minor nits.
The consistency i.e. emptry structs and vms could be done in a separate follow up and then also cover the complete guide.
disks-and-volumes.md
Outdated
|
||
### lun | ||
|
||
A `lun` disk will expose the volume as a LUN device to the vm. |
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.
Could you link to a definition of lun
- especially in contrast to disks.
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.
Not sure where to link. The difference is what you see in the guest. Do you mean linking to the api-reference?
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.
IIUIC then the diffrerence between lun and disk is that with lun more scsi calls are passed through disk is somewhat filtering it - that's at least what I get from the libvirt docs.
Is there any other difference I am missing?
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 can add to the api-reference docs that this allows generic iscsi command passthrough.
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.
Yep, that would work for me, or even just a sentence here differentiating lun from disk. At least some hint how they differ :)
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.
done
- name: mypvcdisk | ||
volumeName: mypvc | ||
# This makes it a lun device | ||
lun: {} |
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 empty structs for - empty structs :)
(lun: {} vs lun:)
I'll see that I have an eye on this in examples in general.
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.
lun:
is equivalent to lun: null
, whereas lun: {}
is an empty struct/dict. So having {}
means it is present, whereas lun:
is the same as not specifying 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.
Oh, yes, Indeed. i didn't realize this initially, thanks.
disks-and-volumes.md
Outdated
|
||
### cloudInitNoCloud | ||
|
||
Allows attaching `cloudInitNoCloud` data-sources to the vm. If the vm contains |
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.
vm vs VM?
@@ -0,0 +1,315 @@ | |||
# Volumes and Disks | |||
|
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.
Maybe a few introductional words to state that th eterm disk coins the guest facing side, whereas volumes specifies the kube source side.
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.
done
9bce89b
to
cdc312c
Compare
Signed-off-by: Roman Mohr <rmohr@redhat.com>
cdc312c
to
eee1e5f
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.
lgtm
Awesome.
Document which volume and disk types are supported in KubeVirt and how connecting them works.