all lists on lists.proxmox.com
 help / color / mirror / Atom feed
* storage API: decompression support for all types in the download-url
@ 2026-02-18 10:08 Charles LESECQ
  2026-02-18 10:41 ` Fabian Grünbichler
  0 siblings, 1 reply; 3+ messages in thread
From: Charles LESECQ @ 2026-02-18 10:08 UTC (permalink / raw)
  To: pve-devel

Hello,

When I requested /api2/json/nodes/{node}/storage/{storage}/download-url 
with the compression option to download .qcow2.xz from Fedora CoreOS, I 
got the message “decompression not supported for qcow2.” This is not 
documented. So I looked in the code, and it seems that only the iso 
format is supported. I can't find the reason for this limitation, so I 
propose removing it. My patch simply removes the associated check, but 
it might be better to do it differently.

Best regards,

Charles LESECQ

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: storage API: decompression support for all types in the download-url
  2026-02-18 10:08 storage API: decompression support for all types in the download-url Charles LESECQ
@ 2026-02-18 10:41 ` Fabian Grünbichler
  2026-02-18 20:41   ` Charles LESECQ
  0 siblings, 1 reply; 3+ messages in thread
From: Fabian Grünbichler @ 2026-02-18 10:41 UTC (permalink / raw)
  To: Charles LESECQ, pve-devel

On February 18, 2026 11:08 am, Charles LESECQ wrote:
> Hello,
> 
> When I requested /api2/json/nodes/{node}/storage/{storage}/download-url 
> with the compression option to download .qcow2.xz from Fedora CoreOS, I 
> got the message “decompression not supported for qcow2.” This is not 
> documented. So I looked in the code, and it seems that only the iso 
> format is supported. I can't find the reason for this limitation, so I 
> propose removing it. My patch simply removes the associated check, but 
> it might be better to do it differently.

I believe this is https://bugzilla.proxmox.com/show_bug.cgi?id=7062 ?
Feel free to subscribe there, and/or assign yourself and send a patch ;)

Fabian




^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: storage API: decompression support for all types in the download-url
  2026-02-18 10:41 ` Fabian Grünbichler
@ 2026-02-18 20:41   ` Charles LESECQ
  0 siblings, 0 replies; 3+ messages in thread
From: Charles LESECQ @ 2026-02-18 20:41 UTC (permalink / raw)
  To: Fabian Grünbichler, pve-devel

Actually, I hadn't seen it. I assigned it to myself. And I've send CLA 
to office@proxmox.com.
However, my patch allows it to work in more cases.

I just added a sign off to the commit message.

Charles

Le 18/02/2026 à 11:41, Fabian Grünbichler a écrit :
> On February 18, 2026 11:08 am, Charles LESECQ wrote:
>> Hello,
>>
>> When I requested /api2/json/nodes/{node}/storage/{storage}/download-url
>> with the compression option to download .qcow2.xz from Fedora CoreOS, I
>> got the message “decompression not supported for qcow2.” This is not
>> documented. So I looked in the code, and it seems that only the iso
>> format is supported. I can't find the reason for this limitation, so I
>> propose removing it. My patch simply removes the associated check, but
>> it might be better to do it differently.
> I believe this ishttps://bugzilla.proxmox.com/show_bug.cgi?id=7062 ?
> Feel free to subscribe there, and/or assign yourself and send a patch ;)
>
> Fabian
>

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2026-02-19  8:24 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-02-18 10:08 storage API: decompression support for all types in the download-url Charles LESECQ
2026-02-18 10:41 ` Fabian Grünbichler
2026-02-18 20:41   ` Charles LESECQ

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal