Merge ~alfonsosanchezbeato/snappy-hwe-snaps/+git/wifi-ap:fix-wizard into ~snappy-hwe-team/snappy-hwe-snaps/+git/wifi-ap:master

[Alfonso Sanchez-Beato]

wizard: select properly wifi iface

We were selecting always 'wlan0' as AP interface when there was only one
wifi interface present, so things were not working if the name of that
interface was not wlan0.

wizard: select properly wifi iface

We were selecting always 'wlan0' as AP interface when there was only one
wifi interface present, so things were not working if the name of that
interface was not wlan0.

To test the change, switch the default renderer on a UC18 system:

   snap set network-manager defaultrenderer=true

'networkctl' should show now that all devices are in "unmanaged" state. This will not work on UC16 systems due to a bug in netplan. See https://bugs.launchpad.net/ubuntu/+source/netplan.io/+bug/1855876.

[System Enablement Bot (system-enablement-ci-bot)]

PASSED: Successfully build documentation, rev: fdb4fc4fef23bb5c4a908d7d4899306ee258bb2d

Generated documentation is available at https://jenkins.canonical.com/system-enablement/job/snappy-hwe-snaps-snap-docs/1269/

[System Enablement Bot (system-enablement-ci-bot)]

PASSED: Continuous integration, rev:fdb4fc4fef23bb5c4a908d7d4899306ee258bb2d
https://jenkins.canonical.com/system-enablement/job/snappy-hwe-snaps-snap-build-prepare/445/
Executed test runs:
    SUCCESS: https://jenkins.canonical.com/system-enablement/job/snappy-hwe-snaps-snap-build/ARCHITECTURE=amd64/477
    SUCCESS: https://jenkins.canonical.com/system-enablement/job/snappy-hwe-snaps-snap-build/ARCHITECTURE=arm64/477
    SUCCESS: https://jenkins.canonical.com/system-enablement/job/snappy-hwe-snaps-snap-build/ARCHITECTURE=armhf/477
    SUCCESS: https://jenkins.canonical.com/system-enablement/job/snappy-hwe-snaps-snap-build/ARCHITECTURE=i386/477
    SUCCESS: https://jenkins.canonical.com/system-enablement/job/snappy-hwe-snaps-snap-docs/1269
    SUCCESS: https://jenkins.canonical.com/system-enablement/job/snappy-hwe-snaps-snap-cleanup/1011
    None: https://jenkins.canonical.com/system-enablement/job/snappy-hwe-snaps-snap-update-mp/1058/console

Click here to trigger a rebuild:
https://jenkins.canonical.com/system-enablement/job/snappy-hwe-snaps-snap-build-prepare/445/rebuild

review: Approve (continuous-integration)

diff --git a/cmd/client/cmd_wizard.go b/cmd/client/cmd_wizard.go
index b2080bc..2390bd4 100644
--- a/cmd/client/cmd_wizard.go
+++ b/cmd/client/cmd_wizard.go
@@ -45,7 +45,7 @@ const ipv4Offset = net.IPv6len - net.IPv4len
 var defaultIp = net.IPv4(10, 0, 60, 1)
 
 // Utility function to read input and strip the trailing \n
-var readUserInput = func (reader *bufio.Reader) string {
+var readUserInput = func(reader *bufio.Reader) string {
 	ret, err := reader.ReadString('\n')
 	if err != nil {
 		panic(err)
@@ -105,7 +105,7 @@ func findFreeSubnet(startIp net.IP) (net.IP, error) {
 
 func generatePassword(length int) string {
 	// base64 produces 4 byte for every 3 byte of input, rounded up
-	password := make([]byte, length / 4 * 3 + 3)
+	password := make([]byte, length/4*3+3)
 	rand.Read(password)
 	return base64.StdEncoding.EncodeToString(password)[:length]
 }
@@ -127,6 +127,7 @@ var allSteps = [...]wizardStep{
 			return fmt.Errorf("There are no valid wireless network interfaces available")
 		} else if len(ifaces) == 1 {
 			fmt.Println("Automatically selected only available wireless network interface " + ifaces[0])
+			configuration["wifi.interface"] = ifaces[0]
 			return nil
 		}