mr-canoehead / network_performance_monitor

Network Performance Monitor - a portable tool for troubleshooting performance issues with home networks
GNU General Public License v3.0
84 stars 21 forks source link

Default to the root namespace for interfaces that don't define one #21

Closed idank closed 3 years ago

idank commented 4 years ago

This is already happening, but adds the edge case where the key is omitted entirely from the configuration.

mr-canoehead commented 4 years ago

I get an error if I try this change on my system:

File "/opt/netperf/configure_interfaces.py", line 127
    print(interface + " has no configured namespace, defaulting to 'root')
                                                                         ^
SyntaxError: EOL while scanning string literal

If you fix this and commit the change, the pull request should get updated.