[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