Update harvester-csi-driver prerequisites#998
Conversation
Signed-off-by: Cooper Tseng <cooper.tseng@suse.com>
github-actions
bot
requested review from
akashraj4261,
dariavladykina and
jillian-maroket
There was a problem hiding this comment.
Pull request overview
Updates the Harvester CSI Driver documentation to clarify an additional prerequisite introduced with CSI driver v0.1.25 when installing without the Harvester cloud provider.
Changes:
- Added a prerequisite note warning that volume snapshot components must be installed for v0.1.25+ installs without the Harvester cloud provider to avoid chart installation failures.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| - The Kubernetes cluster is built on top of Harvester virtual machines. | ||
| - The Harvester virtual machines that run as guest Kubernetes nodes are in the same namespace. | ||
| - Starting with v0.1.25, if you install the CSI driver without enabling the Harvester cloud provider, ensure that [Volume Snapshots](./csi-driver.md#volume-snapshots) are properly installed; otherwise, the chart installation will fail. |
There was a problem hiding this comment.
The prerequisite bullet says "ensure that Volume Snapshots are properly installed", but volume snapshots aren’t something you install directly; what’s required is the CSI snapshot controller + CRDs/manifests (as described later in the Volume Snapshots prerequisites). Consider rephrasing this to explicitly reference installing the snapshot controller/CRDs so readers know what to do and what component causes the chart install to fail.
| - Starting with v0.1.25, if you install the CSI driver without enabling the Harvester cloud provider, ensure that [Volume Snapshots](./csi-driver.md#volume-snapshots) are properly installed; otherwise, the chart installation will fail. | |
| - Starting with v0.1.25, if you install the CSI driver without enabling the Harvester cloud provider, ensure that the CSI snapshot controller and VolumeSnapshot CRDs/manifests described in [Volume Snapshots](./csi-driver.md#volume-snapshots) are installed; otherwise, the chart installation will fail. |
|
|
Sorry I missed that. |
|
You're right, let me file another pr |
|
@Vicente-Cheng, here it is #1008 (comment) |
update the harvester-csi-driver prerequisites