[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 2/6] migration: introduce savevm, loadvm, delvm QMP commands
From: |
Peter Krempa |
Subject: |
Re: [PATCH 2/6] migration: introduce savevm, loadvm, delvm QMP commands |
Date: |
Tue, 7 Jul 2020 08:38:53 +0200 |
User-agent: |
Mutt/1.14.3 (2020-06-14) |
On Mon, Jul 06, 2020 at 18:15:55 +0200, Kevin Wolf wrote:
> Am 03.07.2020 um 18:02 hat Daniel P. Berrangé geschrieben:
> > On Fri, Jul 03, 2020 at 04:49:33PM +0100, Dr. David Alan Gilbert wrote:
> > > * Daniel P. Berrangé (berrange@redhat.com) wrote:
> > > > On Thu, Jul 02, 2020 at 01:12:52PM -0500, Eric Blake wrote:
> > > > > On 7/2/20 12:57 PM, Daniel P. Berrangé wrote:
[...]
> > migration only does vmstate, not disks. The current blockdev commands
> > are all related to external snapshots, nothing for internal snapshots
> > AFAIK. So we still need commands to load/save internal snapshots of
> > the disk data in the qcow2 files.
> >
> > So we could look at loadvm/savevm conceptually as a syntax sugar above
> > a migration transport that targets disk images, and blockdev QMP command
> > that can do internal snapshots. Neither of these exist though and feel
> > like a significantly larger amount of work than using existing functionality
> > that is currently working.
>
> There is blockdev-snapshot-internal-sync, which does a disk-only
> snapshot of a single node. A snapshot of multiple nodes can be taken by
> putting multiple blockdev-snapshot-internal-sync inside a 'transaction'
> command.
Libvirt never implemented support for disk-only internal snapshots as I
didn't think they are worth it. We also made a mistake by using the
VIR_DOMAIN_SNAPSHOT_DISK_ONLY to switch to an external snapshot, so
while the XML can modify the snapshot to be internal it's not very clear
nor user-friendly to force an internal disk only snapshot.
> If we want to build on top of this, we'd have to implement a
> transactionable command for storing only the VM state to a specific
> node. This would probably still be a long-running job.
IMO we really want this also for external snapshots. Driving the
migration as standard migration is really suboptimal especially when the
user wants minimal downtime. Transactioning a post-copy style copy-on
write migration would simplify this a lot. I agree though that this is
for a different conversation.
- Re: [PATCH 2/6] migration: introduce savevm, loadvm, delvm QMP commands, (continued)
- Re: [PATCH 2/6] migration: introduce savevm, loadvm, delvm QMP commands, Dr. David Alan Gilbert, 2020/07/03
- Re: [PATCH 2/6] migration: introduce savevm, loadvm, delvm QMP commands, Daniel P . Berrangé, 2020/07/03
- Re: [PATCH 2/6] migration: introduce savevm, loadvm, delvm QMP commands, Dr. David Alan Gilbert, 2020/07/03
- Re: [PATCH 2/6] migration: introduce savevm, loadvm, delvm QMP commands, Daniel P . Berrangé, 2020/07/03
- Re: [PATCH 2/6] migration: introduce savevm, loadvm, delvm QMP commands, Dr. David Alan Gilbert, 2020/07/03
- Re: [PATCH 2/6] migration: introduce savevm, loadvm, delvm QMP commands, Daniel P . Berrangé, 2020/07/03
- Re: [PATCH 2/6] migration: introduce savevm, loadvm, delvm QMP commands, Dr. David Alan Gilbert, 2020/07/03
- Re: [PATCH 2/6] migration: introduce savevm, loadvm, delvm QMP commands, Peter Krempa, 2020/07/03
- Re: [PATCH 2/6] migration: introduce savevm, loadvm, delvm QMP commands, Dr. David Alan Gilbert, 2020/07/03
- Re: [PATCH 2/6] migration: introduce savevm, loadvm, delvm QMP commands, Kevin Wolf, 2020/07/06
- Re: [PATCH 2/6] migration: introduce savevm, loadvm, delvm QMP commands,
Peter Krempa <=
- Re: [PATCH 2/6] migration: introduce savevm, loadvm, delvm QMP commands, Kevin Wolf, 2020/07/07
- Re: [PATCH 2/6] migration: introduce savevm, loadvm, delvm QMP commands, Peter Krempa, 2020/07/07
Re: [PATCH 2/6] migration: introduce savevm, loadvm, delvm QMP commands, Denis V. Lunev, 2020/07/03
[PATCH 3/6] block: add ability to filter out blockdevs during snapshot, Daniel P . Berrangé, 2020/07/02
[PATCH 4/6] block: allow specifying name of block device for vmstate storage, Daniel P . Berrangé, 2020/07/02
[PATCH 5/6] migration: support excluding block devs in QMP snapshot commands, Daniel P . Berrangé, 2020/07/02
[PATCH 6/6] migration: support picking vmstate disk in QMP snapshot commands, Daniel P . Berrangé, 2020/07/02