in provision.sh check for existence of network file before starting the provisioning

Bug #1276216 reported by Jean-Baptiste Lallement
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu Test Cases
Fix Released
Wishlist
Andy Doan

Bug Description

Running provision.sh on a local machine to reproduce the CI environment it fails with:

+ log 'SETTING UP WIFI'
++ date
+ echo = mardi 4 $'f\303\251vrier' 2014, 10:49:23 '(UTC-0500):' SETTING UP WIFI
= mardi 4 février 2014, 10:49:23 (UTC-0500): SETTING UP WIFI
+ phablet-network -n /home/ubuntu/magners-wifi
Network connection file "/home/ubuntu/magners-wifi" cannot be read

Expected behavior:
THe script should check for the existence of a network file before starting the provisioning and not crashing after the device has been flashed.

Revision history for this message
Paul Larson (pwlars) wrote :

That's just a default location for historical reasons. If you'd like to put one in that path it will pick it up, but in general, you need to use the -n option to specify your network profile (from NetworkManager). This is just passed off to phablet-network for you.

Changed in ubuntu-test-cases:
status: New → Invalid
Revision history for this message
Jean-Baptiste Lallement (jibel) wrote :

Could you at least update the doc to mention this option and a check for existence of the network file in the script so it doesn't fail after the device has been reflashed but print a message before starting the provisioning ?

summary: - provision.sh fails with 'Network connection file "/home/ubuntu/magners-
- wifi" cannot be read'
+ in provision.sh check for existence of network file before starting the
+ provisioning
description: updated
Changed in ubuntu-test-cases:
status: Invalid → Confirmed
importance: Undecided → Wishlist
Revision history for this message
Andy Doan (doanac) wrote :

Now that we have mega-stuff merged in production, I should be able to add a small patch with jibel's suggestions.

Changed in ubuntu-test-cases:
assignee: nobody → Andy Doan (doanac)
Andy Doan (doanac)
Changed in ubuntu-test-cases:
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.