Closed landreasyan closed 1 month ago
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: andyzhangx, landreasyan
The full list of commands accepted by this bot can be found here.
The pull request process is described here
@landreasyan: The following tests failed, say /retest
to rerun all failed tests or /retest-required
to rerun all mandatory failed tests:
Test name | Commit | Details | Required | Rerun command |
---|---|---|---|---|
pull-azuredisk-csi-driver-external-e2e-single-az | 3b3ce0a30681e03f83f0feefd21447a174d4ff3a | link | true | /test pull-azuredisk-csi-driver-external-e2e-single-az |
pull-azuredisk-csi-driver-e2e-capz | 3b3ce0a30681e03f83f0feefd21447a174d4ff3a | link | true | /test pull-azuredisk-csi-driver-e2e-capz |
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR.
/cherrypick release-1.31
/cherrypick release-1.30
/cherrypick release-1.29
/cherrypick release-1.28
@andyzhangx: new pull request created: #2571
@andyzhangx: new pull request created: #2572
@andyzhangx: new pull request created: #2573
@andyzhangx: new pull request created: #2574
What type of PR is this?
What this PR does / why we need it: Customers can use VAC to migrate
PremiumLRS
disks toPremiumV2LRS
disks while they are detached from the VM. While the driver already sets caching mode toNone
when it creates aPremiumV2LRS
disk, it still defaults to either the caching mode set in theVolumeContext
or toReadOnly
caching when it attaches the disk. Because caching in theVolumeContext
defaults and is set toReadOnly
forPremiumLRS
disks, to support attach operation after the migration, the driver must set the caching mode toNone
explicitly when it attaches aPremiumV2LRS
disk.Which issue(s) this PR fixes:
Fixes #
Requirements:
Special notes for your reviewer:
Release note: