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
next 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