Skip to content
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

feat(block): adding block volume support for ZFSPV #102

Merged
merged 2 commits into from
May 5, 2020

Conversation

pawanpraka1
Copy link
Contributor

@pawanpraka1 pawanpraka1 commented May 4, 2020

Fixes: openebs/openebs#2971

We can create a Raw Block Volume request using volumemode as block in PVC :-

kind: PersistentVolumeClaim
apiVersion: v1
metadata:
  name: block-claim
spec:
  volumeMode: Block
  storageClassName: zfspv-block
  accessModes:
    - ReadWriteOnce
  resources:
    requests:
      storage: 5Gi

we can use the below storageclass for RAW block volume:

apiVersion: storage.k8s.io/v1
kind: StorageClass
metadata:
  name: zfspv-block
allowVolumeExpansion: true
parameters:
  poolname: "zfspv-pool"
provisioner: zfs.csi.openebs.io

The driver will create a zvol for this volume and bind mount the block device at the given path.

Signed-off-by: Pawan pawan@mayadata.io

@pawanpraka1 pawanpraka1 added enhancement Add new functionality to existing feature pr/release-note PR has to be added in the release note. labels May 4, 2020
@pawanpraka1 pawanpraka1 added this to the v0.7.0 milestone May 4, 2020
@pawanpraka1 pawanpraka1 requested a review from kmova May 4, 2020 11:42
We can create a Raw Block Volume request using volumemode as block in PVC :-

kind: PersistentVolumeClaim
apiVersion: v1
metadata:
  name: block-claim
spec:
  volumeMode: Block
  storageClassName: zfspv-block
  accessModes:
    - ReadWriteOnce
  resources:
    requests:
      storage: 5Gi

The driver will create a zvol for this volume and bind mount the block device at the given path.

Signed-off-by: Pawan <pawan@mayadata.io>
@codecov-io
Copy link

codecov-io commented May 4, 2020

Codecov Report

Merging #102 into master will decrease coverage by 0.67%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #102      +/-   ##
==========================================
- Coverage   23.57%   22.90%   -0.68%     
==========================================
  Files          14       14              
  Lines         475      489      +14     
==========================================
  Hits          112      112              
- Misses        362      376      +14     
  Partials        1        1              
Impacted Files Coverage Δ
pkg/client/k8s/v1alpha1/node.go 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 49dc997...353a548. Read the comment docs.

pkg/driver/agent.go Outdated Show resolved Hide resolved
pkg/zfs/mount.go Show resolved Hide resolved
Signed-off-by: Pawan <pawan@mayadata.io>
@kmova kmova merged commit dd059a2 into openebs:master May 5, 2020
ZFS Local PV automation moved this from In progress to Done May 5, 2020
1.10 Release Tracker - Due May 15th. automation moved this from RC1 - Due: May 5 2020 to Done May 5, 2020
@pawanpraka1 pawanpraka1 deleted the block branch May 5, 2020 07:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Add new functionality to existing feature pr/release-note PR has to be added in the release note.
Projects
ZFS Local PV
  
Done
Development

Successfully merging this pull request may close these issues.

Create ZFS PV in block mode.
3 participants