the 'rbd' CLI tool has two ways of specifying which images/snapshots to
operate on:
- the deprecated --(dest-)pool, --namespace, --image and --snap parameters
- the new style 'spec'-based variant, passing 
[pool/[namespace/]]image[@snapshot]

we are currently using a mix of both (passing pool, namespace and snap
as needed via parameters, and the image-name as argument). this patch
series switches everything to use image or snap specs, except for `ls`,
which can't be switched over, because it doesn't operate on images at
all.

`unmap` doesn't support `--namespace` (but doesn't need it, since it
takes the kernel dev as argument). `import` doesn't either, but does
require it when a namespace is used as import target (this bug prompted
the whole excursion resulting in this patch series).

I smoke-tested the changes, but it's possible I missed some operation or
corner case when converting, so additional testing is highly
appreciated.

Fabian Grünbichler (3):
  rbd: extend get_rbd_(dev_)path helpers with $snap parameter
  fix #6338: rbd: use image-/snap-spec consistently
  rbd: add protect/unprotect helpers

 src/PVE/Storage/RBDPlugin.pm | 167 ++++++++++++++++++-----------------
 1 file changed, 86 insertions(+), 81 deletions(-)

-- 
2.39.5



_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to