mirrors/ceph-csi
1
0
Fork 0
mirror of https://github.com/ceph/ceph-csi.git synced 2024-11-17 20:00:23 +00:00
Code Issues Packages Projects Releases Wiki Activity

deploy: let the pool/fs name be required

Browse Source 
Obviously expecting a pool with name `rbd` or CephFS name `myfs`
will be a limitation, as the pool/fs is created by admin manually,
let them choose the name that suits their requirement and come back
edit it in the storage class.

Making the pool/fs name as required field will give more attention,
else with new users it will be mostly left unedited until one hit
the errors saying no pool/fs exists.

This patch clips-off the default pool/fs name and make it a mandatory
field.

Signed-off-by: Prasanna Kumar Kalever <prasanna.kalever@redhat.com>
This commit is contained in:
Prasanna Kumar Kalever 2020-09-23 13:59:17 +05:30 committed by mergify[bot]
parent f6de345cc9
commit 0c6d8fbd80
2 changed files with 12 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
examples/cephfs/storageclass.yaml
View File

@ -14,11 +14,12 @@ parameters:
# represent the Ceph cluster in clusterID below # represent the Ceph cluster in clusterID below
clusterID: <cluster-id> clusterID: <cluster-id>
# CephFS filesystem name into which the volume shall be created # (required) CephFS filesystem name into which the volume shall be created
fsName: myfs # eg: fsName: myfs
fsName: <cephfs-name>
# (optional) Ceph pool into which volume data shall be stored # (optional) Ceph pool into which volume data shall be stored
# pool: cephfs_data # pool: <cephfs-data-pool>
# (optional) Comma separated string of Ceph-fuse mount options. # (optional) Comma separated string of Ceph-fuse mount options.
# For eg: # For eg:

12
examples/rbd/storageclass.yaml
View File

@ -17,13 +17,17 @@ parameters:
# csi-config-map-sample.yaml, to accompany the string chosen to # csi-config-map-sample.yaml, to accompany the string chosen to
# represent the Ceph cluster in clusterID below # represent the Ceph cluster in clusterID below
clusterID: <cluster-id> clusterID: <cluster-id>
# If you want to use erasure coded pool with RBD, you need to create
# two pools. one erasure coded and one replicated. # (optional) If you want to use erasure coded pool with RBD, you need to
# create two pools. one erasure coded and one replicated.
# You need to specify the replicated pool here in the `pool` parameter, it is # You need to specify the replicated pool here in the `pool` parameter, it is
# used for the metadata of the images. # used for the metadata of the images.
# The erasure coded pool must be set as the `dataPool` parameter below. # The erasure coded pool must be set as the `dataPool` parameter below.
# dataPool: ec-data-pool # dataPool: <ec-data-pool>
pool: rbd
# (required) Ceph pool into which the RBD image shall be created
# eg: pool: rbdpool
pool: <rbd-pool-name>
# RBD image features, CSI creates image with image-format 2 # RBD image features, CSI creates image with image-format 2
# CSI RBD currently supports only `layering` feature. # CSI RBD currently supports only `layering` feature.