lp:~frankban/juju-quickstart/changeset-from-jujubundlelib
- Get this branch:
- bzr branch lp:~frankban/juju-quickstart/changeset-from-jujubundlelib
Branch merges
- Juju GUI Hackers: Pending requested
-
Diff: 453 lines (+186/-61)4 files modifiedquickstart/models/bundles.py (+17/-25)
quickstart/tests/helpers.py (+42/-3)
quickstart/tests/models/test_bundles.py (+123/-29)
tox.ini (+4/-4)
Branch information
Recent revisions
- 124. By Francesco Banconi
-
Always retrieve bundle from new cs endpoint.
This is a follow up from previous branch,
and also includes the remaining bits before
2.0.1 release.Even if a legacy bundle is specified, and even
if the legacy bundle data is retrieved for name
validation, always use the new bundle.yaml
charm store endpoint when returning the bundle
contents.This branch also include a drive by fix to always
run `apt-get update` before installing packages,
even in the case --distro-only is enabled.Tests: `make check`.
QA:
install bundles with quickstart:
`devenv/bin/juju- quickstart {bundle}`
Try the following bundles:
- devenv/bin/juju- quickstart mediawiki-single
- devenv/bin/juju- quickstart u/bigdata- dev/apache- analytics- sql
- devenv/bin/juju- quickstart bundle: mediawiki/ scalable
- devenv/bin/juju- quickstart bundle: ~landscape/ landscape- dense-maas/ landscape- dense-maas
- devenv/bin/juju- quickstart bundle: django/ example- single Those instead should return errors:
- devenv/bin/juju- quickstart mediawiki/trusty
- devenv/bin/juju- quickstart mediawiki-nosuch
- devenv/bin/juju- quickstart no-such
- devenv/bin/juju- quickstart bundle:no/such
- devenv/bin/juju- quickstart bundle:invalid
- devenv/bin/juju- quickstart bundle: ~landscape/ landscape- dense-maas/ landscape Thank you!
R=bac, rharding
CC=
https://codereview. appspot. com/215750043 - 123. By Francesco Banconi
-
Fix the old-style bundle regression.
This branch fixes
https://bugs.launchpad .net/juju- quickstart/ +bug/1429129
In essence, legacy bundles are converted by the
ingestion process like the following:
- if a basked includes multiple bundles, the resulting
v4 bundle name is {basket-name}-{ bundle- name} for
each {bundle-name};
- if a basket only includes one bundle, the v4 bundle
is just {basket-name}.
Previously quickstart always assumed the former: this
branch adds a check for the latter before exiting with
an error.This branch also introduces a charmstore module in
quickstart. All the interactions between quickstart
and the charm store are now collected in this module.As part of this refactoring, quickstart is now able
to distinguish HTTP 404 errors and all the other
generic IOErrors that can be raised when connecting
to the network.Also simplified the logging format and bootstrap logging
earlier in the application execution.Tests: `make check`.
QA:
install bundles with quickstart:
`devenv/bin/juju- quickstart {bundle}`
Try the following bundles:
- devenv/bin/juju- quickstart mediawiki-single
- devenv/bin/juju- quickstart u/bigdata- dev/apache- analytics- sql
- devenv/bin/juju- quickstart bundle: mediawiki/ scalable
- devenv/bin/juju- quickstart bundle: ~landscape/ landscape- dense-maas/ landscape- dense-maas
- devenv/bin/juju- quickstart bundle: django/ example- single Those instead should return errors:
- devenv/bin/juju- quickstart mediawiki/trusty
- devenv/bin/juju- quickstart mediawiki-nosuch
- devenv/bin/juju- quickstart no-such
- devenv/bin/juju- quickstart bundle:no/such
- devenv/bin/juju- quickstart bundle:invalid
- devenv/bin/juju- quickstart bundle: ~landscape/ landscape- dense-maas/ landscape R=jeff.pihach, rharding, bac
CC=
https://codereview. appspot. com/215070043
Branch metadata
- Branch format:
- Branch format 7
- Repository format:
- Bazaar repository format 2a (needs bzr 1.16 or later)
- Stacked on:
- lp:juju-quickstart