all lists on lists.proxmox.com
 help / color / mirror / Atom feed
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

      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.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal