Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
Swap bootstrap controller and cloud args #6444
Conversation
anastasiamac
approved these changes
Oct 13, 2016
The code change in itself looks ok. Thank you for doing...
I have reservations doing this work hours away from release but this is nor here nor there :D
|
$$merge$$ |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
jujubot
merged commit a2499fe
into
juju:master
Oct 13, 2016
frankban
referenced this pull request
Oct 13, 2016
Merged
bootstrap command: update usage message. #6445
added a commit
that referenced
this pull request
Oct 13, 2016
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
wallyworld commentedOct 13, 2016
•
Edited 1 time
-
wallyworld
Oct 13, 2016
Fixes: https://bugs.launchpad.net/juju/+bug/1632919
The arg order for cloud and controller for bootstrap are swapped.
$ juju bootstrap aws myctrlname
Controller name is optional (controller name will be cloud-region):
$ juju bootstrap aws
Another change as per the bug is that if the cloud has no regions, the default controller name will just be the cloud and, and not user-cloud.
QA
$ juju bootstrap aws
Creating Juju controller "aws-us-east-1" on aws/us-east-1
...
$ juju bootstrap aws/us-west-1
Creating Juju controller "aws-us-west-1" on aws/us-east-1
...
$ juju bootstrap aws ian
Creating Juju controller "ian" on aws/us-east-1
...