-
Notifications
You must be signed in to change notification settings - Fork 950
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
oep(zfspv): ability to provision application utilizing zfs pool already running on the node via CSI. #2705
Conversation
zfs pool already running on the node via CSI. Signed-off-by: Pawan <pawan@mayadata.io>
We should be able to create and delete a volume in a kubernetes cluster | ||
using CSI where the volume has to be created on the ZFS pool which is | ||
already running on the node and the higher order applications should be | ||
able to conume this volume. |
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.
spellcheck
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, done.
## Summary | ||
|
||
This proposal describs how OpenEBS can support dynamic provisioning of | ||
volumes via **CSI** using ZFS dataset that are running on local nodes. |
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 we mention or rephrase that a ZFS dataset will represent a Kubernetes volume.
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.
yeah, done.
#### Volume Creation Workflow | ||
- CSI driver will handle CSI request for volume create | ||
- CSI driver will read the request parameters and create following resources: | ||
- ZfsVolumeCR _(Kubernetes custom resource)_ |
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 we rename this to ZFSVolume
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.
When kubernetes schedules the application pod according the Topology mentioned in | ||
the StorageClass, The CSI will get a NodePublish event. It will get all the volume | ||
properties from ZfsVolumeCR custom resource and it will fire zfs create command | ||
and creates the volume. It will get the pool name from the storage class and create |
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.
Let CSI take in all the values/config/properties from ZFSVolume object.
Let us avoid making use of StorageClass at this stage.
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.
corrected it. we are taking that from CR only.
### 1. CSI create volume | ||
At CSI when we get a volume create request, we will create a PV object. | ||
The actual volume will be create at the Node publish time. We will have | ||
StorageClass at this point. We will create a ZfsVolumeCR with all the |
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 makes sense to publish the ZFSVolume schema in this doc.
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, added the sample CR for now it may change in future..
Signed-off-by: Pawan <pawan@mayadata.io>
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
Signed-off-by: Pawan <pawan@mayadata.io>
Signed-off-by: Pawan <pawan@mayadata.io>
Signed-off-by: Pawan <pawan@mayadata.io>
Signed-off-by: Pawan <pawan@mayadata.io>
Signed-off-by: Pawan <pawan@mayadata.io>
Signed-off-by: Pawan <pawan@mayadata.io>
Signed-off-by: Pawan <pawan@mayadata.io>
Signed-off-by: Pawan <pawan@mayadata.io>
…dy running on the node via CSI. (openebs#2705) Initial Design Proposal for supporting provisioning of Local PVs backed by ZFS using CSI Driver. Signed-off-by: Pawan <pawan@mayadata.io> Signed-off-by: pranjalv9 <pranjalvvyas@gmail.com>
This proposal describs how OpenEBS can support dynamic provisioning of
volumes via CSI using ZFS dataset that are running on local nodes.
Signed-off-by: Pawan pawan@mayadata.io