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

docs: update the rwx volume document #699

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Vicente-Cheng
Copy link
Contributor

@Vicente-Cheng Vicente-Cheng commented Jan 2, 2025

Related issues:
harvester/harvester#7158
harvester/harvester#7138

For v1.4, we need to use storage network for this feature.
Update the RWX volume document.

@Vicente-Cheng
Copy link
Contributor Author

I will move these changes to v1.4 after the reviews

@innobead
Copy link

innobead commented Jan 2, 2025

Let's associate this with a relevant issue for clarity.

Copy link

github-actions bot commented Jan 2, 2025

Name Link
🔨 Latest commit 806cacd
😎 Deploy Preview https://6777e00edffd10d2933aa812--harvester-preview.netlify.app

@Vicente-Cheng
Copy link
Contributor Author

Let's associate this with a relevant issue for clarity.

updated

Copy link
Contributor

@jillian-maroket jillian-maroket left a comment

Choose a reason for hiding this comment

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

Review done

docs/rancher/csi-driver.md Outdated Show resolved Hide resolved
docs/rancher/csi-driver.md Outdated Show resolved Hide resolved
docs/rancher/csi-driver.md Outdated Show resolved Hide resolved
docs/rancher/csi-driver.md Outdated Show resolved Hide resolved
docs/rancher/csi-driver.md Outdated Show resolved Hide resolved
Signed-off-by: Vicente Cheng <[email protected]>
Co-authored-by: Jillian <[email protected]>
@Vicente-Cheng
Copy link
Contributor Author

Review done

Updated to both dev/v1.4


```
$ ip link set <storage network nic> up
$ ip a add <reserved IP> dev <storage network nic>
Copy link

Choose a reason for hiding this comment

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

This can lead to misunderstandings, because these commands are not persistent after a reboot.

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.

4 participants