[libvirt] [tck PATCH 0/2] set CTRL_IP_LEARNING and DHCPSERVER in filter during no-ip-spoofing test

Laine Stump posted 2 patches 6 years, 1 month ago
Failed in applying to current master (apply log)
There is a newer version of this series
lib/Sys/Virt/TCK.pm                   | 11 ++++++++---
lib/Sys/Virt/TCK/DomainBuilder.pm     |  8 +++++++-
lib/Sys/Virt/TCK/NetworkHelpers.pm    | 10 ++++++++++
scripts/nwfilter/220-no-ip-spoofing.t |  9 ++++++++-
4 files changed, 33 insertions(+), 5 deletions(-)
[libvirt] [tck PATCH 0/2] set CTRL_IP_LEARNING and DHCPSERVER in filter during no-ip-spoofing test
Posted by Laine Stump 6 years, 1 month ago
We've recently discovered two separate bugs that caused libvirt's
"DHCP Snooping" feature to not work:

https://bugzilla.redhat.com/show_bug.cgi?id=1529338 - libvirt regression
https://bugzilla.redhat.com/show_bug.cgi?id=1547237 - libpcap regression

Since we didn't have any test suite covering that code, we had the
embarrassment of learning of it from someone else's QE (RHV/oVirt QE
at Red Hat).

This series adds the necessary stuff to the test domain config of
libvirt-tck's "no-ip-spoofing" test to exercise the DHCPSnoop thread.

(There may be a much better way of dealing with a hash-inside-a-hash;
I am an imbecile at perl, and arrived at this code by trial, error,
and google searches).


Laine Stump (2):
  new helper function get_network_ip()
  set CTRL_IP_LEARNING and DHCPSERVER in filter during no-ip-spoofing
    test

 lib/Sys/Virt/TCK.pm                   | 11 ++++++++---
 lib/Sys/Virt/TCK/DomainBuilder.pm     |  8 +++++++-
 lib/Sys/Virt/TCK/NetworkHelpers.pm    | 10 ++++++++++
 scripts/nwfilter/220-no-ip-spoofing.t |  9 ++++++++-
 4 files changed, 33 insertions(+), 5 deletions(-)

-- 
2.14.3

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [tck PATCH 0/2] set CTRL_IP_LEARNING and DHCPSERVER in filter during no-ip-spoofing test
Posted by Laine Stump 6 years, 1 month ago
Self-NACK to this series. I found a bug in one of the patches, and
modified the other to be more useful in a couple other places.

I posted a V2 series with the same subject line.

On 02/28/2018 01:30 PM, Laine Stump wrote:
> We've recently discovered two separate bugs that caused libvirt's
> "DHCP Snooping" feature to not work:
>
> https://bugzilla.redhat.com/show_bug.cgi?id=1529338 - libvirt regression
> https://bugzilla.redhat.com/show_bug.cgi?id=1547237 - libpcap regression
>
> Since we didn't have any test suite covering that code, we had the
> embarrassment of learning of it from someone else's QE (RHV/oVirt QE
> at Red Hat).
>
> This series adds the necessary stuff to the test domain config of
> libvirt-tck's "no-ip-spoofing" test to exercise the DHCPSnoop thread.
>
> (There may be a much better way of dealing with a hash-inside-a-hash;
> I am an imbecile at perl, and arrived at this code by trial, error,
> and google searches).


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