public inbox for pve-devel@lists.proxmox.com
 help / color / mirror / Atom feed
From: Aaron Lauterer <a.lauterer@proxmox.com>
To: pve-devel@lists.proxmox.com
Subject: [pve-devel] [PATCH manager 0/3] fix 2515 use default sizes for new ceph
Date: Fri, 13 Jan 2023 16:09:27 +0100	[thread overview]
Message-ID: <20230113150930.857270-1-a.lauterer@proxmox.com> (raw)

We usually set default size and min_size parameters upon Ceph
initialization.

This patch series changes the UI, so that these defaults will be used
when creating a new pool.
In order to fetch Ceph configuration keys, a new API endpoint is
introduced (configkey). I made it so that any Ceph config keys can be
retrieved, to make it more flexible for any potential future use.

It gathers Ceph config options from the ceph.conf file and the config
DB. Settings in the ceph.conf file override settings from the config DB.

If there are better ideas for the name, please, as I am not really happy
with it, but couldn't come up with something else :)
We already have 'config' and 'configdb'. 'config' returns the ceph.conf
file as is and 'configdb' returns the contents of the config DB. Both
are primarily (AFAICT) used to show the contents in the
Ceph->Configuration UI panel.


Aaron Lauterer (3):
  api: ceph: add endpoint to fetch config keys
  fix #2515: ui: ceph pool create: use configured defaults for size and
    min_size
  ui: ceph pool edit: rework with controller and formulas

 PVE/API2/Ceph.pm          |  84 ++++++++++++++++++++++
 www/manager6/ceph/Pool.js | 142 ++++++++++++++++++++++++++++----------
 2 files changed, 191 insertions(+), 35 deletions(-)

-- 
2.30.2





             reply	other threads:[~2023-01-13 15:10 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-01-13 15:09 Aaron Lauterer [this message]
2023-01-13 15:09 ` [pve-devel] [PATCH manager 1/3] api: ceph: add endpoint to fetch config keys Aaron Lauterer
2023-03-08 12:14   ` Dominik Csapak
2023-03-11 17:07     ` Thomas Lamprecht
2023-03-13 12:58       ` Aaron Lauterer
2023-03-13 16:31         ` Thomas Lamprecht
2023-01-13 15:09 ` [pve-devel] [PATCH manager 2/3] fix #2515: ui: ceph pool create: use configured defaults for size and min_size Aaron Lauterer
2023-03-08 12:14   ` Dominik Csapak
2023-01-13 15:09 ` [pve-devel] [PATCH manager 3/3] ui: ceph pool edit: rework with controller and formulas Aaron Lauterer
2023-03-08 12:15   ` Dominik Csapak

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=20230113150930.857270-1-a.lauterer@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal