Re: [pve-devel] [PATCH v2 storage] rbd: alloc image: fix #3970 avoid ambiguous rbd path

[Thomas Lamprecht <t.lamprecht@proxmox.com>]

On 11.04.22 16:49, Aaron Lauterer wrote:
> On 4/11/22 14:17, Thomas Lamprecht wrote:
>> FWIW, with storage getting the following patch the symlinks get created (may need
>> an trigger for reloading udev (or manually `udevadm control -R`).
>>
>> We'd only need to check to prioritize /deb/rbd-pve/$fsid/... paths first;
>> do you have time to give that a go?
> 
> 
> The final `fi` in the renamer script was missing from the diff. Once I fixed that, it seems to work fine. Situation with a local hyperconverged cluster and an external one:

Oh sorry, seems my copy-paste has gone slightly wrong.

> ---------------------------------
> root@cephtest1:/dev/rbd-pve# tree
> .
> ├── ce99d398-91ab-4667-b4f2-307ba0bec358
> │   └── ecpool-metadata
> │       ├── vm-103-disk-0 -> ../../../rbd0
> │       ├── vm-103-disk-0-part1 -> ../../../rbd0p1
> │       ├── vm-103-disk-0-part2 -> ../../../rbd0p2
> │       ├── vm-103-disk-0-part5 -> ../../../rbd0p5
> │       ├── vm-103-disk-1 -> ../../../rbd1
> │       └── vm-103-disk-2 -> ../../../rbd2
> └── e78d9b15-d5a1-4660-a4a5-d2c1208119e9
>     └── rbd
>         └── vm-103-disk-0 -> ../../../rbd3

Glad that it works for you too, but what I actually meant is if you had time to
give the remaining perl-side of the fix a go (sorry if I did not made that clear
enough).