andyzhangx / kubernetes-drivers

This repository contains all Kubernetes volume drivers on Azure
MIT License
10 stars 1 forks source link

Write azure disk CSI driver #5

Open andyzhangx opened 6 years ago

andyzhangx commented 6 years ago

may update the status here @karataliu

karataliu commented 6 years ago

Refer to: https://github.com/karataliu/csi-plugins-azuredisk

For k8s example: It supports using secret from environment/secret reference for createVolume/deleteVolume. publishVolume/unpublishVolume now accepts secret from environment only, since controllerPublishSecretRef support is pending on: https://github.com/kubernetes-csi/external-provisioner/pull/69

colemickens commented 6 years ago

What's the status now that that PR is merged? Does the AzureDisk CSI implementation then no longer rely on the external Azure CloudProvider itself being deployed?

karataliu commented 6 years ago

@colemickens Yes, the plugin does not rely on cloud provider.

That PR was merged but no release was shipped yet, I built a image with latest master of 'external-provisioner' and verified it would work well: Now publish/unpublish volume also supports reading from secrets, thus no environment variable needed.

If you would like to give a try, please follow the example. https://github.com/karataliu/csi-plugins-azuredisk/tree/master/examples/kubernetes Any feedback appreciated.

Currently the basic pod-pvc flow works, but still need more work on handling exception cases.