Re: [PATCH v3 0/5] test: fix nodedev mdev XML regression

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 4/9/24 16:56, Cole Robinson wrote:
> The virt-manager test suite is busted with libvirt 10.1.0+ after
> this commit:
> 
> commit e67bca23e4fe38a3491749f724b9edf743d0e916
> Author: Boris Fiuczynski <fiuczy@xxxxxxxxxxxxx>
> Date:   Thu Feb 22 14:02:01 2024 +0100
> 
>     nodedev: add an active config to mdev
> 
> See patch #5 for the full explanation. First 4 patches are nodedev
> test driver improvements I hit when debugging this
> 
> v3:
>   really truly send to the correct list
> v2:
>   Send to the correct mailing list
>   Fix version strings in test driver table
> 
> Cole Robinson (5):
>   test: Fix `virsh nodedev-list`
>   test: Implement virNodeDeviceIsActive
>   test: Implement virNodeDeviceIsPersistent
>   test: make nodedevs active by default
>   conf: nodedev: Fill active_config at XML parse time
> 
>  src/conf/node_device_conf.c |  5 ++++-
>  src/test/test_driver.c      | 35 ++++++++++++++++++++++++++++++++++-
>  tests/nodedevxml2xmltest.c  | 15 ---------------
>  3 files changed, 38 insertions(+), 17 deletions(-)
> 

Reviewed-by: Michal Privoznik <mprivozn@xxxxxxxxxx>

But before pushing, also please add Boris' R-B as he also reviewed these
patches.

Michal
_______________________________________________
Devel mailing list -- devel@xxxxxxxxxxxxxxxxx
To unsubscribe send an email to devel-leave@xxxxxxxxxxxxxxxxx




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux