public inbox for pve-devel@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal