Merge ~bladernr/plainbox-provider-checkbox:1711156-cleanup-eth-info-automated into plainbox-provider-checkbox:master

[Jeff Lane ]

Simple mod to the network_info script to clean up the output. Old behaviour caused the last line of one NIC's info to be run right into the first line of the next NIC, with a space improperly between lines in relevant data:

Interface: enP2p233s0f0
Connected: Yes
IPv4: 10.228.68.228
IPv6: 1000
MAC: f4:4c:7f:30:ab:14

Connect Speed: 10000
Interface: enP2p233s0f1
Connected: Yes
IPv4: 10.228.68.223
IPv6: 1000
MAC: f4:4c:7f:30:ab:15

Connect Speed: 10000
Interface: enP12p33s0f0
Connected: Yes
IPv4: 10.228.68.219
IPv6: 1000
MAC: ec:38:8f:6a:71:bc

Connect Speed: 1000

Note it runs the Connect Speed from the first nic into the information of hte second nic, and so forth, and gets REALLY confusing later. Now, the output is cleaned up so it will look like this:

Interface: enP2p233s0f0
Connected: Yes
IPv4: 10.228.68.228
IPv6: 1000
MAC: f4:4c:7f:30:ab:14
Connect Speed: 10000

Interface: enP2p233s0f1
Connected: Yes
IPv4: 10.228.68.223
IPv6: 1000
MAC: f4:4c:7f:30:ab:15
Connect Speed: 10000

Interface: enP12p33s0f0
Connected: Yes
IPv4: 10.228.68.219
IPv6: 1000
MAC: ec:38:8f:6a:71:bc
Connect Speed: 1000

Keeping all the data from each NIC in a nice, uniform group, with no confusing spacing issues.

[Maciej Kisielewski (kissiel)]

LGTM! +1

review: Approve

diff --git a/bin/network_info b/bin/network_info
index 38d5ada..7dbe609 100755
--- a/bin/network_info
+++ b/bin/network_info
@@ -84,8 +84,8 @@ def main(args):
             print("IPv6: n/a")
         except:
             print("IPv6: n/a")
-        print("MAC: %s\n" % get_mac_address(interface))
-        print("Connect Speed: %s" % get_speed(interface))
+        print("MAC: %s" % get_mac_address(interface))
+        print("Connect Speed: %s\n" % get_speed(interface))
 
     return 0