all lists on lists.proxmox.com
 help / color / mirror / Atom feed
From: Alwin Antreich <a.antreich@proxmox.com>
To: pve-devel@lists.proxmox.com
Subject: [pve-devel] [PATCH manager v3 00/10] ceph: allow pools settings to be changed
Date: Tue, 12 Jan 2021 11:21:43 +0100	[thread overview]
Message-ID: <20210112102153.3215121-1-a.antreich@proxmox.com> (raw)

This set allows to edit pools via GUI & CLI. This should make it easier
for users to adjust pool settings, since they don't have to go the ceph
tool route.

v1 -> v2:
    - move pools endpoint to a subclass
    - add pg autsocale status and settings
    - rework and flatten the grid view of ceph pools
    - rework the create input panel
    - add an edit button using the reworked input panel
    - fix broken add_storages
    - extend setp_pool function to avoid a race condition
    - remove the pg_autoscale_mode default to allow Ceph's setting to
      take precedence

v2 -> v3:
    - incorporate suggestions and comments from Dominik
    - drop 'fix broken add_storages', similar patch has been applied prior
    - drop 'remove default pg_autoscale_mode', kept on warning - see
      forum thread [0].
    - add adjustment of pg_num_min, tuning for the pg_autoscaler

[0] https://forum.proxmox.com/threads/ceph-octopus-upgrade-notes-think-twice-before-enabling-auto-scale.80105

Alwin Antreich (10):
  api: ceph: subclass pools
  ceph: setpool, use parameter extraction instead
  ceph: add titles to ceph_pool_common_options
  ceph: add get api call for single pool
  ceph: add autoscale_status to api calls
  ceph: gui: add autoscale & flatten pool view
  ceph: set allowed minimal pg_num down to 1
  ceph: gui: rework pool input panel
  ceph: gui: add min num of PG
  fix: ceph: always set pool size first

 PVE/API2/Ceph/Makefile    |   1 +
 PVE/API2/Ceph.pm          | 378 +------------------------
 PVE/API2/Ceph/Pools.pm    | 573 ++++++++++++++++++++++++++++++++++++++
 PVE/CLI/pveceph.pm        |  16 +-
 PVE/Ceph/Tools.pm         |  61 ++--
 www/manager6/ceph/Pool.js | 403 +++++++++++++++++++--------
 6 files changed, 922 insertions(+), 510 deletions(-)
 create mode 100644 PVE/API2/Ceph/Pools.pm

-- 
2.29.2





             reply	other threads:[~2021-01-12 10:22 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-01-12 10:21 Alwin Antreich [this message]
2021-01-12 10:21 ` [pve-devel] [PATCH manager v3 01/10] api: ceph: subclass pools Alwin Antreich
2021-02-06 13:28   ` [pve-devel] applied: " Thomas Lamprecht
2021-01-12 10:21 ` [pve-devel] [PATCH manager v3 02/10] ceph: setpool, use parameter extraction instead Alwin Antreich
2021-02-06 13:29   ` [pve-devel] applied: " Thomas Lamprecht
2021-01-12 10:21 ` [pve-devel] [PATCH manager v3 03/10] ceph: add titles to ceph_pool_common_options Alwin Antreich
2021-02-06 13:29   ` [pve-devel] applied: " Thomas Lamprecht
2021-01-12 10:21 ` [pve-devel] [PATCH manager v3 04/10] ceph: add get api call for single pool Alwin Antreich
2021-01-14 16:49   ` Alwin Antreich
2021-01-12 10:21 ` [pve-devel] [PATCH manager v3 05/10] ceph: add autoscale_status to api calls Alwin Antreich
2021-01-12 10:21 ` [pve-devel] [PATCH manager v3 06/10] ceph: gui: add autoscale & flatten pool view Alwin Antreich
2021-01-12 10:21 ` [pve-devel] [PATCH manager v3 07/10] ceph: set allowed minimal pg_num down to 1 Alwin Antreich
2021-01-12 10:21 ` [pve-devel] [PATCH manager v3 08/10] ceph: gui: rework pool input panel Alwin Antreich
2021-01-12 10:21 ` [pve-devel] [PATCH manager v3 09/10] ceph: gui: add min num of PG Alwin Antreich
2021-01-12 10:21 ` [pve-devel] [PATCH manager v3 10/10] fix: ceph: always set pool size first Alwin Antreich

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=20210112102153.3215121-1-a.antreich@proxmox.com \
    --to=a.antreich@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