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

Implement support for storage class parameter - zone/zones/allowedTopologies #64

Closed
leakingtapan opened this issue Oct 14, 2018 · 2 comments
Assignees
Milestone

Comments

@leakingtapan
Copy link
Contributor

zone/zones are needed for k8s version v1.10 and v1.11. These two fields are deprecated in v1.12 in favor of allowedTopologies.

@leakingtapan
Copy link
Contributor Author

ref #31

@leakingtapan
Copy link
Contributor Author

leakingtapan commented Oct 22, 2018

Tested allowedTopologies is supported once topology is enabled using VolumeScheduling.

sc used for testing:

kind: StorageClass
apiVersion: storage.k8s.io/v1
metadata:
  name: zone-sc
  annotations:
    storageclass.kubernetes.io/is-default-class: "true"
provisioner: com.amazon.aws.csi.ebs
volumeBindingMode: WaitForFirstConsumer
allowedTopologies:
- matchLabelExpressions:
  - key: com.amazon.aws.csi.ebs/zone
    values:
    - us-east-1b

No driver change will be needed in v1.12+ for this feature. I will add documentation for this.

@leakingtapan leakingtapan self-assigned this Oct 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant