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
Add notes about using Filesystem on block devices #130
Add notes about using Filesystem on block devices #130
Conversation
Welcome @mattymo! |
/assign @msau42 |
$ sudo ln -s /dev/disk/by-id/lvm-pv-uuid-yyTnct-TpUS-U93g-JoFs-6seh-Yy29-Dn6Irf /mnt/disks | ||
``` | ||
|
||
Additional notes: Your storage class must have VolumeMode set to "Filesystem" |
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 PVC is the one that specifies volumeMode. Can you also add clarify here which provisioner configmap parameters need to be set?
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.
It needs to be set in the provisioner, the storageclass, and in the pvc for it to work. Otherwise, it appears as a block PV
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.
There's no volumeMode field in the StorageClass object. Can you clarify in the doc here exactly which K8s objects and provisioner configmap values need to be set?
@@ -154,6 +154,38 @@ Link it into discovery directory: | |||
$ sudo ln -s /dev/disk/by-id/lvm-pv-uuid-yyTnct-TpUS-U93g-JoFs-6seh-Yy29-Dn6Irf /mnt/disks | |||
``` | |||
|
|||
### Link devices into directory to be discovered as Filesystem PVs |
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.
Can you add this to the TOC above?
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.
@mattymo you can generate it with markdown-toc
, e.g. markdown-toc --no-firsth1 docs/operations.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.
Thanks for reminding me to do this.
Issues go stale after 90d of inactivity. If this issue is safe to close now please do so with Send feedback to sig-testing, kubernetes/test-infra and/or fejta. |
/remove-lifecycle stale |
@mattymo do you still have time to work on this? |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cofyc, mattymo The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
…m-doc Add notes about using Filesystem on block devices
Fixes #129