Skip to content

Commit

Permalink
Merge pull request #9289 from rook/mergify/bp/release-1.7/pr-9287
Browse files Browse the repository at this point in the history
helm: addition of mountOptions into storage class configuration (backport #9287)
  • Loading branch information
mergify[bot] committed Dec 1, 2021
2 parents f95602b + aab2537 commit 703b327
Show file tree
Hide file tree
Showing 4 changed files with 16 additions and 0 deletions.
2 changes: 2 additions & 0 deletions Documentation/helm-ceph-cluster.md
Expand Up @@ -88,6 +88,7 @@ The `cephBlockPools` array in the values file will define a list of CephBlockPoo
| `storageClass.parameters` | See [Block Storage](ceph-block.md) documentation or the helm values.yaml for suitable values | see values.yaml |
| `storageClass.reclaimPolicy` | The default [Reclaim Policy](https://kubernetes.io/docs/concepts/storage/storage-classes/#reclaim-policy) to apply to PVCs created with this storage class. | `Delete` |
| `storageClass.allowVolumeExpansion` | Whether [volume expansion](https://kubernetes.io/docs/concepts/storage/storage-classes/#allow-volume-expansion) is allowed by default. | `true` |
| `storageClass.mountOptions` | Specifies the mount options for storageClass | `[]` |

### Ceph File Systems

Expand All @@ -101,6 +102,7 @@ The `cephFileSystems` array in the values file will define a list of CephFileSys
| `storageClass.name` | The name of the storage class | `ceph-filesystem` |
| `storageClass.parameters` | See [Shared Filesystem](ceph-filesystem.md) documentation or the helm values.yaml for suitable values | see values.yaml |
| `storageClass.reclaimPolicy` | The default [Reclaim Policy](https://kubernetes.io/docs/concepts/storage/storage-classes/#reclaim-policy) to apply to PVCs created with this storage class. | `Delete` |
| `storageClass.mountOptions` | Specifies the mount options for storageClass | `[]` |

### Ceph Object Stores

Expand Down
6 changes: 6 additions & 0 deletions cluster/charts/rook-ceph-cluster/templates/cephblockpool.yaml
Expand Up @@ -22,5 +22,11 @@ parameters:
{{ toYaml $blockpool.storageClass.parameters | indent 2 }}
reclaimPolicy: {{ default "Delete" $blockpool.storageClass.reclaimPolicy }}
allowVolumeExpansion: {{ default "true" $blockpool.storageClass.allowVolumeExpansion }}
{{- if $blockpool.storageClass.mountOptions }}
mountOptions:
{{- range $blockpool.storageClass.mountOptions }}
- {{ . }}
{{- end }}
{{- end }}
{{ end }}
{{ end }}
Expand Up @@ -23,5 +23,11 @@ parameters:
{{ toYaml $filesystem.storageClass.parameters | indent 2 }}
reclaimPolicy: {{ default "Delete" $filesystem.storageClass.reclaimPolicy }}
allowVolumeExpansion: {{ default "true" $filesystem.storageClass.allowVolumeExpansion }}
{{- if $filesystem.storageClass.mountOptions }}
mountOptions:
{{- range $filesystem.storageClass.mountOptions }}
- {{ . }}
{{- end }}
{{- end }}
{{ end }}
{{ end }}
2 changes: 2 additions & 0 deletions cluster/charts/rook-ceph-cluster/values.yaml
Expand Up @@ -326,6 +326,7 @@ cephBlockPools:
isDefault: true
reclaimPolicy: Delete
allowVolumeExpansion: true
mountOptions: []
# see https://github.com/rook/rook/blob/master/Documentation/ceph-block.md#provision-storage for available configuration
parameters:
# (optional) mapOptions is a comma-separated list of map options.
Expand Down Expand Up @@ -378,6 +379,7 @@ cephFileSystems:
name: ceph-filesystem
reclaimPolicy: Delete
allowVolumeExpansion: true
mountOptions: []
# see https://github.com/rook/rook/blob/master/Documentation/ceph-filesystem.md#provision-storage for available configuration
parameters:
# The secrets contain Ceph admin credentials.
Expand Down

0 comments on commit 703b327

Please sign in to comment.