KEP-1885: Update Azure File csi migration

[andyzhangx]

• One-line PR description: Update Azure File csi migration test plan

• Issue link: Azure file in-tree to CSI driver migration #1885

• Other comments: related k/k PR: make Azure File CSI migration as GA in 1.26 kubernetes#110458

cc @Jiawei0227 @msau42 @xing-yang

[gnufied]

Please note Azurefile migration should not be marked as GA until #3363 feature goes GA too. Without #3363 feature, users can't read/write azurefile volumes unless they are either root or volume has 777 permissions.

[andyzhangx]

Please note Azurefile migration should not be marked as GA until #3363 feature goes GA too. Without #3363 feature, users can't read/write azurefile volumes unless they are either root or volume has 777 permissions.

@gnufied sure, I have removed the GA word in test plan. PTAL, thanks.

[k8s-triage-robot]

The Kubernetes project currently lacks enough contributors to adequately respond to all issues and PRs.

This bot triages issues and PRs according to the following rules:

• After 90d of inactivity, lifecycle/stale is applied
• After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
• After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

• Mark this issue or PR as fresh with /remove-lifecycle stale
• Mark this issue or PR as rotten with /lifecycle rotten
• Close this issue or PR with /close
• Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle stale

[xing-yang]

/remove-lifecycle stale

[xing-yang]

@andyzhangx Can you also update the PRR file to stable?
https://github.com/kubernetes/enhancements/blob/master/keps/prod-readiness/sig-storage/1885.yaml#L2

[xing-yang]

@andyzhangx Have you run in-tree volumes tests?

https://github.com/kubernetes/kubernetes/blob/master/test/e2e/storage/in_tree_volumes.go

[wojtek-t]

I'm fine with this from PRR POV, but you need SIG approval first.

[andyzhangx]

@andyzhangx Have you run in-tree volumes tests?

https://github.com/kubernetes/kubernetes/blob/master/test/e2e/storage/in_tree_volumes.go

@bertinatto do you have the passing tests logs for in-tree azure file tests? related to kubernetes/kubernetes#106678

[bertinatto]

@andyzhangx Have you run in-tree volumes tests?
https://github.com/kubernetes/kubernetes/blob/master/test/e2e/storage/in_tree_volumes.go

@bertinatto do you have the passing tests logs for in-tree azure file tests? related to kubernetes/kubernetes#106678

@andyzhangx yes, but only for Openshift: https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift_azure-file-csi-driver/17/pull-ci-openshift-azure-file-csi-driver-master-e2e-azure-file-csi/1569990026979184640

[xing-yang]

@bertinatto I don't see the "In-tree Volumes" label in the test like what you have shown here: kubernetes/kubernetes#106678 (comment)

[bertinatto]

@bertinatto I don't see the "In-tree Volumes" label in the test like what you have shown here: kubernetes/kubernetes#106678 (comment)

I'm sorry, wrong link. Here's a testgrid link:

https://testgrid.k8s.io/redhat-openshift-ocp-release-4.12-blocking#aggregated-azure-ovn-upgrade-4.12-micro-release-openshift-release-analysis-aggregator

[xing-yang]

/lgtm
/approve

[xing-yang]

/assign @wojtek-t
for PRR approval

[wojtek-t]

/lgtm
/approve PRR

/label tide/merge-method-squash

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: andyzhangx, wojtek-t, xing-yang

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• keps/prod-readiness/OWNERS [wojtek-t]
• keps/sig-storage/OWNERS [xing-yang]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment