Fix bonehead errors in <hostdev> <teaming> support

[libvirt PATCH 0/2] Fix bonehead errors in <hostdev> <teaming> support

Laine Stump posted 2 patches 3 years, 2 months ago

Download series mbox

Test syntax-check failed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20210223224221.683479-1-laine@redhat.com

docs/formatdomain.rst     |  3 +--
src/qemu/qemu_migration.c | 16 ++++++++++------
2 files changed, 11 insertions(+), 8 deletions(-)

Expand all Fold all

[libvirt PATCH 0/2] Fix bonehead errors in <hostdev> <teaming> support

Posted by Laine Stump 3 years, 2 months ago

I hadn't caught the code omission because my migration tester system
was out of commission, and I stupidly assumed migration would
work. The docs typo was pure lack of attention.

This (sub) feature will be new for this release, and there is an open
BZ for it downstream, so it would be nice to get the fix ACKed and
pushed in time for the release.

Laine Stump (2):
  qemu: allow migration of generic <hostdev> with <teaming>
  docs: fix bad cut/paste in <teaming> example

 docs/formatdomain.rst     |  3 +--
 src/qemu/qemu_migration.c | 16 ++++++++++------
 2 files changed, 11 insertions(+), 8 deletions(-)

-- 
2.29.2

Re: [libvirt PATCH 0/2] Fix bonehead errors in <hostdev> <teaming> support

Posted by Ján Tomko 3 years, 1 month ago

On a Tuesday in 2021, Laine Stump wrote:
>I hadn't caught the code omission because my migration tester system
>was out of commission, and I stupidly assumed migration would
>work. The docs typo was pure lack of attention.
>
>This (sub) feature will be new for this release, and there is an open
>BZ for it downstream, so it would be nice to get the fix ACKed and
>pushed in time for the release.
>
>Laine Stump (2):
>  qemu: allow migration of generic <hostdev> with <teaming>
>  docs: fix bad cut/paste in <teaming> example
>
> docs/formatdomain.rst     |  3 +--
> src/qemu/qemu_migration.c | 16 ++++++++++------
> 2 files changed, 11 insertions(+), 8 deletions(-)
>

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

Jano