]> git.proxmox.com Git - pve-common.git/commit
job registry: avoid injecting the section id unconditionally in configs
authorThomas Lamprecht <t.lamprecht@proxmox.com>
Sun, 13 Nov 2022 10:50:40 +0000 (11:50 +0100)
committerThomas Lamprecht <t.lamprecht@proxmox.com>
Sun, 13 Nov 2022 10:50:42 +0000 (11:50 +0100)
commitec9e46fae6581ea0e48ed7530a8c1a10eed9828c
tree745da5a09b34cb61c5956aea6e82dbd0a5cb1864
parent2bbf0eae8c1629b7f624512fa87e69e4bbb9afcd
job registry: avoid injecting the section id unconditionally in configs

this can result in a broken config due to it getting written out on
write_config serialization, and if a plugin did not declare `id` as
an option it understood (none do currently), it would then fail the
next parse, far from ideal...

As the section ID is available already anyway we should probably just
drop this, but for now avoid rushed changes and just make it
conforming to section config semantics and check if the option is
actually understood by the respective section type we're working on.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
src/PVE/Job/Registry.pm