public inbox for pdm-devel@lists.proxmox.com
 help / color / mirror / Atom feed
From: Dominik Csapak <d.csapak@proxmox.com>
To: pdm-devel@lists.proxmox.com
Subject: [pdm-devel] [RFC PATCH datacenter-manager 0/3] implement bulk start
Date: Wed, 29 Jan 2025 11:51:39 +0100	[thread overview]
Message-ID: <20250129105142.1291843-1-d.csapak@proxmox.com> (raw)

Sending as RFC, because it's still very rough and i want to get some
early feedback.

This series implements an api call 'bulk-start' which is running on
the pdm itself, that mimics the bulkstart from pve, but without the
node limitation of pve.

Does that make sense? Or would it be better to try to implement that
on pve side? The advantage we have here is that we have an
external view of the cluster, which means that things like node
failures, synchronisation, etc. are much easier to handle.

If we'd implment something like this on PVE, there has to be a node
that has control of the api calls to make (or to schedule something via
pmxcfs) and that is probably much harder to do there (pmxcfs sync queue)
or brings some problems with it (node dies in the middle of an api call)

It's very early, so please don't judge the actual api call code just
now, I'd extend it with failure resulotion, polling the task, etc.

OTOH there is the question if the UI makes sense this way, or if we want
to combine the 'select to view details' and 'select to to a bulk action'
into one. Or if we want to do the bulk actions more like in pve with
a popup that shows the vm list again.

Dominik Csapak (3):
  server: pve api: add new bulkstart api call
  pdm-client: add bulk_start method
  ui: pve tree: add bulk start action

 lib/pdm-client/src/lib.rs |   9 ++-
 server/src/api/pve/mod.rs |  98 +++++++++++++++++++++++++++-
 ui/src/pve/tree.rs        | 133 ++++++++++++++++++++++++++++++++++++--
 3 files changed, 234 insertions(+), 6 deletions(-)

-- 
2.39.5



_______________________________________________
pdm-devel mailing list
pdm-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pdm-devel


             reply	other threads:[~2025-01-29 10:52 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2025-01-29 10:51 Dominik Csapak [this message]
2025-01-29 10:51 ` [pdm-devel] [RFC PATCH datacenter-manager 1/3] server: pve api: add new bulkstart api call Dominik Csapak
2025-01-29 10:51 ` [pdm-devel] [RFC PATCH datacenter-manager 2/3] pdm-client: add bulk_start method Dominik Csapak
2025-01-29 10:51 ` [pdm-devel] [RFC PATCH datacenter-manager 3/3] ui: pve tree: add bulk start action Dominik Csapak
2025-01-29 18:48 ` [pdm-devel] [RFC PATCH datacenter-manager 0/3] implement bulk start Thomas Lamprecht
2025-01-30  8:14   ` Dominik Csapak
2025-01-30 16:15     ` 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=20250129105142.1291843-1-d.csapak@proxmox.com \
    --to=d.csapak@proxmox.com \
    --cc=pdm-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