Merge lp://staging/~1chb1n/openstack-charm-testing/use-cs into lp://staging/openstack-charm-testing

Proposed by Ryan Beisner
Status: Rejected
Rejected by: Ryan Beisner
Proposed branch: lp://staging/~1chb1n/openstack-charm-testing/use-cs
Merge into: lp://staging/openstack-charm-testing
Diff against target: 261 lines (+40/-40)
2 files modified
bundles/sparse/default.yaml (+20/-20)
bundles/sparse/next.yaml (+20/-20)
To merge this branch: bzr merge lp://staging/~1chb1n/openstack-charm-testing/use-cs
Reviewer Review Type Date Requested Status
Ryan Beisner Needs Resubmitting
Corey Bryant Approve
Review via email: mp+316177@code.staging.launchpad.net

Description of the change

Use charmstore urls for next and default sparse bundles.

This breaks [Juju1 + juju-deployer] usage due to bug 1643027.

However, cs:foo is necessary:
 - To render Juju2 native deployer bundles out of the reference next/default bundles.
 - To deploy layered (built) charms.

https://bugs.launchpad.net/juju-deployer/+bug/1643027

To post a comment you must log in.
Revision history for this message
Corey Bryant (corey.bryant) wrote :

LGTM

review: Approve
Revision history for this message
Ryan Beisner (1chb1n) wrote :

Let's not land this, pending further discussion around tooling support for cs:. Juju-deployer is known to deploy the wrong series from cs: with juju < 2.0.

Revision history for this message
Ryan Beisner (1chb1n) wrote :

The code repo for this project is now at the following location:
https://github.com/openstack-charmers/openstack-charm-testing

review: Needs Resubmitting

Unmerged revisions

266. By Ryan Beisner

Use charmstore url for next and default sparse bundles

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to status/vote changes: