Re: [pve-devel] [PATCH storage/qemu-server/pve-manager] implement ova/ovf import for directory type storages

[Dominik Csapak <d.csapak@proxmox.com>]

ok after a bit of thinking and discussing off-list
the plan to go forward from my side is this:

(please tell if there is something obviously wrong with it or you'd
strongly prefer something differently)

extract on demand vs on upload:
  i'd go with extract on demand because managing the extraction of the tarball + subdir etc is not 
really a win in my book, since we have to have most safeguards anyway and we have the same
issue of where to store/map it etc. also it's not convenient for users that have already
a bunch of ovas and want to store them in a central place, now they'd have to extract
them just for us (and importing should be as small a hassle as it can be)

for placing the extract code, i'd strongly prefer the (future) PVE::GuestImport namespace in 
pve-storage, as that does not pollute the plugin api with irrelevant stuff and is relatively
far away from qemu-server (so we could reuse it later for other things if we need to)

as for the extraction/cleanup step:
i'll reuse the 'images' part of the storage to extract it there with a valid vm disk name.
that way if the cleanup fails, it can be deleted from the ui at least
if the storage does not have an images content type or the user does not have the relevant
privileges, i'd force the user to provide (with a new parameter for creating) a file based
storage with content type images. that can be shown in the gui only for ova imports.
(and would default to the import storage if possible)

i think that were all the "big" questions for this series, please do tell if i forgot something ;)
if no one objects to these things (or has even better ideas to solve some of these),
i'd get started on that ASAP


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