Code review comment for lp:~brendan-donegan/checkbox/bug861502_reconnect_wireless

Revision history for this message
Ara Pulido (ara) wrote :

Instead of saving just the SSID name, nmcli saves "Auto <SSID>", so this fix fails for me:

ara@sushirider:~/curro/checkbox/checkbox/trunk/scripts$ nmcli -t -f NAME con status > $CHECKBOX_DATA/connections && ./connect_wireless && ./internet_test --interface=`nmcli dev list | grep -B 1 wireless | grep GENERAL.DEVICE | awk '{print $2}'` && while read CON; do nmcli con up "$CON"; done < $CHECKBOX_DATA/connections
Error: Device 'eth0' (/org/freedesktop/NetworkManager/Devices/0) disconnecting failed: This device is not active
Device state: 30 (disconnected)
Active connection state: activating
Active connection path: /org/freedesktop/NetworkManager/ActiveConnection/6
state: activated
Connection activated
Internet connection fully established
Unknown parameter: Auto Musireta
Error: id or uuid has to be specified.

review: Needs Fixing

« Back to merge proposal