-
Notifications
You must be signed in to change notification settings - Fork 113
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
BlockDevice name not unique, same ssd disk generate dulplicate BlockDevices #660
Comments
Can you share the logs from the the NDM pod and also the output of |
@akhilerm I found the bug in these code, when ID_TYPE is empty and the k8s node hostname changed, the bd name will change.
|
This is the old algorithm that we use to generate UUIDs. Currently there is a |
@akhilerm For many online customer reason, now we couldn't upgrade NDM Feauture: GPTBasedUUID. We plan to think about upgrade smoothly. The ndm new version 1.7.2 compatible with ndm 0..7.2 and openebs/provisioner-localpv:2.12.0 and openebs/m-apiserver:1.12.0? |
@akhilerm Ask for help. we only use device LocalPV, If only upgrade ndm to new version v1.7.2. But openebs keep v1.7.2. will it work? |
Do you mean keeping NDM version at 1.7.2 and localpv version at 2.12.0? Yes it will work. Also from your above comments, it was mentioned about using m-apiserver, which is not required to run localpv provisioner. You can just have localpv provisioner 2.12.0 and NDM 1.7.2. Also a few more questions before you perform the upgrade:
To upgrade :
|
@akhilerm I have a idea. If I make it PR for implement lke this, could be meet my needs:
|
Unfortunately we do not support upgrade for local PV using BlockMode, from old UUID to the new GPTScheme, @Icedroid This is the design that we followed for generating the UUID.
There are some issues with this approach:
|
@akhilerm For physical disk unique WWN and model assigned by the manufacturer. So attaching to a different machine or a different SCSI port of the same machine or different node all keep same. For us, we only use bd for pyhical disk.
I compare WWN for 2 disk , not use bd UUID1 and UUID2. Cloud change md5 hash to uuid algorithm?
physical disk has unique WWN. |
@akhilerm The new version of NDM not support NVME disk scan for BlockDevice ? If fail log :
|
It does support @Icedroid . its an issue with SMART not able to detect some NVMe details. It can be ignored. |
What steps did you take and what happened:
[A clear and concise description of what the bug is, and the sequence of operations performed / commands you ran.]
When the BlockDevice Claimed, I run into container use the bd in LVM filesystem. Then after upgarde node-disk-manager to 0.7.2 from 0.7.1. I found dulplicate BlockDevices generated.
What did you expect to happen:
Same nvme disk only one bd.
The output of the following commands will help us better understand what's going on:
[Pasting long output into a GitHub gist or other pastebin is fine.]
kubectl get pods -n openebs -o wide | grep -i '189.42'
kubectl logs -f pods/openebs-ndm-cwhbw -n openebs
192.168.189.42-ndm.log
kubectl get blockdevices -o wide | grep -i '189.42' | grep -i '/dev/nvme0n1'
lsblk
from nodes where ndm daemonset is runningudevadm info --query=all --name=/dev/nvme0n1|grep -wE 'ID_WWN|ID_VENDOR|ID_MODEL|ID_SERIAL'
kubectl get bd blockdevice-230a46a79b69f26d977880c97e3768cc -n openebs -o yaml
*
kubectl get bd blockdevice-9996981c2155727b2271f51ad74de685 -n openebs -o yaml
Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]
Environment:
openebs/provisioner-localpv:2.12.0
/openebs/m-apiserver:1.12.0
/openebs/node-disk-manager:0.7.2 upgrade from 0.7.1
kubectl version
):v1.16.9
Physical drives
/etc/os-release
):CentOS Linux release 7.7.1908 (Core)
The text was updated successfully, but these errors were encountered: