Re: [libvirt] [PATCH] migration: Document we don't copy storage during offline migration

2016-09-21 Thread Martin Kletzander

On Wed, Sep 21, 2016 at 11:27:20AM +0200, Michal Privoznik wrote:

https://bugzilla.redhat.com/show_bug.cgi?id=1322717

During offline migration, no storage is copied. Nor disks, nor
NVRAM file, nor anything. We use qemu for that and because domain
is not running there's nobody to copy that for us.
We should document this to avoid confusing users.

Signed-off-by: Michal Privoznik 
---


ACK


signature.asc
Description: Digital signature
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

[libvirt] [PATCH] migration: Document we don't copy storage during offline migration

2016-09-21 Thread Michal Privoznik
https://bugzilla.redhat.com/show_bug.cgi?id=1322717

During offline migration, no storage is copied. Nor disks, nor
NVRAM file, nor anything. We use qemu for that and because domain
is not running there's nobody to copy that for us.
We should document this to avoid confusing users.

Signed-off-by: Michal Privoznik 
---
 docs/migration.html.in |  5 +++--
 src/libvirt-domain.c   | 12 
 2 files changed, 15 insertions(+), 2 deletions(-)

diff --git a/docs/migration.html.in b/docs/migration.html.in
index 8efdd7f..2768955 100644
--- a/docs/migration.html.in
+++ b/docs/migration.html.in
@@ -148,8 +148,9 @@
   virsh dumpxml on source host followed by
   virsh define on destination host, as offline migration
   will run the pre-migration hook to update the domain XML on
-  destination host. Currently, copying non-shared storage is not
-  supported during offline migration.
+  destination host. Currently, copying non-shared storage or other file
+  based storages (e.g. UEFI variable storage) is not supported during
+  offline migration.
 
 
 Migration URIs
diff --git a/src/libvirt-domain.c b/src/libvirt-domain.c
index fa82e49..9ba9f49 100644
--- a/src/libvirt-domain.c
+++ b/src/libvirt-domain.c
@@ -3574,6 +3574,10 @@ virDomainMigrateUnmanaged(virDomainPtr domain,
  * not support this feature and will return an error if bandwidth
  * is not 0.
  *
+ * Users should note that implementation of VIR_MIGRATE_OFFLINE
+ * flag in some drivers does not copy storage or any other file
+ * based storage (e.g. UEFI variable storage).
+ *
  * Enabling the VIR_MIGRATE_POSTCOPY flag tells libvirt to enable post-copy
  * migration.  Use virDomainMigrateStartPostCopy to switch migration into
  * the post-copy mode.  See virDomainMigrateStartPostCopy for more details
@@ -3791,6 +3795,10 @@ virDomainMigrate(virDomainPtr domain,
  * not support this feature and will return an error if bandwidth
  * is not 0.
  *
+ * Users should note that implementation of VIR_MIGRATE_OFFLINE
+ * flag in some drivers does not copy storage or any other file
+ * based storage (e.g. UEFI variable storage).
+ *
  * Enabling the VIR_MIGRATE_POSTCOPY flag tells libvirt to enable post-copy
  * migration.  Use virDomainMigrateStartPostCopy to switch migration into
  * the post-copy mode.  See virDomainMigrateStartPostCopy for more details
@@ -3980,6 +3988,10 @@ virDomainMigrate2(virDomainPtr domain,
  * can use either VIR_MIGRATE_NON_SHARED_DISK or
  * VIR_MIGRATE_NON_SHARED_INC as they are mutually exclusive.
  *
+ * Users should note that implementation of VIR_MIGRATE_OFFLINE
+ * flag in some drivers does not copy storage or any other file
+ * based storage (e.g. UEFI variable storage).
+ *
  * Enabling the VIR_MIGRATE_POSTCOPY flag tells libvirt to enable post-copy
  * migration.  Use virDomainMigrateStartPostCopy to switch migration into
  * the post-copy mode.  See virDomainMigrateStartPostCopy for more details
-- 
2.8.4

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list