[pve-devel] [pbs-devel] [PATCH proxmox-backup] fix #4165: SMART: add raw field

Thomas Lamprecht t.lamprecht at proxmox.com
Fri Sep 23 12:31:57 CEST 2022


Am 22/09/2022 um 12:35 schrieb Dominik Csapak:
> 
> the only (minor) thing is that the wt patch could have handled the current situation
> (pve raw+value, pbs normalized+value), e.g by using the field 'convert' or 'calculate'
> methods of extjs (we could have had a 'real_raw' and 'real_normalized' that uses
> the values depending on which exist) then we would not need a depends/breaks on the
> versions (not sure which direction here)
> 
> if everybody else is ok with these dependencies, consider these patches:

This cannot really be solved by dependencies, as the frontend (normally) doesn't
run on the same host as the backend, so a newer frontend (e.g., connected through
another cluster node) "breaks". So as long as its easy as in this case, I really
would do backward compat handling in the frontend.


More information about the pve-devel mailing list