a value is required for the control bucket field
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
juju-quickstart |
Fix Released
|
High
|
Brad Crittenden | ||
juju-quickstart (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
Trusty |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
[Impact]
If the control-bucket field is not present in the environments.yaml file juju will generate one. This patch makes control-bucket an optional field for juju-quickstart.
[Test Case]
* Create an ec2 environment in $JUJU_HOME/
* Bootstrap that environment using juju, not juju-quickstart.
* Ensure a control-bucket is in the generated environments/
* Run juju-quickstart using the existing environment and see that no error is raised.
[Regression Potential]
The overly strict error checking has been relaxed to stay in sync with juju-core. Potential for regression is minimal.
[Original Bug Report]
Having bootstraped an amazon environment with juju, I run juju-quickstart and get the following error
$ juju-quickstart
usage: juju-quickstart [-h] [-e ENV_NAME] [-n BUNDLE_NAME] [-i]
juju-quickstart: error: cannot use the amazon environment:
a value is required for the control bucket field
I checked that I did have a value for `control-bucket` in ~/.juju/
$ juju-quickstart -i
There I saw that some properties were copied from environments.yaml (it apparently did not use the .jenv file) and some not and that it needed the `control-bucket` value (which is not defined in environments.yaml).
So, properties like `access-key`, `secret-key`, `type` were extracted successfully from environments.yam but not the `region`.
And it ignored the bucket field in environments/
ENV info:
$ juju version
1.19.0-trusty-amd64
$ juju-quickstart --version
juju-quickstart 1.3.1
Related branches
- Juju GUI Hackers: Pending requested
-
Diff: 197 lines (+33/-30)6 files modifiedquickstart/app.py (+10/-8)
quickstart/manage.py (+2/-2)
quickstart/models/envs.py (+2/-2)
quickstart/tests/models/test_envs.py (+2/-3)
quickstart/tests/test_app.py (+8/-6)
quickstart/tests/test_manage.py (+9/-9)
Changed in juju-quickstart: | |
importance: | Undecided → High |
status: | New → Triaged |
Changed in juju-quickstart: | |
assignee: | nobody → Brad Crittenden (bac) |
status: | Triaged → In Progress |
Changed in juju-quickstart: | |
status: | In Progress → Fix Released |
description: | updated |
Concerning the `region` attribute not being retrieved, it is probably due to the fact that I entered an invalid value:
`eu-west-1a` instead of `eu-west-1`