From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: 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 D0BD6798B7 for ; Wed, 27 Oct 2021 12:48:11 +0200 (CEST) Received: from firstgate.proxmox.com (localhost [127.0.0.1]) by firstgate.proxmox.com (Proxmox) with ESMTP id CECE6114AD for ; Wed, 27 Oct 2021 12:48:11 +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 1B6381149F for ; Wed, 27 Oct 2021 12:48:11 +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 E5F6845F8B for ; Wed, 27 Oct 2021 12:48:10 +0200 (CEST) Message-ID: <92b7ef2d-0d1d-1366-063f-0f907082fd47@proxmox.com> Date: Wed, 27 Oct 2021 12:48:09 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.2.1 Content-Language: en-US To: Proxmox VE development discussion , Dominik Csapak References: <20211025140139.2015470-1-d.csapak@proxmox.com> From: Aaron Lauterer In-Reply-To: <20211025140139.2015470-1-d.csapak@proxmox.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-SPAM-LEVEL: Spam detection results: 0 AWL 0.260 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 NICE_REPLY_A -0.215 Looks like a legit reply (A) SPF_HELO_NONE 0.001 SPF: HELO does not publish an SPF Record SPF_PASS -0.001 SPF: sender matches SPF record URIBL_BLOCKED 0.001 ADMINISTRATOR NOTICE: The query to URIBL was blocked. See http://wiki.apache.org/spamassassin/DnsBlocklists#dnsbl-block for more information. [fs.pm, cephfsplugin.pm, services.pm, pveceph.pm, tools.pm] Subject: Re: [pve-devel] [PATCH storage/manager/docs v2] 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 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 27 Oct 2021 10:48:11 -0000 Works like expected. The one big problem I had previously with MDS not being ready is gone as one can see now in the task log, that it is waiting for the MDS to become active before continuing. Tested the removal procedure outlined in the docs patch and as long as the storage is still active, it gives a nice warning. Of course, being lazy I did not unmount it on all the nodes first and once it is removed, the nodes and child items all show question marks in the tree view. Once it gets unmounted with -f -l (force & lazy) parameters, the nodes are shown as okay again in the GUI. Not sure if we want to check against it not being mounted on all nodes before proceeding or if that is the responsibility of the admin. Tested-By: Aaron Lauterer On 10/25/21 16:01, Dominik Csapak wrote: > 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. > > changes from v1: > * moved 'destroyfs' from api to cli only > * removed 'destroy cephfs' from the gui > * added docs patch to document the exact steps on how to remove a cephfs > * added 'disable' check on remove-storages > * change 'is mds active' check to check for specific fs_name > > 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 > api: cephfs: add fs_name to 'is mds active' check > 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 > pveceph: add 'fs destroy' command > > PVE/API2/Ceph/FS.pm | 31 ++- > PVE/CLI/pveceph.pm | 120 +++++++++++ > PVE/Ceph/Services.pm | 33 +-- > PVE/Ceph/Tools.pm | 51 +++++ > www/manager6/Makefile | 1 + > www/manager6/Utils.js | 1 + > www/manager6/ceph/FS.js | 24 +-- > www/manager6/ceph/ServiceList.js | 313 +++++++++++++++------------- > www/manager6/form/CephFSSelector.js | 42 ++++ > www/manager6/storage/CephFSEdit.js | 25 +++ > 10 files changed, 449 insertions(+), 192 deletions(-) > create mode 100644 www/manager6/form/CephFSSelector.js > > pve-docs: > > Dominik Csapak (1): > pveceph: improve documentation for destroying cephfs > > pveceph.adoc | 49 +++++++++++++++++++++++++++++++++++++------------ > 1 file changed, 37 insertions(+), 12 deletions(-) >