LGTM now, and tested fine (this time with ISOs *and* CT templates ;) )

two tiny nits:

the commit message of the first message could mention that the parameter is 
necessary
to differentiate between different needs for different content types, but
imho not super important now (could be fixed up e.g.)

the last hunk in the second patch seems leftover, but it only removes an empty 
line,
so it should do any harm.

Reviewed-by: Dominik Csapak <d.csa...@proxmox.com>
Tested-by: Dominik Csapak <d.csa...@proxmox.com>


_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel

Reply via email to