Ensure VLANs get bridged #4148

Merged
merged 5 commits into from Jan 21, 2016

Conversation

Projects
None yet
2 participants
Contributor

frobware commented Jan 19, 2016

We were previously only creating a bridge for active interfaces where
'active' == 'dhcp' || 'static' but that approach omits VLANs where the
underlying raw device is not 'active'.

VLANs are now bridged irrespective of the 'active' state of the
underlying raw device.

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

Andrew McDermott added some commits Jan 14, 2016

Ensure VLANs get bridged
We were previously only creating a bridge for active interfaces where
'active' == 'dhcp' || 'static' but that approach omits VLANs where the
underlying raw device is not 'active'.

VLANs are now bridged irrespective of the 'active' state of the
underlying raw device.
Contributor

frobware commented Jan 21, 2016

$$merge$$

Contributor

jujubot commented Jan 21, 2016

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

jujubot added a commit that referenced this pull request Jan 21, 2016

Merge pull request #4148 from frobware/maas-space-bridge-party
Ensure VLANs get bridged

We were previously only creating a bridge for active interfaces where
'active' == 'dhcp' || 'static' but that approach omits VLANs where the
underlying raw device is not 'active'.

VLANs are now bridged irrespective of the 'active' state of the
underlying raw device.

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

@jujubot jujubot merged commit 607a243 into juju:maas-spaces Jan 21, 2016

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