Juju Charms Collection
glance-simplestreams-sync package

Merge lp://qastaging/~freyes/charms/trusty/glance-simplestreams-sync/lp1522929 into lp://qastaging/charms/trusty/glance-simplestreams-sync

Trusty Tahr (14.04)
lp1522929
Merge into trunk

Proposed by Felipe Reyes on 2015-12-04

Status:

Merged

Merged at revision:

Proposed branch:

lp://qastaging/~freyes/charms/trusty/glance-simplestreams-sync/lp1522929

Merge into:

lp://qastaging/charms/trusty/glance-simplestreams-sync

Diff against target:

12 lines (+0/-2)

1 file modified

hooks/hooks.py (+0/-2)

To merge this branch:

bzr merge lp://qastaging/~freyes/charms/trusty/glance-simplestreams-sync/lp1522929

High

Fix Released

Link a bug report

Reviewer	Date Requested	Status
Mike McCracken (community)		Approve on 2016-01-15
Edward Hope-Morley	2015-12-04	Approve on 2016-01-13
Review via email: mp+279644@code.qastaging.launchpad.net

Description of the change

Dear Maintainers,

This patch removed the addition of ppa:cloud-installer/simplestreams-testing during the install hook, that PPA contains a buggy version of simplestreams, the trusty version has a few security fixes applied and one patch to make it work ceph-radosgw

Best,

Revision history for this message

Mike McCracken (mikemc) wrote on 2015-12-04:

Please hold off on merging this until we can resolve the problem that the version of simplestreams in Trusty does not support functionality needed by the cloud installer.

If this is urgently blocking you, I can offer to update the PPA to a current version of simplestreams that fixes LP #1346935.

Revision history for this message

Edward Hope-Morley (hopem) wrote on 2016-01-13:

@mikemc Charms should not install PPAs (some people are doing offline installs and want to control apt/image sources without interference from the charm). If this is absolutely necessary then it should at least be an option e.g. the ceph charm allows an alternate source to be installed with the 'source' option.

review: Approve

Revision history for this message

Mike McCracken (mikemc) wrote on 2016-01-15:

@hopem, that sounds like a good approach.
Let's merge this branch now and we can add the option and use that to fix the cloud installer afterwards.

review: Approve

Revision history for this message

Felipe Reyes (freyes) wrote on 2016-01-18:

Submitted another MP that targets /next[0], probably that one should be merged first.

https://code.launchpad.net/~freyes/charms/trusty/glance-simplestreams-sync/lp1522929-next/+merge/282958