[libvirt] [tck PATCH] network tests: change network address to avoid commonly-used subnet

Laine Stump posted 1 patch 5 years, 9 months ago
Failed in applying to current master (apply log)
scripts/networks/networkxml2hostout/tck-testnet-3.dat      | 4 ++--
scripts/networks/networkxml2hostout/tck-testnet-3.post.dat | 4 ++--
scripts/networks/networkxml2xmlin/tck-testnet-3.xml        | 2 +-
3 files changed, 5 insertions(+), 5 deletions(-)
[libvirt] [tck PATCH] network tests: change network address to avoid commonly-used subnet
Posted by Laine Stump 5 years, 9 months ago
My own machine has a virtual network on 192.168.123.0/24, and I've
been told that it's a common selection for people who create a second
network after libvirt's default (and in the past was used as a manual
alternative for the default network itself when 192.168.122.0/24
casued a conflict). Changing to 192.168.97.0/24 also isn't foolproof,
but it's much less likely to encounter a conflict leading to a failed
test.

Signed-off-by: Laine Stump <laine@laine.org>
---
 scripts/networks/networkxml2hostout/tck-testnet-3.dat      | 4 ++--
 scripts/networks/networkxml2hostout/tck-testnet-3.post.dat | 4 ++--
 scripts/networks/networkxml2xmlin/tck-testnet-3.xml        | 2 +-
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/scripts/networks/networkxml2hostout/tck-testnet-3.dat b/scripts/networks/networkxml2hostout/tck-testnet-3.dat
index 535a9c7..5c61690 100644
--- a/scripts/networks/networkxml2hostout/tck-testnet-3.dat
+++ b/scripts/networks/networkxml2hostout/tck-testnet-3.dat
@@ -24,13 +24,13 @@ dhcp-range=2001:db8:ac10:fd01::1,ra-only
 tck-testbr yes
 #ip -o addr show dev tck-testbr | gawk '{print $4" "$6}' | grep -v link
 10.1.2.1/24 10.1.2.255
-192.168.123.1/24 192.168.123.255
+192.168.97.1/24 192.168.97.255
 172.28.255.241/28 172.28.255.255
 2001:db8:ac10:fd01::1/64 global
 2001:db8:ac10:fe01::1/64 global
 #ip -o route show dev tck-testbr | gawk '{print $1" "$7}'
 10.1.2.0/24 10.1.2.1
 172.28.255.240/28 172.28.255.241
-192.168.123.0/24 192.168.123.1
+192.168.97.0/24 192.168.97.1
 #virsh net-list | grep tck-testnet
  tck-testnet          active     no   no
diff --git a/scripts/networks/networkxml2hostout/tck-testnet-3.post.dat b/scripts/networks/networkxml2hostout/tck-testnet-3.post.dat
index 5a8906c..e525e72 100644
--- a/scripts/networks/networkxml2hostout/tck-testnet-3.post.dat
+++ b/scripts/networks/networkxml2hostout/tck-testnet-3.post.dat
@@ -2,14 +2,14 @@
 #iptables -n -L FORWARD | grep ' 10\.1\.2\.'
 #ip6tables -n -L FORWARD | grep ' 2001:db8:ac10'
 #ps aux | sed -n '/dnsmasq .*tck-testnet/ s|.*\(listen-address 10\.1\.2\.1\).*|\1|p'
-#ps aux | sed -n '/dnsmasq .*tck-testnet/ s|.*\(listen-address 192\.168\.123\.1\).*|\1|p'
+#ps aux | sed -n '/dnsmasq .*tck-testnet/ s|.*\(listen-address 192\.168\.97\.1\).*|\1|p'
 #ps aux | sed -n '/dnsmasq .*tck-testnet/ s|.*\(listen-address 172\.28\.255\.241\).*|\1|p'
 #ps aux | sed -n '/dnsmasq .*tck-testnet/ s|.*\(dhcp-range 10\.1\.2\.2,10\.1\.2\.254\).*|\1|p'
 #ps aux | sed -n '/dnsmasq .*tck-testnet/ s|.*\(listen-address 2001\:db8\:ac10\:fe01\:\:1\).*|\1|p'
 #ps aux | sed -n '/dnsmasq .*tck-testnet/ s|.*\(listen-address 2001\:db8\:ac10\:fd01\:\:1\).*|\1|p'
 #ps aux | sed -n '/radvd --.*tck-testnet-/ s|.*\(\/radvd\/tck-testnet-radvd.conf\).*|\1|p'
 #route -n | grep '10\.1\.2\.'
-#route -n | grep '192\.168\.123\.'
+#route -n | grep '192\.168\.97\.'
 #route -n | grep '172\.28\.255\.240'
 #brctl show | grep tck-testbr
 #ifconfig tck-testbr 2>/dev/null | grep ':10\.1\.2\.'
diff --git a/scripts/networks/networkxml2xmlin/tck-testnet-3.xml b/scripts/networks/networkxml2xmlin/tck-testnet-3.xml
index 9691241..043ae3b 100644
--- a/scripts/networks/networkxml2xmlin/tck-testnet-3.xml
+++ b/scripts/networks/networkxml2xmlin/tck-testnet-3.xml
@@ -10,7 +10,7 @@
       <host mac="00:16:3e:3e:a9:1a" name="b.example.com" ip="10.1.2.11" />
     </dhcp>
   </ip>
-  <ip family="ipv4" address="192.168.123.1" netmask="255.255.255.0">
+  <ip family="ipv4" address="192.168.97.1" netmask="255.255.255.0">
   </ip>
   <ip family="ipv6" address="2001:db8:ac10:fe01::1" prefix="64">
   </ip>
-- 
2.14.4

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [tck PATCH] network tests: change network address to avoid commonly-used subnet
Posted by Daniel P. Berrangé 5 years, 9 months ago
On Fri, Jun 08, 2018 at 10:59:58AM -0400, Laine Stump wrote:
> My own machine has a virtual network on 192.168.123.0/24, and I've
> been told that it's a common selection for people who create a second
> network after libvirt's default (and in the past was used as a manual
> alternative for the default network itself when 192.168.122.0/24

Heh, me too in fact :-)

> casued a conflict). Changing to 192.168.97.0/24 also isn't foolproof,
> but it's much less likely to encounter a conflict leading to a failed
> test.
> 
> Signed-off-by: Laine Stump <laine@laine.org>
> ---
>  scripts/networks/networkxml2hostout/tck-testnet-3.dat      | 4 ++--
>  scripts/networks/networkxml2hostout/tck-testnet-3.post.dat | 4 ++--
>  scripts/networks/networkxml2xmlin/tck-testnet-3.xml        | 2 +-
>  3 files changed, 5 insertions(+), 5 deletions(-)

Reviewed-by: Daniel P. Berrangé <berrange@redhat.com>

 
Regards,
Daniel
-- 
|: https://berrange.com      -o-    https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org         -o-            https://fstop138.berrange.com :|
|: https://entangle-photo.org    -o-    https://www.instagram.com/dberrange :|

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