Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix bug 1721604 #8077

Merged
merged 1 commit into from Nov 15, 2017
Merged

fix bug 1721604 #8077

merged 1 commit into from Nov 15, 2017

Conversation

hmlanigan
Copy link
Member

Description of change

Juju bootstrap cli results should be the same as running bootstrap interactive. If bootstrap, cli only, doesn't specify a region, but finds one to use, use it for config too.

QA steps

One:

  1. set region config for an openstack cloud, suggest network since you can't bootstrap without it most of the time.
  2. juju bootstrap

Two:
bootstrap a cloud without regions, such as localhost

Documentation changes

n/a

Bug reference

https://bugs.launchpad.net/juju/+bug/1721604

@jujubot
Copy link
Collaborator

jujubot commented Nov 15, 2017

Can one of the admins verify this patch?

@hmlanigan
Copy link
Member Author

@hmlanigan
Copy link
Member Author

$$merge$$

@jujubot
Copy link
Collaborator

jujubot commented Nov 15, 2017

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

@jujubot jujubot merged commit 469e113 into juju:develop Nov 15, 2017
@hmlanigan hmlanigan deleted the bug1721604 branch November 15, 2017 18:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants