From: "Fabian Grünbichler" <f.gruenbichler@proxmox.com>
To: Proxmox VE development discussion <pve-devel@lists.proxmox.com>
Subject: [pve-devel] applied: [PATCH-SERIES v13 qemu-server/manager] API for disk import and OVF
Date: Mon, 04 Apr 2022 16:59:58 +0200 [thread overview]
Message-ID: <1649084340.1kiggnmvht.astroid@nora.none> (raw)
In-Reply-To: <20220317113107.60466-1-f.ebner@proxmox.com>
the qemu-server part with one slight fixup (readovf API endpoint renamed
from 'index' to 'readovf'). the manager part should be applied once
we've bumped qemu-server or when the GUI patches are ready.
On March 17, 2022 12:30 pm, Fabian Ebner wrote:
> Extend qm importdisk/importovf functionality to the API.
>
> Changes from v12:
> * Drop already applied patches.
> * Add some follow-up improvements related to clone:
> * Aborting early for TPM state restriction.
> * Check against a corner case with drive-mirror with different
> drive names but with the same VM ID.
> * Clone EFI disk from snapshot when specified.
> * Make readovf endpoint 'protected => 1'.
> * Group schema-related changes into a patch.
> * Some smaller cleanups/improvements to checks (see individual
> patches).
>
> Still missing GUI integration for import from ovf, but that will be it's
> own series.
>
> Previous discussion here:
> https://lists.proxmox.com/pipermail/pve-devel/2022-March/052005.html
>
>
> qemu-server:
>
> Dominic Jäger (1):
> api: support VM disk import
>
> Fabian Ebner (7):
> clone disk: assert that drive name is the same for drive-mirror on
> single VM
> clone disk: move check against cloning TPM state of running VM to
> beginning
> api: clone vm: check against cloning running TPM state early
> clone disk: also clone EFI disk from snapshot
> api: add endpoint for parsing .ovf files
> schema: drive: use separate schema when disk allocation is possible
> api: update vm: print drive string for newly allocated/imported drives
>
> PVE/API2/Qemu.pm | 255 ++++++++++++++++++++++++++++++-----
> PVE/API2/Qemu/Makefile | 2 +-
> PVE/API2/Qemu/OVF.pm | 53 ++++++++
> PVE/QemuServer.pm | 70 ++++++++--
> PVE/QemuServer/Drive.pm | 95 +++++++++----
> PVE/QemuServer/ImportDisk.pm | 4 +-
> 6 files changed, 412 insertions(+), 67 deletions(-)
> create mode 100644 PVE/API2/Qemu/OVF.pm
>
>
> manager:
>
> Fabian Ebner (1):
> api: nodes: add readovf endpoint
>
> PVE/API2/Nodes.pm | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> --
> 2.30.2
>
>
>
> _______________________________________________
> pve-devel mailing list
> pve-devel@lists.proxmox.com
> https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel
>
prev parent reply other threads:[~2022-04-04 15:00 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-03-17 11:30 [pve-devel] " Fabian Ebner
2022-03-17 11:30 ` [pve-devel] [PATCH v13 qemu-server 1/8] clone disk: assert that drive name is the same for drive-mirror on single VM Fabian Ebner
2022-03-17 11:31 ` [pve-devel] [PATCH v13 qemu-server 2/8] clone disk: move check against cloning TPM state of running VM to beginning Fabian Ebner
2022-03-17 11:31 ` [pve-devel] [PATCH v13 qemu-server 3/8] api: clone vm: check against cloning running TPM state early Fabian Ebner
2022-03-17 11:31 ` [pve-devel] [PATCH v13 qemu-server 4/8] clone disk: also clone EFI disk from snapshot Fabian Ebner
[not found] ` <<20220317113107.60466-5-f.ebner@proxmox.com>
2022-04-04 14:58 ` Fabian Grünbichler
2022-03-17 11:31 ` [pve-devel] [PATCH v13 qemu-server 5/8] api: add endpoint for parsing .ovf files Fabian Ebner
2022-03-17 11:31 ` [pve-devel] [PATCH v13 qemu-server 6/8] schema: drive: use separate schema when disk allocation is possible Fabian Ebner
2022-03-17 11:31 ` [pve-devel] [PATCH v13 qemu-server 7/8] api: support VM disk import Fabian Ebner
[not found] ` <CAOKSTBvWfnXW9RUo2ddKACTgZV9Be-7d=9g8zPoduaDH=XP1Zw@mail.gmail.com>
2022-03-17 12:35 ` Thomas Lamprecht
2022-03-17 11:31 ` [pve-devel] [PATCH v13 qemu-server 8/8] api: update vm: print drive string for newly allocated/imported drives Fabian Ebner
2022-03-17 11:31 ` [pve-devel] [PATCH v13 manager 1/1] api: nodes: add readovf endpoint Fabian Ebner
2022-04-04 14:59 ` Fabian Grünbichler [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1649084340.1kiggnmvht.astroid@nora.none \
--to=f.gruenbichler@proxmox.com \
--cc=pve-devel@lists.proxmox.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.