[PATCH 00/10] Unify argument name of migration APIs

Michal Privoznik via Devel posted 10 patches 3 months ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1749125656.git.mprivozn@redhat.com
src/driver-hypervisor.h             | 16 +++----
src/esx/esx_driver.c                |  2 +-
src/libvirt_internal.h              | 16 +++----
src/qemu/qemu_driver.c              | 18 ++++----
src/qemu/qemu_migration.c           | 72 ++++++++++++++---------------
src/remote/remote_daemon_dispatch.c | 10 ++--
src/remote/remote_driver.c          | 24 +++++-----
src/remote/remote_protocol.x        | 16 +++----
src/remote_protocol-structs         | 16 +++----
src/rpc/gendispatch.pl              | 36 +++++++--------
10 files changed, 113 insertions(+), 113 deletions(-)
[PATCH 00/10] Unify argument name of migration APIs
Posted by Michal Privoznik via Devel 3 months ago
Some of our APIs have 'bandwidth' argument but then, at internal impl
level it's renamed to 'resource', inconsistently. Since it's really
describing bandwidth that the migration can use, let's rename it.

Michal Prívozník (10):
  src: Unify argument name of virDomainMigratePrepare()
  src: Unify argument name of virDomainMigratePerform()
  src: Unify argument name of virDomainMigratePrepare2()
  src: Unify argument name of virDomainMigratePrepareTunnel()
  src: Unify argument name of virDomainMigratePrepare3()
  src: Unify argument name of virDomainMigratePrepareTunnel3()
  src: Unify argument name of virDomainMigrateBegin3()
  src: Unify argument name of virDomainMigratePerform3()
  qemu: Finish argument rename
  gendispatch: Finish rename of the migration argument

 src/driver-hypervisor.h             | 16 +++----
 src/esx/esx_driver.c                |  2 +-
 src/libvirt_internal.h              | 16 +++----
 src/qemu/qemu_driver.c              | 18 ++++----
 src/qemu/qemu_migration.c           | 72 ++++++++++++++---------------
 src/remote/remote_daemon_dispatch.c | 10 ++--
 src/remote/remote_driver.c          | 24 +++++-----
 src/remote/remote_protocol.x        | 16 +++----
 src/remote_protocol-structs         | 16 +++----
 src/rpc/gendispatch.pl              | 36 +++++++--------
 10 files changed, 113 insertions(+), 113 deletions(-)

-- 
2.49.0
Re: [PATCH 00/10] Unify argument name of migration APIs
Posted by Ján Tomko via Devel 2 months, 1 week ago
On a Thursday in 2025, Michal Privoznik via Devel wrote:
>Some of our APIs have 'bandwidth' argument but then, at internal impl
>level it's renamed to 'resource', inconsistently. Since it's really
>describing bandwidth that the migration can use, let's rename it.
>
>Michal Prívozník (10):
>  src: Unify argument name of virDomainMigratePrepare()
>  src: Unify argument name of virDomainMigratePerform()
>  src: Unify argument name of virDomainMigratePrepare2()
>  src: Unify argument name of virDomainMigratePrepareTunnel()
>  src: Unify argument name of virDomainMigratePrepare3()
>  src: Unify argument name of virDomainMigratePrepareTunnel3()
>  src: Unify argument name of virDomainMigrateBegin3()
>  src: Unify argument name of virDomainMigratePerform3()
>  qemu: Finish argument rename
>  gendispatch: Finish rename of the migration argument
>
> src/driver-hypervisor.h             | 16 +++----
> src/esx/esx_driver.c                |  2 +-
> src/libvirt_internal.h              | 16 +++----
> src/qemu/qemu_driver.c              | 18 ++++----
> src/qemu/qemu_migration.c           | 72 ++++++++++++++---------------
> src/remote/remote_daemon_dispatch.c | 10 ++--
> src/remote/remote_driver.c          | 24 +++++-----
> src/remote/remote_protocol.x        | 16 +++----
> src/remote_protocol-structs         | 16 +++----
> src/rpc/gendispatch.pl              | 36 +++++++--------
> 10 files changed, 113 insertions(+), 113 deletions(-)
>

Reviewed-by: Ján Tomko <jtomko@redhat.com>

Jano