From: Aaron Lauterer <a.lauterer@proxmox.com>
To: pve-devel@lists.proxmox.com
Subject: Re: [pve-devel] [PATCH manager v3 3/6 follow up] ui: ceph pool: add rbd namespace panel
Date: Fri, 4 Apr 2025 17:20:23 +0200 [thread overview]
Message-ID: <5fa90dae-778c-4545-97b0-b7d28d53f107@proxmox.com> (raw)
In-Reply-To: <20250404151818.1099169-1-a.lauterer@proxmox.com>
Sorry for the noise. Please ignore this one. I sent it again with the
correct in-reply-to to get the threading right.
On 2025-04-04 17:18, Aaron Lauterer wrote:
> This needs a bit of a rework of the Ceph Pool panel because we want to
> have it right next/below to the pool grid. Additionally we want to
> en-/disable it if the select pool has RBD as application enabled.
>
> Therefore we introduce a new small panel (Ceph.PoolView) that holds the
> pool and namespace grid. This also means that we need to redirect the
> submenu in Config.js to use the new intermediate Ceph.PoolView panel
> instead of the pool grid directly.
>
> Signed-off-by: Aaron Lauterer <a.lauterer@proxmox.com>
> ---
> Please use this follow up instead of the original!
> I forgot to remove two listeners at the end of the
> pveNodeCephRbdNamespacelist component that caused problems when
> navigating away from the ceph pool panel!
>
> changes since v2:
> * make new PoolView declarative
> * add interface to pveNodeCephRbdNamespacelist that handles changes to
> the pool name instead of doing that in detail from the
> pveNodeCephPoolList grid.
> * switch to basic extjs.data.Store
> * some smaller bug and style fixes that were reported
>
> v1: none
_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
next prev parent reply other threads:[~2025-04-04 15:21 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-04-04 15:18 Aaron Lauterer
2025-04-04 15:20 ` Aaron Lauterer [this message]
-- strict thread matches above, loose matches on Subject: below --
2025-04-04 14:53 [pve-devel] [PATCH manager v3 3/6] " Aaron Lauterer
2025-04-04 15:19 ` [pve-devel] [PATCH manager v3 3/6 follow up] " 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=5fa90dae-778c-4545-97b0-b7d28d53f107@proxmox.com \
--to=a.lauterer@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal