Merge lp://qastaging/~frankban/juju-quickstart/uncommitted-bundles into lp://qastaging/juju-quickstart
Proposed by
Francesco Banconi
Status: | Merged |
---|---|
Merged at revision: | 135 |
Proposed branch: | lp://qastaging/~frankban/juju-quickstart/uncommitted-bundles |
Merge into: | lp://qastaging/juju-quickstart |
Diff against target: |
665 lines (+332/-57) 13 files modified
quickstart/__init__.py (+1/-1) quickstart/app.py (+62/-6) quickstart/juju.py (+9/-0) quickstart/jujugui.py (+28/-0) quickstart/jujutools.py (+2/-5) quickstart/manage.py (+31/-21) quickstart/settings.py (+7/-2) quickstart/tests/functional/test_functional.py (+8/-0) quickstart/tests/test_app.py (+99/-16) quickstart/tests/test_juju.py (+12/-0) quickstart/tests/test_jujugui.py (+46/-0) quickstart/tests/test_manage.py (+23/-2) tox.ini (+4/-4) |
To merge this branch: | bzr merge lp://qastaging/~frankban/juju-quickstart/uncommitted-bundles |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Brad Crittenden (community) | code + qa | Approve | |
Review via email: mp+261200@code.qastaging.launchpad.net |
Commit message
Add support for uncommitted bundles.
Description of the change
Add support for uncommitted bundles.
Introduce the -u/--uncommitted flag, which enables
uncommitted bundle support.
Improve output messages and tokens handling.
Also update jujubundlelib dep to latest version.
TESTS:
`make fcheck` and wait a while for the functional tests
to complete.
QA:
- deploy bundles as usual:
`devenv/
- deploy uncommitted bundles:
`devenv/
To post a comment you must log in.
Reviewers: mp+261200_ code.launchpad. net,
Message:
Please take a look.
Description:
Add support for uncommitted bundles.
Introduce the -u/--uncommitted flag, which enables
uncommitted bundle support.
Improve output messages and tokens handling.
Also update jujubundlelib dep to latest version.
TESTS:
`make fcheck` and wait a while for the functional tests
to complete.
QA: bin/juju- quickstart mediawiki-single`; bin/juju- quickstart -u u/openstack- charmers/ openstack` ;
- deploy bundles as usual:
`devenv/
- deploy uncommitted bundles:
`devenv/
https:/ /code.launchpad .net/~frankban/ juju-quickstart /uncommitted- bundles/ +merge/ 261200
(do not edit description out of merge proposal)
Please review this at https:/ /codereview. appspot. com/247800043/
Affected files (+329, -57 lines): __init_ _.py jujugui. py jujutools. py manage. py settings. py tests/functiona l/test_ functional. py tests/test_ app.py tests/test_ juju.py tests/test_ jujugui. py tests/test_ manage. py
A [revision details]
M quickstart/
M quickstart/app.py
M quickstart/juju.py
M quickstart/
M quickstart/
M quickstart/
M quickstart/
M quickstart/
M quickstart/
M quickstart/
M quickstart/
M quickstart/
M tox.ini