From: Thomas Lamprecht <t.lamprecht@proxmox.com>
To: Proxmox Datacenter Manager development discussion
<pdm-devel@lists.proxmox.com>,
Dominik Csapak <d.csapak@proxmox.com>
Subject: Re: [pdm-devel] [RFC PATCH datacenter-manager 0/2] use streaming content api
Date: Thu, 9 Oct 2025 17:10:14 +0200 [thread overview]
Message-ID: <a6d7056f-01c1-4127-97d1-f9ebc4e86a49@proxmox.com> (raw)
In-Reply-To: <0d773f60-66e2-45a1-8345-36c992be019e@proxmox.com>
Am 09.10.25 um 11:32 schrieb Dominik Csapak:
> one thing i forgot to ask here is:
>
> how should we handle pbs3/4 compatibility?
> We have IMHO the following options:
>
> * Implement a fallback in the UI for pbs3
> can be ok, we could duplicate the panel and drop
> the old one when pbs3 is not supported anymore?
Hmm, the pdm CLI client got not much love recently, but it is there,
so IMO it would be a bit nicer if the PDM backend abstracts such things
away, at least if doing so is relatively easy (as in, same response
can be generated without much extra work).
> * Implement a fallback in the API for pbs3
> most robust solution IMO, gui can stay mostly the same, code
> should not be that much, can also be dropped if we don't
> want to support pbs3 anymore
That would be an OK option to do.
> * don't support pbs3 at all
> not really an option i think
No, IMO we really should try very hard to provide at least one version
of backward compat and one version of forward compat (based off Debian
releases).
I.e. PDM released for a Debian release X should support X, X-1 and X+1.
As otherwise upgrades will get a real PITA fast for users, as they would
need to upgrade all in a lockstep.
> * also commit the changes to pbs stable-3 branch
> maybe the easiest way, but here's the question
> where we draw the line for older products
> regarding features/bugs/etc.
The good thing is that we're relatively free to chose and can do so
differently on a case by case basis.
Here I'd find it OK to backport the whole thing, as it can also be
nice for the PBS CLI client (when it actually uses this in the
future). But that only covers latest 3.4, which is fine, but naturally
not _that_ great. So maybe either do both (as of now the master and the
stable-3 branch did not diverge too much, so might be justifiable) or
just do the fallback in the API, which then will also work for even older
PBS releases (not that I want to support these EOL versions actively,
but it's a slight benefit for UX).
_______________________________________________
pdm-devel mailing list
pdm-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pdm-devel
prev parent reply other threads:[~2025-10-09 15:10 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-10-08 13:54 Dominik Csapak
2025-10-08 13:54 ` [pdm-devel] [RFC PATCH datacenter-manager 1/2] server: add new streaming 'content' api call for pbs Dominik Csapak
2025-10-08 13:54 ` [pdm-devel] [RFC PATCH datacenter-manager 2/2] ui: pbs: snapshot list: change to streaming 'content' api call Dominik Csapak
2025-10-09 9:32 ` [pdm-devel] [RFC PATCH datacenter-manager 0/2] use streaming content api Dominik Csapak
2025-10-09 15:10 ` Thomas Lamprecht [this message]
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=a6d7056f-01c1-4127-97d1-f9ebc4e86a49@proxmox.com \
--to=t.lamprecht@proxmox.com \
--cc=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 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.