Merge lp://qastaging/~bjornt/charms/xenial/glance-simplestreams-sync/no-python2 into lp://qastaging/~landscape/charms/trusty/glance-simplestreams-sync/landscape
Proposed by
Björn Tillenius
Status: | Merged |
---|---|
Merged at revision: | 69 |
Proposed branch: | lp://qastaging/~bjornt/charms/xenial/glance-simplestreams-sync/no-python2 |
Merge into: | lp://qastaging/~landscape/charms/trusty/glance-simplestreams-sync/landscape |
Diff against target: |
39 lines (+21/-1) 2 files modified
hooks/hooks.py (+1/-1) hooks/install (+20/-0) |
To merge this branch: | bzr merge lp://qastaging/~bjornt/charms/xenial/glance-simplestreams-sync/no-python2 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Alberto Donato (community) | Approve | ||
Adam Collard (community) | Approve | ||
Review via email:
|
Description of the change
Allow the charm to work on xenial.
Xenial doesn't have python2 installed by default, so it needs to be installed before the real install hook gets executed.
The pattern used is the same as for other openstack charms. The install script was copied from the glance charm.
To post a comment you must log in.
Why not just add python into the DEPS array, as it stands right now we'll attempt installation of python for every dep (wasteful)
That said, +1