On 11.03.21 11:26, Fabian Ebner wrote: > Commit d02262048cbbe91ca8b12f98e3dc7bbab28e4c64 made the property de-facto > non-optional. Partially revert this and instead adapt the printing, making the > behavior match the API description again. The conditional assignment is > already there further down the vmstatus function. > > Signed-off-by: Fabian Ebner <f.eb...@proxmox.com> > --- > > New in v2. > > Alternatively, the property description should be updated, but this patch > makes > it in line with what we do for VMs and is preferable IMHO. >
The schema is what our ABI is, so if that stated this is optional we can make it so. > src/PVE/CLI/pct.pm | 3 ++- > src/PVE/LXC.pm | 1 - > 2 files changed, 2 insertions(+), 2 deletions(-) > > applied, thanks! _______________________________________________ pve-devel mailing list pve-devel@lists.proxmox.com https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
