/juju

Port of https://github.com/juju/juju/pull/5263 #5281

Closed
wants to merge 1 commit into
from

Conversation

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

dooferlad commented Apr 26, 2016 •
Edited 1 time

Fixes https://launchpad.net/bugs/1565644

IP commands are used to immediately reconfigure networking instead of relying on /etc/network/interfaces and ifup/ifdown since they can cause coudinit to hang.
Bridge script is stored in /var/tmp until it is run. Precise seemed to reboot after writing the script to /tmp and before it was run, so it wasn't there when needed.

(Original review: http://reviews.vapour.ws/r/4688/)

In addition, remove the 'quick hack' to speed up multiple bridges.

(Review request: http://reviews.vapour.ws/r/4708/)
@dooferlad
Port of #5263 
Fixes https://launchpad.net/bugs/1565644

IP commands are used to immediately reconfigure networking instead of relying on /etc/network/interfaces and ifup/ifdown since they can cause coudinit to hang.
Bridge script is stored in /var/tmp until it is run. Precise seemed to reboot after writing the script to /tmp and before it was run, so it wasn't there when needed.

(Original review: http://reviews.vapour.ws/r/4688/)

In addition, remove the 'quick hack' to speed up multiple bridges.
3c7cc1b
@dooferlad
Contributor

dooferlad commented Apr 27, 2016

$$merge$$
@jujubot
Contributor

jujubot commented Apr 27, 2016

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

jujubot commented Apr 27, 2016

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

@dooferlad dooferlad closed this Apr 29, 2016

@dooferlad dooferlad deleted the dooferlad:master-bridge branch Apr 29, 2016

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