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

Dynamically provisioned storage account with shared key access disabled #1462

Closed
specialforest opened this issue Jul 9, 2024 · 4 comments · Fixed by #1470
Closed

Dynamically provisioned storage account with shared key access disabled #1462

specialforest opened this issue Jul 9, 2024 · 4 comments · Fixed by #1470
Labels
kind/feature Categorizes issue or PR as related to a new feature.

Comments

@specialforest
Copy link
Contributor

Is your feature request related to a problem?/Why is this needed
I'd like to have dynamically provisioned storage account have shared key access disabled.

Describe the solution you'd like in detail
Introduce new driver parameter that will control storage account shared key access setting (AccountOptions.AllowSharedKeyAccess)

Describe alternatives you've considered
NA

Additional context
https://learn.microsoft.com/en-us/azure/storage/common/shared-key-authorization-prevent

@andyzhangx
Copy link
Member

@specialforest that means you can only use nfs mount, disabling shared key access would disable blobfuse mount, is that what you want?

@andyzhangx andyzhangx added the kind/feature Categorizes issue or PR as related to a new feature. label Jul 10, 2024
@specialforest
Copy link
Contributor Author

I'm using blobfuse. As I understand blobfuse translates to Storage REST API and MSI/SPN auth are already supported, so seems technically possible. Could you please explain more "disabling shared key access would disable blobfuse mount" part, because I'm not really familiar with blobfuse/CSI driver implementation details?

@andyzhangx
Copy link
Member

@specialforest yes, you could use msi auth, follow this guide: https://github.com/qxsch/Azure-Aks/tree/master/aks-blobfuse-mi, while you need to create a storage account first, and you could disable the shared key access in the storage account setting manually.

@specialforest
Copy link
Contributor Author

Thanks, I saw that guide for an existing storage account. I'm interested to have shared key access disabled for storage account dynamically provisioned by Blob CSI driver.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/feature Categorizes issue or PR as related to a new feature.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants