public inbox for pbs-devel@lists.proxmox.com
 help / color / mirror / Atom feed
From: Dominik Csapak <d.csapak@proxmox.com>
To: pbs-devel@lists.proxmox.com
Subject: [pbs-devel] [PATCH proxmox-backup 0/4] improve datastore removal/creation
Date: Tue,  1 Jun 2021 14:13:46 +0200	[thread overview]
Message-ID: <20210601121350.19919-1-d.csapak@proxmox.com> (raw)

by fixing the leftover datastore references
adding an api call that creates the datastore as a worker
and shows a removal butto in the gui

i am not sure if adding another api call for creation is a good idea,
or if we want to use the chance of a major release to do a breaking
api change and simply change the existing api call to a worker?

also i'd like to know if anybody thinks it would make sense to add another remove
button to the datastore panel itself? (e.g. summary?, or in the panel at
the top?). it *may* be possible to add one in the list itself, though
this seems more complicated (extjs treelists are not that flexible)

Dominik Csapak (4):
  proxmox-backup-proxy: fix leftover references on datastore removal
  api2/confif/datastore: add create datastore api call in a worker
  backup/chunk_store: optionally log progress on creation
  ui: DataStoreList: add remove button

 src/api2/config/datastore.rs    | 113 +++++++++++++++++++++++++++++++-
 src/backup/chunk_store.rs       |  11 ++--
 src/backup/datastore.rs         |  16 +++++
 src/bin/proxmox-backup-proxy.rs |  11 ++++
 src/server.rs                   |   8 +++
 www/Utils.js                    |   1 +
 www/datastore/DataStoreList.js  |  39 +++++++++++
 www/window/DataStoreEdit.js     |   3 +-
 8 files changed, 194 insertions(+), 8 deletions(-)

-- 
2.20.1





             reply	other threads:[~2021-06-01 12:13 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-06-01 12:13 Dominik Csapak [this message]
2021-06-01 12:13 ` [pbs-devel] [PATCH proxmox-backup 1/4] proxmox-backup-proxy: fix leftover references on datastore removal Dominik Csapak
2021-06-01 14:25   ` Thomas Lamprecht
2021-06-01 12:13 ` [pbs-devel] [PATCH proxmox-backup 2/4] api2/confif/datastore: add create datastore api call in a worker Dominik Csapak
2021-06-01 14:33   ` Thomas Lamprecht
2021-06-02  6:34     ` Dietmar Maurer
2021-06-01 12:13 ` [pbs-devel] [PATCH proxmox-backup 3/4] backup/chunk_store: optionally log progress on creation Dominik Csapak
2021-06-01 12:13 ` [pbs-devel] [PATCH proxmox-backup 4/4] ui: DataStoreList: add remove button Dominik Csapak
2021-06-01 14:37   ` Thomas Lamprecht

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=20210601121350.19919-1-d.csapak@proxmox.com \
    --to=d.csapak@proxmox.com \
    --cc=pbs-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