provider/maas: remove MTU option from the bridge (backport) #6013

Merged
merged 1 commit into from Aug 19, 2016

Conversation

Projects
None yet
2 participants

voidspace commented Aug 17, 2016

Backport of PR #5791.

The MTU statement must be on the interface to work and not on the bridge
itself; bridges inherit the lowest MTU of all sub-interfaces. Further to
this, ifupdown seems to bug out and always fails to add the default
route when an MTU is set on the bridge. Leaving the MTU option on the
bridge completely breaks the network on reboot.

This commit also removes:

address
gateway
netmask
dns-nameservers
dns-search
dns-sortlist

from the sub-interfaces, leaving them on the bridge interface only.

(Review request: http://reviews.vapour.ws/r/5460/)

$$merge$$

Contributor

jujubot commented Aug 17, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

Contributor

jujubot commented Aug 17, 2016

Build failed: Does not match ['fixes-1474607', 'fixes-1571457']
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/8819

$$merge$$

Contributor

jujubot commented Aug 18, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

Contributor

jujubot commented Aug 18, 2016

Build failed: Does not match ['fixes-1474607', 'fixes-1571457']
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/8833

$$JFDI$$

Contributor

jujubot commented Aug 19, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

@jujubot jujubot merged commit bbda325 into juju:1.25 Aug 19, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment