Chris Evich
Chris Evich
@ypu Today I learned of a tricky circular dependency that was causing unrelated unittests to fail, but only when run in parallel. Turns out, it was related to some changes...
@ldoktor I'm looking at updating virtnet in `qemu_vm` and `libvirt_vm` to fix a state-mismatch problem @ypu is running into. However, I've come across a few uses of your `qemu_devices`, and...
@ldoktor hahahaha, but will networking avoid you! It is very tricky with many corner cases, so I understand. Though by my reading of what you have already in `vm.create` and...
@ldoktor yeah, looking at this more, it really seems to me that `qemu_vm` doesn't need `VirtNetDB` at all. It can just parse params into qdevice. Then `needs_restart` can just check...
@ypu Okay, after discussing much with @lmr, he is on-board with having `qemu-devices` take over for a the `address_pool` part of `virtnet` in `qemu_vm` (formerly VirtNetDB). However, I just finished...
@ypu I've got most of the basic qemu-kvm tests "working", including multi-nic. However, as I know from experience there are likely some corner-cases and odd situations I need to test...
@ypu Amazingly, after I got past a few unrelated problems, I am running qemu-kvm tests for boot, ping.multi_nics, reboot, and shutdown with no problem and new networking code. I decided...
@ypu @lmr @ldoktor Yay! First draft: https://github.com/autotest/virt-test/pull/1217 Please check it out when you have time. I know it's _HUGE_ set of changes, so I'm opening it up early so y'all...
@ldoktor As far as qemu_devices go, let me know what you think about using it to replace the BaseVM's use of VirtnetDB. It's really qemu-kvm specific and probably doesn't need...
@kylazhang would you mind taking a look at this? IIRC There were some changes in the autotest cgroups stuff recently and this test never got updated. I'd fix it if...