From: Dominik Csapak <d.csapak@proxmox.com>
To: pve-devel@lists.proxmox.com
Subject: [pve-devel] [PATCH storage/manager] fix #3616: support multiple ceph filesystems
Date: Tue, 19 Oct 2021 11:33:41 +0200 [thread overview]
Message-ID: <20211019093353.2451987-1-d.csapak@proxmox.com> (raw)
this series support for multiple cephfs. no single patch fixes the bug,
so it's in no commit subject... (feel free to change the commit subject
when applying if you find one patch most appropriate?)
a user already can create multiple cephfs via 'pveceph' (or manually
with the ceph tools), but the ui does not support it properly
storage patch can be applied independently, it only adds a new parameter
that does nothing if not set.
manager:
patches 1,2 enables basic gui support for showing correct info
for multiple cephfs
patches 3,4,5 are mostly preparation for the following patches
(though 4 enables some additional checks that should not hurt either way)
patch 6 enables additional gui support for multiple fs
patch 7,8 depend on the storage patch
patch 9,10,11 are for actually creating multiple cephfs via the gui
so those can be left out if we do not want to support that
---
so if we only want to support basic display functionality, we could only apply
manager 1,2 & maybe 5+6
for being able to configure multiple cephfs on a ceph cluster, we'd need
storage 1/1 and manager 7,8
sorry that it's so complicated, if wanted, i can ofc reorder the patches
or send it in multiple series
pve-storage:
Dominik Csapak (1):
cephfs: add support for multiple ceph filesystems
PVE/Storage/CephFSPlugin.pm | 8 ++++++++
1 file changed, 8 insertions(+)
pve-manager:
Dominik Csapak (11):
api: ceph-mds: get mds state when multple ceph filesystems exist
ui: ceph: catch missing version for service list
api: cephfs: refactor {ls,create}_fs
api: cephfs: more checks on fs create
ui: ceph/ServiceList: refactor controller out
ui: ceph/fs: show fs for active mds
api: cephfs: add 'fs-name' for cephfs storage
ui: storage/cephfs: make ceph fs selectable
ui: ceph/fs: allow creating multiple cephfs
api: cephfs: add destroy cephfs api call
ui: ceph/fs: allow destroying cephfs
PVE/API2/Ceph/FS.pm | 148 +++++++++--
PVE/Ceph/Services.pm | 16 +-
PVE/Ceph/Tools.pm | 51 ++++
www/manager6/Makefile | 2 +
www/manager6/Utils.js | 1 +
www/manager6/ceph/FS.js | 52 +++-
www/manager6/ceph/ServiceList.js | 313 ++++++++++++-----------
www/manager6/form/CephFSSelector.js | 42 +++
www/manager6/storage/CephFSEdit.js | 25 ++
www/manager6/window/SafeDestroyCephFS.js | 22 ++
10 files changed, 492 insertions(+), 180 deletions(-)
create mode 100644 www/manager6/form/CephFSSelector.js
create mode 100644 www/manager6/window/SafeDestroyCephFS.js
--
2.30.2
next reply other threads:[~2021-10-19 9:33 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-10-19 9:33 Dominik Csapak [this message]
2021-10-19 9:33 ` [pve-devel] [PATCH storage 1/1] cephfs: add support for " Dominik Csapak
2021-10-19 9:33 ` [pve-devel] [PATCH manager 01/11] api: ceph-mds: get mds state when multple ceph filesystems exist Dominik Csapak
2021-10-19 9:33 ` [pve-devel] [PATCH manager 02/11] ui: ceph: catch missing version for service list Dominik Csapak
2021-10-19 9:33 ` [pve-devel] [PATCH manager 03/11] api: cephfs: refactor {ls, create}_fs Dominik Csapak
2021-10-19 9:33 ` [pve-devel] [PATCH manager 04/11] api: cephfs: more checks on fs create Dominik Csapak
2021-10-19 9:33 ` [pve-devel] [PATCH manager 05/11] ui: ceph/ServiceList: refactor controller out Dominik Csapak
2021-10-19 9:33 ` [pve-devel] [PATCH manager 06/11] ui: ceph/fs: show fs for active mds Dominik Csapak
2021-10-19 9:33 ` [pve-devel] [PATCH manager 07/11] api: cephfs: add 'fs-name' for cephfs storage Dominik Csapak
2021-10-19 9:33 ` [pve-devel] [PATCH manager 08/11] ui: storage/cephfs: make ceph fs selectable Dominik Csapak
2021-10-19 9:33 ` [pve-devel] [PATCH manager 09/11] ui: ceph/fs: allow creating multiple cephfs Dominik Csapak
2021-10-19 9:33 ` [pve-devel] [PATCH manager 10/11] api: cephfs: add destroy cephfs api call Dominik Csapak
2021-10-19 9:33 ` [pve-devel] [PATCH manager 11/11] ui: ceph/fs: allow destroying cephfs Dominik Csapak
2021-10-20 14:40 ` [pve-devel] [PATCH storage/manager] fix #3616: support multiple ceph filesystems Aaron Lauterer
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20211019093353.2451987-1-d.csapak@proxmox.com \
--to=d.csapak@proxmox.com \
--cc=pve-devel@lists.proxmox.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox