From: Fabian Ebner <f.ebner@proxmox.com>
To: pve-devel@lists.proxmox.com
Subject: [pve-devel] [PATCH-SERIES v11 qemu-server/manager] API for disk import and OVF
Date: Mon, 7 Mar 2022 13:17:28 +0100 [thread overview]
Message-ID: <20220307121743.60206-1-f.ebner@proxmox.com> (raw)
Extend qm importdisk/importovf functionality to the API.
Changes from v10:
* Add fix for device unplug issue (patch #1).
* Add fixes related to calling create_disks() (patches #2 #3).
* Refactor clone_disk() in preparation to re-use it for import
(patches #4 #5 #6).
* Add patch to print the newly allocated drive (patch #14).
* Switch to using clone_disk for PVE-managed volumes and check for
VM.Clone in the permission check if there is an owner ID.
* Require <storeid>:0 syntax when using import-from. Allowing
other values than 0 for the size would be confusing, because
with import-from that size is never used (the size of the source
image is).
* Avoid making all foreach_volume iterators parse with the
extended schema. Instead, provide a custom iterator for the
places where it's actually required.
Still missing GUI integration for import from ovf, but that will be it's
own series.
Previous discussion:
https://lists.proxmox.com/pipermail/pve-devel/2022-January/051379.html
qemu-server:
Dominic Jäger (1):
api: support VM disk import
Fabian Ebner (13):
device unplug: verify that unplugging scsi disk completed
api: create disks: always activate/update size when attaching existing
volume
api: update: pass correct config when creating disks
clone disk: remove check for min QEMU version 2.7
clone disk: group source and target parameters
clone disk: allow cloning from an unused or unreferenced disk
schema: add pve-volume-id-or-absolute-path
parse ovf: untaint path when calling file_size_info
api: add endpoint for parsing .ovf files
image convert: allow block device as source
api: factor out check/cleanup for drive params
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 | 365 ++++++++++++++++++++++++++---------
PVE/API2/Qemu/Makefile | 2 +-
PVE/API2/Qemu/OVF.pm | 55 ++++++
PVE/QemuServer.pm | 99 +++++++---
PVE/QemuServer/Drive.pm | 94 ++++++---
PVE/QemuServer/ImportDisk.pm | 2 +-
PVE/QemuServer/OVF.pm | 9 +-
7 files changed, 487 insertions(+), 139 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
next reply other threads:[~2022-03-07 12:17 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-03-07 12:17 Fabian Ebner [this message]
2022-03-07 12:17 ` [pve-devel] [PATCH v11 qemu-server 01/14] device unplug: verify that unplugging scsi disk completed Fabian Ebner
2022-03-07 12:17 ` [pve-devel] [PATCH v11 qemu-server 02/14] api: create disks: always activate/update size when attaching existing volume Fabian Ebner
2022-03-07 12:17 ` [pve-devel] [PATCH v11 qemu-server 03/14] api: update: pass correct config when creating disks Fabian Ebner
2022-03-07 12:17 ` [pve-devel] [PATCH v11 qemu-server 04/14] clone disk: remove check for min QEMU version 2.7 Fabian Ebner
2022-03-07 12:17 ` [pve-devel] [PATCH v11 qemu-server 05/14] clone disk: group source and target parameters Fabian Ebner
2022-03-07 12:17 ` [pve-devel] [PATCH v11 qemu-server 06/14] clone disk: allow cloning from an unused or unreferenced disk Fabian Ebner
2022-03-07 12:17 ` [pve-devel] [PATCH v11 qemu-server 07/14] schema: add pve-volume-id-or-absolute-path Fabian Ebner
2022-03-07 12:17 ` [pve-devel] [PATCH v11 qemu-server 08/14] parse ovf: untaint path when calling file_size_info Fabian Ebner
2022-03-07 12:17 ` [pve-devel] [PATCH v11 qemu-server 09/14] api: add endpoint for parsing .ovf files Fabian Ebner
2022-03-07 12:17 ` [pve-devel] [PATCH v11 qemu-server 10/14] image convert: allow block device as source Fabian Ebner
2022-03-07 12:17 ` [pve-devel] [PATCH v11 qemu-server 11/14] api: factor out check/cleanup for drive params Fabian Ebner
2022-03-07 12:17 ` [pve-devel] [PATCH v11 qemu-server 12/14] schema: drive: use separate schema when disk allocation is possible Fabian Ebner
2022-03-07 12:17 ` [pve-devel] [PATCH v11 qemu-server 13/14] api: support VM disk import Fabian Ebner
2022-03-07 12:17 ` [pve-devel] [PATCH v11 qemu-server 14/14] api: update vm: print drive string for newly allocated/imported drives Fabian Ebner
2022-03-07 12:17 ` [pve-devel] [PATCH v11 manager 1/1] api: nodes: add readovf endpoint Fabian Ebner
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=20220307121743.60206-1-f.ebner@proxmox.com \
--to=f.ebner@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.
Service provided by Proxmox Server Solutions GmbH | Privacy | Legal