Merge lp://qastaging/~frankban/juju-quickstart/apt-update-fixes into lp://qastaging/juju-quickstart
Status: | Merged |
---|---|
Merged at revision: | 124 |
Proposed branch: | lp://qastaging/~frankban/juju-quickstart/apt-update-fixes |
Merge into: | lp://qastaging/juju-quickstart |
Diff against target: |
363 lines (+102/-23) 6 files modified
quickstart/models/bundles.py (+9/-6) quickstart/platform_support.py (+6/-2) quickstart/tests/models/test_bundles.py (+48/-6) quickstart/tests/test_app.py (+26/-4) quickstart/tests/test_utils.py (+12/-5) quickstart/utils.py (+1/-0) |
To merge this branch: | bzr merge lp://qastaging/~frankban/juju-quickstart/apt-update-fixes |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Juju GUI Hackers | Pending | ||
Review via email: mp+252430@code.qastaging.launchpad.net |
Description of the change
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/
Try the following bundles:
- devenv/
- devenv/
- devenv/
- devenv/
- devenv/
Those instead should return errors:
- devenv/
- devenv/
- devenv/
- devenv/
- devenv/
- devenv/
Thank you!
Reviewers: mp+252430_ code.launchpad. net,
Message:
Please take a look.
Description:
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: bin/juju- quickstart {bundle}` bin/juju- quickstart mediawiki-single bin/juju- quickstart u/bigdata- dev/apache- analytics- sql bin/juju- quickstart bundle: mediawiki/ scalable bin/juju- quickstart ~landscape/ landscape- dense-maas/ landscape- dense-maas bin/juju- quickstart bundle: django/ example- single
install bundles with quickstart:
`devenv/
Try the following bundles:
- devenv/
- devenv/
- devenv/
- devenv/
bundle:
- devenv/
Those instead should return errors: bin/juju- quickstart mediawiki/trusty bin/juju- quickstart mediawiki-nosuch bin/juju- quickstart no-such bin/juju- quickstart bundle:no/such bin/juju- quickstart bundle:invalid bin/juju- quickstart ~landscape/ landscape- dense-maas/ landscape
- devenv/
- devenv/
- devenv/
- devenv/
- devenv/
- devenv/
bundle:
Thank you!
https:/ /code.launchpad .net/~frankban/ juju-quickstart /apt-update- fixes/+ merge/252430
(do not edit description out of merge proposal)
Please review this at https:/ /codereview. appspot. com/215750043/
Affected files (+104, -23 lines): models/ bundles. py platform_ support. py tests/models/ test_bundles. py tests/test_ app.py tests/test_ utils.py
A [revision details]
M quickstart/
M quickstart/
M quickstart/
M quickstart/
M quickstart/
M quickstart/utils.py