Merge lp://qastaging/~frankban/juju-quickstart/jujucharms-bundles into lp://qastaging/juju-quickstart
Status: | Merged |
---|---|
Merged at revision: | 122 |
Proposed branch: | lp://qastaging/~frankban/juju-quickstart/jujucharms-bundles |
Merge into: | lp://qastaging/juju-quickstart |
Diff against target: |
3303 lines (+1519/-940) 19 files modified
HACKING.rst (+2/-2) quickstart/__init__.py (+1/-1) quickstart/app.py (+19/-10) quickstart/juju.py (+3/-3) quickstart/jujutools.py (+22/-21) quickstart/manage.py (+63/-90) quickstart/models/bundles.py (+300/-88) quickstart/models/references.py (+191/-70) quickstart/netutils.py (+2/-2) quickstart/settings.py (+4/-7) quickstart/tests/functional/test_functional.py (+1/-1) quickstart/tests/helpers.py (+8/-21) quickstart/tests/models/test_bundles.py (+384/-196) quickstart/tests/models/test_references.py (+386/-184) quickstart/tests/test_app.py (+50/-32) quickstart/tests/test_juju.py (+14/-11) quickstart/tests/test_jujutools.py (+38/-28) quickstart/tests/test_manage.py (+30/-172) tox.ini (+1/-1) |
To merge this branch: | bzr merge lp://qastaging/~frankban/juju-quickstart/jujucharms-bundles |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Juju GUI Hackers | Pending | ||
Review via email: mp+251156@code.qastaging.launchpad.net |
Description of the change
Support retrieving bundles from charm store v4.
This branch implements the ability to deploy
bundles from the new charm store, retrieving
them with the v4 API.
Also introduce the new preferred bundle id
spelling, i.e. reflecting jujucharms.com paths,
like "mediawiki-single" or "u/who/
The old "bundle:
still supported but deprecated.
Deploying a bundle by specifying a directory
containing the YAML file is instead not
supported anymore.
Ok, after this brief summary let me take two
lines to really apologize for the huge diff.
While I was there, I refactored some historical
inconsistencies (e.g. models.Charm really being
just a charm or bundle reference), and I also
improved the bundle model API so that the work
is done in the model and not in manage as before.
There are a lot of tests too, and some documentation.
Nonetheless, let me say sorry again, this is
really too much stuff.
With this branch Juju Quickstart is quite ready for
the v4 world. The "deploy bundle" API call to the GUI
server still uses the legacy format, but the ugliness
of being backward compatible with namespaced bundles
is very restrained and implemented in private logic
in the bundles model module.
Tests: `make check`.
QA: run `devenv/
new style and old style bundles, with both version
3 and 4 formats. Note that version 3 can only be
provided with arbitrary URLs or local files.
Thanks a lot!
Reviewers: mp+251156_ code.launchpad. net,
Message:
Please take a look.
Description:
Support retrieving bundles from charm store v4.
This branch implements the ability to deploy
bundles from the new charm store, retrieving
them with the v4 API.
Also introduce the new preferred bundle id bundle- name".
spelling, i.e. reflecting jujucharms.com paths,
like "mediawiki-single" or "u/who/
The old "bundle: basket/ name" identifiers are
still supported but deprecated.
Deploying a bundle by specifying a directory
containing the YAML file is instead not
supported anymore.
Ok, after this brief summary let me take two
lines to really apologize for the huge diff.
While I was there, I refactored some historical
inconsistencies (e.g. models.Charm really being
just a charm or bundle reference), and I also
improved the bundle model API so that the work
is done in the model and not in manage as before.
There are a lot of tests too, and some documentation.
Nonetheless, let me say sorry again, this is
really too much stuff.
With this branch Juju Quickstart is quite ready for
the v4 world. The "deploy bundle" API call to the GUI
server still uses the legacy format, but the ugliness
of being backward compatible with namespaced bundles
is very restrained and implemented in private logic
in the bundles model module.
Tests: `make check`.
QA: run `devenv/ bin/juju- quickstart` to deploy
new style and old style bundles, with both version
3 and 4 formats. Note that version 3 can only be
provided with arbitrary URLs or local files.
Thanks a lot!
https:/ /code.launchpad .net/~frankban/ juju-quickstart /jujucharms- bundles/ +merge/ 251156
(do not edit description out of merge proposal)
Please review this at https:/ /codereview. appspot. com/207040043/
Affected files (+1520, -939 lines): __init_ _.py jujutools. py manage. py models/ bundles. py models/ references. py netutils. py settings. py tests/functiona l/test_ functional. py tests/helpers. py tests/models/ test_bundles. py tests/models/ test_references .py tests/test_ app.py tests/test_ juju.py tests/test_ jujutools. py tests/test_ manage. py
M HACKING.rst
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 quickstart/
M quickstart/
M quickstart/
M quickstart/
M quickstart/