/juju

provider/maas: add missing auto stanza #5792

Merged
merged 1 commit into from Jul 14, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@frobware @jujubot
@frobware
Contributor

frobware commented Jul 13, 2016 •
Edited 1 time

Ensure that the sub-interface has an 'auto' stanza if the original
declaration had an 'auto' stanza.

This is particularly important if the sub-interface has configuration
options (e.g., 'mtu') because our subsequent invocation of 'ifup -a'
only applies to those interfaces that are marked 'auto'. If 'auto' is
missing, then any MTU value would not be applied.

Fixes LP:#1596597

(Review request: http://reviews.vapour.ws/r/5235/)
provider/maas: add missing auto stanza 
Ensure that the sub-interface has an 'auto' stanza if the original
declaration had an 'auto' stanza.

This is particularly important if the sub-interface has configuration
options (e.g., 'mtu') because our subsequent invocation of 'ifup -a'
only applies to those interfaces that are marked 'auto'. If 'auto' is
missing, then any MTU value would not be applied.

Fixes [LP:#1596597](https://bugs.launchpad.net/juju-core/+bug/1596597)
395cd8d
@frobware
Contributor

frobware commented Jul 14, 2016

$$merge$$ $$fixes-1596597$$
@jujubot
Contributor

jujubot commented Jul 14, 2016

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

@jujubot jujubot merged commit c7fb5bf into juju:master Jul 14, 2016

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