[pbs-devel] applied: [PATCH proxmox-backup] api2/config/tape_backup_job: fix duplicate id parameter

Dietmar Maurer dietmar at proxmox.com
Wed Feb 24 07:13:18 CET 2021


> On 02/23/2021 8:27 PM Dietmar Maurer <dietmar at proxmox.com> wrote:
> 
>  
> Will fix this tomorrow - we need to set the #[fixed] attribute for id

Just saw that we already set the  #[updater(fixed)] attribute.

The docs say:

> Additionally the #[updater(fixed)] option is available to make it illegal 
> for an updater to modify a field (generating an error if it is set), 
> while still allowing it to be used to create a new object via the build_from() method.

So the Updater includes all "fixed" fields (which leads to the duplicate id problem).

I would prefer an Updater which simply omits fixed fields.

Can we change that?





More information about the pbs-devel mailing list