[pve-devel] [PATCH storage] volume import: assume target API version is at least 9
Thomas Lamprecht
t.lamprecht at proxmox.com
Thu Jul 25 14:38:52 CEST 2024
For the record, we talked about this in person for a bit with the following
outcome:
- there was a bit of a misunderstanding w.r.t. my heavy exaggeration for the
point's sake, I really did not mean that as accusation at all, but that's
now talked out
- it's a good point that the package that was included in the first 7.0 ISO
release was already new enough API version wise, and we also checked the
package archives, and there we also got only new enough libpve-storage-perl
package versions, so we can keep the change as is.
We also agreed that it would be great to mention such analysis in the commit
message the next time, and I know Fiona is very thorough with that stuff, and
that this time it was just not mentioned due to the difference in how upgrade
requirements and recommendations got interpreted by her and me, so I mention
this mostly for other readers, as this applies to all of us.
- we might want to document this expectation w.r.t API backward compat more
definitively in a more approachable way, but should ensure that its clarified
that this is for developers, not users, to avoid users thinking its always
fine to upgrade from an outdated point release to a newer major release.
tl;dr: got sorted out and change can be kept as is
More information about the pve-devel
mailing list