[pve-devel] applied: [PATCH manager] fix #5753: api: add 'pstart' member to task status return schema

Thomas Lamprecht t.lamprecht at proxmox.com
Fri Oct 4 16:26:14 CEST 2024


On 04/10/2024 10:34, Fabian Grünbichler wrote:
> using the definition already used in the task index API schema in the same
> module.
> 
> Signed-off-by: Fabian Grünbichler <f.gruenbichler at proxmox.com>
> ---
> starttime has a disagreeing schema atm - 'number' vs 'integer'. since this is
> the return schema it doesn't really matter I guess, but might be worth it to
> settle on one or the other for all 4 instances of pstart/starttime here?

yeah, that would be nice. Using integer, like you did, might be the best
option for non-fractional UNIX seconds I think. 

> 
>  PVE/API2/Tasks.pm | 3 +++
>  1 file changed, 3 insertions(+)
> 
>

applied, thanks!




More information about the pve-devel mailing list