Merge lp://staging/~awe/lxc-android-config/fix-emulator-networking-xenial into lp://staging/lxc-android-config

Proposed by Tony Espy
Status: Merged
Approved by: Ɓukasz Zemczak
Approved revision: 23
Merged at revision: 23
Proposed branch: lp://staging/~awe/lxc-android-config/fix-emulator-networking-xenial
Merge into: lp://staging/lxc-android-config
Diff against target: 24 lines (+17/-1)
1 file modified
etc/NetworkManager/dispatcher.d/02default_route_workaround (+17/-1)
To merge this branch: bzr merge lp://staging/~awe/lxc-android-config/fix-emulator-networking-xenial
Reviewer Review Type Date Requested Status
Pat McGowan (community) Approve
Review via email: mp+301416@code.staging.launchpad.net

Commit message

Update NM routing dispatcher script to fix emulator networking ( LP: #1587808).

Description of the change

This change updates a network-manager dispatcher script which is used to wipe routes which are sometimes created by non-compliant rild implementations.

The latest version of NM we released (1.2.2) includes a new event which triggers the dispatcher scripts when NM sets the hostname of the device. This was causing the script, which didn't check either parametres ( interface & status ), to wipe out the default route for the ethernet device which is configured in /etc/network/interfaces.

See the following bug for details:

To test, build an emulator with this package, and verify that a default route exists for eth0, and that the browser is able to access the network.

To post a comment you must log in.
Revision history for this message
Pat McGowan (pat-mcgowan) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: