search

ccremer / kubernetes-zfs-provisioner

Dynamic ZFS persistent volume provisioner for Kubernetes
Apache License 2.0
80 stars 7 forks source link

Thin provisioning #101

Closed Foxite closed 1 year ago

Foxite commented 1 year ago

Currently space is reserved when the dataset is created. What do you think about making this optional, essentially making the PVs thinly-provisioned? This should probably be an option in the storage class.

Foxite commented 1 year ago

I don't have much experience with go but I've looked over the code and I'm pretty sure I can implement and test this. I'll open a PR (hopefully) today.

ccremer commented 1 year ago

Nice idea for probably a niche usage.

In quick online searches I couldn't find any hints about the capacity parameter in PVC, indicating whether it's OK to under-provision storage. As a user, when I request a PVC of 50GiB I would be very surprised and confused in cases where storage runs out earlier than 50Gib because underlying storage-box is overprovisioned with thinly-provisioned-volumes.

I'm not against this feature, since this provisioner is probably niche itself, being not-so-scalable anyway ;) But thin-provisioning has its drawback and I think we should make users properly aware :)