Merge lp://staging/~penguin359/network-manager/ubuntu-fix-for-1005091 into lp://staging/~network-manager/network-manager/ubuntu
Status: | Merged |
---|---|
Approved by: | Mathieu Trudel-Lapierre |
Approved revision: | 519 |
Merged at revision: | 522 |
Proposed branch: | lp://staging/~penguin359/network-manager/ubuntu-fix-for-1005091 |
Merge into: | lp://staging/~network-manager/network-manager/ubuntu |
Diff against target: |
11 lines (+1/-0) 1 file modified
debian/ifblacklist_migrate.sh (+1/-0) |
To merge this branch: | bzr merge lp://staging/~penguin359/network-manager/ubuntu-fix-for-1005091 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Mathieu Trudel-Lapierre | Approve | ||
Review via email:
|
Description of the change
Fixed issue with NetworkManager failing to disable all lines related to network interface when taking over management of the device. Prior to this fix, if IPv6 was employed on the network, it would not disable the inet6 lines for the interface thereby preventing NetworkManager from taking over control of the interface, but still disabling dhcp in the /etc/network/
This is a slightly different fix than the previous pull request due to a possible failure when a more complicated interfaces file is used. Here is my test case, only eth0 and all of eth0 should be commented out:
# The loopback network interface
auto lo
iface lo inet loopback
# The primary network interface
auto eth0
iface eth0 inet dhcp
hostname unknown
# Enable IPv6 Autoconfiguration
iface eth0 inet6 auto
dhcp 1
auto eth1
iface eth1 inet static
address 192.168.1.1
netmask 255.255.255.0
gateway 192.168.1.1
Hmm, it looks like I've just repeated the exact same procedure. I thought I had the right branch, but it appears to be identical to my previous request. I'll have to do a little more reading looks like.