lp:~frankban/juju-quickstart/update-deps
- Get this branch:
- bzr branch lp:~frankban/juju-quickstart/update-deps
Branch merges
- Juju GUI Hackers: Pending requested
-
Diff: 96 lines (+7/-26)6 files modifiedquickstart/__init__.py (+1/-1)
quickstart/juju.py (+0/-20)
quickstart/settings.py (+1/-1)
quickstart/tests/test_juju.py (+1/-0)
requirements.pip (+1/-1)
test-requirements.pip (+3/-3)
Branch information
Recent revisions
- 56. By Francesco Banconi
-
Improve version handling.
Modify the app.ensure_
dependencies function
so that it returns the Juju version tuple.
This way we allow for reusing the version
through the app and we can avoid calling
"juju version" multiple times.Support local environments by installing
the juju-local meta-package in place of
specific packages (e.g. lxc, mongo).
This way we don't have to update quickstart
if core devs introduce new local env
dependencies.Fix the destroy-environment message displayed
at the end of the process. The -e flag is
included when an old Juju is used.Tests: `make check`.
QA: no strictly required.
Use .venv/bin/python juju-quickstart, check everything
works well, check the destroy-environment message
at the end of the process.R=rharding, matthew.scott
CC=
https://codereview. appspot. com/74010044 - 55. By Francesco Banconi
-
SSH code reorganization.
This branch introduces a new ssh module
containing SSH handling related code.The SSH functions previously in app are now
library functions in the ssh module.ensure_ssh_keys is now the only app entry point
for the whole SSH keys management logic.For the reasons above this branch mostly includes
code movements + some docstring/comments changes +
some exception handling changes.R=jeff.pihach, bac
CC=
https://codereview. appspot. com/72520044 - 54. By Francesco Banconi
-
Minor changes in preparation for release.
Bump version up to 1.1.0.
Re-organize the "requires sudo" code.
Restore 100% unit test coverage (compulsive mode on).
Update license headers of modified files.Tests: `make check`
QA: use quickstart as usual, bootstrapping
both ec2 and local environments.R=rharding
CC=
https://codereview. appspot. com/65970043 - 53. By Brad Crittenden
-
[r=makyo] Use existing ssh-agent if possible. Don't start ssh-agent for realz in tests.
- 52. By Jeff Pihach
-
When bootstrapping, if the Juju version is less than 1.17.2
then quickstart will use sudo when deploying a new environment.On versions higher than 1.17.2 sudo is no longer required and
quickstart will no longer use it. - 51. By Francesco Banconi
-
Minor fixes to code comments and documentation
Also fixed a possible encoding problem in error handling.
The get_admin_secret function and its test case are just moved, no
need to re-review.Tests: `make check`.
No QA.R=bac
CC=
https://codereview. appspot. com/58680043 - 50. By Brad Crittenden
-
Get admin_secret from juju-generated jenv file.
As of Juju v 1.17 the admin-secret is stored in a generated file, located in
$JUJU_HOME/environments/ <environment> .jenv. If an admin-secret is provided in environments.yaml, it is used and also
stored in the jenv file. If none is given, then juju fabricates one and
stores it in the jenv file.The admin-secret *field* is marked as optional now. For the case where a
local environment is used, Quickstart previously would generate an
admin-secret. That behavior is maintained by forcing 'admin-secret' to be in
the missing_fields list. This approach ensures backwards compatibility.Note that the Juju-generated admin-secret is not available until after
bootstrapping the environment. Due to this behavior, when using juju v1.16
and not admin-secret is configured in the environments.yaml file, the failure
will not be reported until after bootstrapping. This approach seemed better
than introducing hard version checking.R=frankban
CC=
https://codereview. appspot. com/57900043 - 49. By Francesco Banconi
-
Update for PyPI release.
R=gary.poster
CC=
https://codereview. appspot. com/51790045
Branch metadata
- Branch format:
- Branch format 7
- Repository format:
- Bazaar repository format 2a (needs bzr 1.16 or later)
- Stacked on:
- lp:juju-quickstart