diff options
author | Jeff Lane <jeffrey.lane@canonical.com> | 2017-02-24 16:31:25 -0500 |
---|---|---|
committer | Jeff Lane <jeffrey.lane@canonical.com> | 2017-02-24 16:31:25 -0500 |
commit | 16e040e7c95256529e40feebfae3bf6196152c1a (patch) | |
tree | c6b724751e8534e336f8c5f4b8d51b460807df13 /bin | |
parent | b8e5fb6158ef5ee9b819228aee58266a3b23506b (diff) |
bin/network: don't remove targets that are not in the right subnet, just warn and try them anyway. LP: #1667781
Diffstat (limited to 'bin')
-rwxr-xr-x | bin/network | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/bin/network b/bin/network index 88fd931..4ebb786 100755 --- a/bin/network +++ b/bin/network @@ -448,10 +448,9 @@ def make_target_list(iface, test_targets, log_warnings): target = ipaddress.IPv4Address(test_target_ip) if (target < first_addr) or (target > last_addr): if log_warnings: - logging.warning("test server {} ({}) is NOT ". - format(test_target, target)) - logging.warning("within {}; skipping".format(net)) - return_list.remove(test_target) + logging.warning("Test server {} ({}) is NOT within {}". + format(test_target, target, net)) + logging.warning("This may cause test issues") except ValueError: if log_warnings: logging.warning("Invalid address: {}; skipping". |