/juju

Back port of PR7905 #7940

Merged
merged 1 commit into from Oct 19, 2017

Conversation

Reviewers

@axw axw

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
4 participants
@hmlanigan @jujubot @nskaggs @axw
@hmlanigan
Member

hmlanigan commented Oct 17, 2017

If no external-network specified, pick one from same AZ as the
instance network. If external-network specified, use that.

#7905
@hmlanigan
Fix for BUG 1689683 
If no external-network specified, pick one from same AZ as the
instance network.  If external-network specified, use that.
aa73c55
@jujubot
Contributor

jujubot commented Oct 17, 2017

Can one of the admins verify this patch?
@axw

axw approved these changes Oct 17, 2017
View changes

@hmlanigan
Member

hmlanigan commented Oct 17, 2017

$$merge$$
@jujubot
Contributor

jujubot commented Oct 17, 2017

Status: merge request accepted. Url: http://ci.jujucharms.com/job/github-merge-juju
@jujubot
Contributor

jujubot commented Oct 17, 2017

Build failed: Tests failed
build url: http://ci.jujucharms.com/job/github-merge-juju/356
@jujubot
Contributor

jujubot commented Oct 17, 2017

Build failed: Tests failed
build url: http://ci.jujucharms.com/job/github-merge-juju/357
@hmlanigan
Member

hmlanigan commented Oct 17, 2017

$$merge$$
@jujubot
Contributor

jujubot commented Oct 17, 2017

Status: merge request accepted. Url: http://ci.jujucharms.com/job/github-merge-juju
@jujubot
Contributor

jujubot commented Oct 17, 2017

Build failed: Tests failed
build url: http://ci.jujucharms.com/job/github-merge-juju/361
@nskaggs
Owner

nskaggs commented Oct 19, 2017

$$merge$$
@jujubot
Contributor

jujubot commented Oct 19, 2017

Status: merge request accepted. Url: http://ci.jujucharms.com/job/github-merge-juju

@jujubot jujubot merged commit c20d745 into juju:2.2 Oct 19, 2017
1 check passed

1 check passed

continuous-integration/jenkins/pr-merge This commit looks good
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment