]> git.proxmox.com Git - pve-manager-legacy.git/commit
api: ceph: deprecate pools in favor of pool
authorAaron Lauterer <a.lauterer@proxmox.com>
Mon, 20 Mar 2023 10:32:30 +0000 (11:32 +0100)
committerThomas Lamprecht <t.lamprecht@proxmox.com>
Mon, 20 Mar 2023 14:31:04 +0000 (15:31 +0100)
commit9ab403e31ef133933c5831e317cbcfd624f1a980
tree60c1f46ebcf6c10e6bf8159b227ee81ac9601c9d
parent48dcbd1e991bf33d0382ba01406032c74b806044
api: ceph: deprecate pools in favor of pool

/nodes/{node}/ceph/pools/{pool} returns the pool details right away on a
GET.  This makes it bad practice to add additional sub API endpoints.

By deprecating it and replacing it with /nodes/{node}/ceph/pool/{pool}
(singular instead of plural) we can turn that into an index GET
response, making it possible to expand it more in the future.

The GET call returning the pool details is moved into
/nodes/{node}/ceph/pool/{pool}/status

The code in the new Pool.pm is basically a copy of Pools.pm to avoid
a close coupling with the old code as it is possible that it will divert
until we can entirely remove the old code.

Signed-off-by: Aaron Lauterer <a.lauterer@proxmox.com>
Reviewed-by: Dominik Csapak <d.csapak@proxmox.com>
Tested-by:  Dominik Csapak <d.csapak@proxmox.com>
PVE/API2/Ceph.pm
PVE/API2/Ceph/Makefile
PVE/API2/Ceph/Pool.pm [new file with mode: 0644]
PVE/API2/Ceph/Pools.pm