Re: [pve-devel] [PATCH widget-toolkit/manager v2 0/4] Ceph OSD: add detail infos

2022-10-18 Thread Dominik Csapak
On 10/18/22 11:14, Aaron Lauterer wrote: On 10/17/22 16:29, Dominik Csapak wrote: high level looks mostly good, a small question: is there a special reason why we ignore pre-lvm osds here? AFAICS, we simply error out for osds that don't live on lvm (though we can add additional types later i

Re: [pve-devel] [PATCH widget-toolkit/manager v2 0/4] Ceph OSD: add detail infos

2022-10-18 Thread Aaron Lauterer
On 10/17/22 16:29, Dominik Csapak wrote: high level looks mostly good, a small question: is there a special reason why we ignore pre-lvm osds here? AFAICS, we simply error out for osds that don't live on lvm (though we can add additional types later i guess) Mainly because with a recent Cep

Re: [pve-devel] [PATCH widget-toolkit/manager v2 0/4] Ceph OSD: add detail infos

2022-10-17 Thread Dominik Csapak
high level looks mostly good, a small question: is there a special reason why we ignore pre-lvm osds here? AFAICS, we simply error out for osds that don't live on lvm (though we can add additional types later i guess) comments in the individual patches _

Re: [pve-devel] [PATCH widget-toolkit/manager v2 0/4] Ceph OSD: add detail infos

2022-10-12 Thread Aaron Lauterer
ping? patches still apply cleanly On 7/6/22 15:01, Aaron Lauterer wrote: This patch series adds 2 new API endpoints for OSDs to fetch more detailed information about a single OSD. One for overall information and one for a single volume (block, db, wal). Because the api endpoint for the volume

[pve-devel] [PATCH widget-toolkit/manager v2 0/4] Ceph OSD: add detail infos

2022-07-06 Thread Aaron Lauterer
This patch series adds 2 new API endpoints for OSDs to fetch more detailed information about a single OSD. One for overall information and one for a single volume (block, db, wal). Because the api endpoint for the volume infos can take a few moments, we most likely want to show that loading state.