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(configure-mayastor): updated note to bypath or byid #183

Merged
merged 3 commits into from
Nov 14, 2023

Conversation

AJDatacore
Copy link
Contributor

In reference to this issue: openebs/mayastor#1530

Hi @tiagolobocastro Could you please confirm if this change is relevant? If now, can you suggest the right place to add this note?

quickstart/configure-mayastor.md Outdated Show resolved Hide resolved
quickstart/configure-mayastor.md Outdated Show resolved Hide resolved

Usage of the device name (for example, /dev/sdx) is not advised, as it may change if the node reboots, which might cause data corruption.
{% endhint %}

| Type | Format | Example |
| :--- | :--- | :--- |
| Disk(non PCI) with disk-by-guid reference <i><b>(Best Practice)</b></i> | Device File | aio:////dev/disk/by-uuid/<uuid> OR uring:////dev/disk/by-uuid/<uuid> |
| Disk(non PCI) with disk-by-guid reference <i><b>(Best Practice)</b></i> | Device File | aio:////dev/disk/by-id/<id> OR uring:////dev/disk/by-id/<id> |
Copy link
Contributor

Choose a reason for hiding this comment

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

sorry one more nit, there's an extra slash here

Suggested change
| Disk(non PCI) with disk-by-guid reference <i><b>(Best Practice)</b></i> | Device File | aio:////dev/disk/by-id/<id> OR uring:////dev/disk/by-id/<id> |
| Disk(non PCI) with disk-by-guid reference <i><b>(Best Practice)</b></i> | Device File | aio:///dev/disk/by-id/<id> OR uring:///dev/disk/by-id/<id> |

@tiagolobocastro tiagolobocastro merged commit 4a0c708 into version/2.4 Nov 14, 2023
3 checks passed
@tiagolobocastro
Copy link
Contributor

tiagolobocastro commented Nov 14, 2023

Thanks @AJDatacore, could you please cherry-pick to develop as well?

cherry-picked

tiagolobocastro pushed a commit that referenced this pull request Nov 28, 2023
* docs(configure-mayastor): updated note to by-path or by-id

Signed-off-by: AJ_Datacore <[email protected]>
@niladrih niladrih deleted the link-correction branch May 20, 2024 16:25
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.

3 participants