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

[Feature] Allow configuring the resulting StorageClass name in Container Storage StoragePool #4721

Open
felfa01 opened this issue Dec 20, 2024 · 0 comments
Labels
feature-request Requested Features

Comments

@felfa01
Copy link

felfa01 commented Dec 20, 2024

Is your feature request related to a problem? Please describe.
I'd like to be able to configure the name that the resulting StorageClass will have when deploying a StoragePool. One of my use cases would be to migrate from CSI Drivers to Azure Container Storage without requiring my tenants to update their PVC manifests with a new StorageClassName.

This was previously available in the now deprecated apiVersion v1alpha1 under .spec.poolType.azureDisk.storageClassName.

Describe the solution you'd like

apiVersion: containerstorage.azure.com/v1alpha1
kind: StoragePool
metadata:
  name: my-storage-pool-name
  namespace: acstor
spec:
  storageClassName: my-storage-class-name # A new field for setting the resulting StorageClass name
  poolType:
    azureDisk:
      skuName: Premium_LRS
  resources:
    requests:
      storage: 1Ti

Describe alternatives you've considered
Use the deprecated apiVersion v1alpha1?

@felfa01 felfa01 added the feature-request Requested Features label Dec 20, 2024
@felfa01 felfa01 changed the title [Feature] Allow configuring the StorageClassName in Container Storage StoragePool [Feature] Allow configuring the resulting StorageClass name in Container Storage StoragePool Dec 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request Requested Features
Projects
None yet
Development

No branches or pull requests

1 participant