From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <d.csapak@proxmox.com>
Received: from firstgate.proxmox.com (firstgate.proxmox.com [212.224.123.68])
 (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)
 key-exchange X25519 server-signature RSA-PSS (2048 bits))
 (No client certificate requested)
 by lists.proxmox.com (Postfix) with ESMTPS id 1E33D767A7
 for <pve-devel@lists.proxmox.com>; Tue, 19 Oct 2021 11:33:57 +0200 (CEST)
Received: from firstgate.proxmox.com (localhost [127.0.0.1])
 by firstgate.proxmox.com (Proxmox) with ESMTP id DD7F82AF58
 for <pve-devel@lists.proxmox.com>; Tue, 19 Oct 2021 11:33:56 +0200 (CEST)
Received: from proxmox-new.maurer-it.com (proxmox-new.maurer-it.com
 [94.136.29.106])
 (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)
 key-exchange X25519 server-signature RSA-PSS (2048 bits))
 (No client certificate requested)
 by firstgate.proxmox.com (Proxmox) with ESMTPS id 2D6B02AF12
 for <pve-devel@lists.proxmox.com>; Tue, 19 Oct 2021 11:33:55 +0200 (CEST)
Received: from proxmox-new.maurer-it.com (localhost.localdomain [127.0.0.1])
 by proxmox-new.maurer-it.com (Proxmox) with ESMTP id BFAA6468D5
 for <pve-devel@lists.proxmox.com>; Tue, 19 Oct 2021 11:33:54 +0200 (CEST)
From: Dominik Csapak <d.csapak@proxmox.com>
To: pve-devel@lists.proxmox.com
Date: Tue, 19 Oct 2021 11:33:41 +0200
Message-Id: <20211019093353.2451987-1-d.csapak@proxmox.com>
X-Mailer: git-send-email 2.30.2
MIME-Version: 1.0
Content-Transfer-Encoding: 8bit
X-SPAM-LEVEL: Spam detection results:  0
 AWL 0.291 Adjusted score from AWL reputation of From: address
 BAYES_00                 -1.9 Bayes spam probability is 0 to 1%
 KAM_DMARC_STATUS 0.01 Test Rule for DKIM or SPF Failure with Strict Alignment
 SPF_HELO_NONE           0.001 SPF: HELO does not publish an SPF Record
 SPF_PASS               -0.001 SPF: sender matches SPF record
Subject: [pve-devel] [PATCH storage/manager] fix #3616: support multiple
 ceph filesystems
X-BeenThere: pve-devel@lists.proxmox.com
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Proxmox VE development discussion <pve-devel.lists.proxmox.com>
List-Unsubscribe: <https://lists.proxmox.com/cgi-bin/mailman/options/pve-devel>, 
 <mailto:pve-devel-request@lists.proxmox.com?subject=unsubscribe>
List-Archive: <http://lists.proxmox.com/pipermail/pve-devel/>
List-Post: <mailto:pve-devel@lists.proxmox.com>
List-Help: <mailto:pve-devel-request@lists.proxmox.com?subject=help>
List-Subscribe: <https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel>, 
 <mailto:pve-devel-request@lists.proxmox.com?subject=subscribe>
X-List-Received-Date: Tue, 19 Oct 2021 09:33:57 -0000

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