Skip to content

[AKS] support blob csi driver - #5027

Merged
Xing Zhou (zhoxing-ms) merged 13 commits into
Azure:mainfrom
cvvz:aks-preview-support-blob-csi-driver
Jun 23, 2022
Merged

Xing Zhou (zhoxing-ms) merged 13 commits into
Azure:mainfrom
cvvz:aks-preview-support-blob-csi-driver

Conversation

@cvvz

Copy link
Copy Markdown
Contributor

This checklist is used to make sure that common guidelines for a pull request are followed.

Related command

General Guidelines

  • Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
  • Have you run python scripts/ci/test_index.py -q locally?

For new extensions:

About Extension Publish

There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
The precondition is to put your code inside this repository and upgrade the version in the pull request but do not modify src/index.json.

@yonzhan

Copy link
Copy Markdown
Collaborator

AKS

@cvvz weizhi (cvvz) changed the title [WIP]support blob csi driver [WIP][AKS] support blob csi driver Jun 22, 2022
@andyzhangx

Copy link
Copy Markdown
Contributor

cc Ji'an Liu (@ZeroMagic)

Comment thread src/aks-preview/azext_aks_preview/decorator.py Outdated

if self.decorator_mode == DecoratorMode.UPDATE:
if enable_blob_driver:
msg = "Please make sure there is no open-source Blob CSI driver installed before enabling."

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Meagan (@olsenme) could you review the msg syntax? thanks.

Comment thread src/aks-preview/azext_aks_preview/managed_cluster_decorator.py
Comment thread src/aks-preview/azext_aks_preview/managed_cluster_decorator.py Outdated
@cvvz weizhi (cvvz) changed the title [WIP][AKS] support blob csi driver [AKS] support blob csi driver Jun 22, 2022

@FumingZhang FumingZhang left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@zhoxing-ms
Xing Zhou (zhoxing-ms) merged commit 9286058 into Azure:main Jun 23, 2022
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

Successfully merging this pull request may close these issues.

6 participants