Merge lp://staging/~1chb1n/openstack-charm-testing/pxc-vs-mysql-vs-precise-config-options into lp://staging/openstack-charm-testing

Proposed by Ryan Beisner
Status: Merged
Merged at revision: 214
Proposed branch: lp://staging/~1chb1n/openstack-charm-testing/pxc-vs-mysql-vs-precise-config-options
Merge into: lp://staging/openstack-charm-testing
Diff against target: 720 lines (+420/-138)
3 files modified
bundles/dev/next-msg-split.yaml (+332/-0)
bundles/sparse/default.yaml (+43/-68)
bundles/sparse/next.yaml (+45/-70)
To merge this branch: bzr merge lp://staging/~1chb1n/openstack-charm-testing/pxc-vs-mysql-vs-precise-config-options
Reviewer Review Type Date Requested Status
David Ames (community) Approve
Ryan Beisner Needs Information
Review via email: mp+296963@code.staging.launchpad.net

Commit message

Update mysql vs pxc inheritance and drop split-msg in sparse bundles

Precise needs to use the mysql charm which has different charm config options than percona-cluster.

Non-Precise needs to use the percona-cluster charm which has different charm config options than mysql proper.

Also simplify sparse bundles for single amqp only and save split-msg to a new dev bundle for later use if necessary.

Description of the change

Update mysql vs pxc inheritance and drop split-msg in sparse bundles

Precise needs to use the mysql charm which has different charm config options than percona-cluster.

Non-Precise needs to use the percona-cluster charm which has different charm config options than mysql proper.

Also simplify sparse bundles for single amqp only and save split-msg to a new dev bundle for later use if necessary.

The resultant next.yaml looks like:
http://bazaar.launchpad.net/~1chb1n/openstack-charm-testing/pxc-vs-mysql-vs-precise-config-options/view/head:/bundles/sparse/next.yaml

To post a comment you must log in.
Revision history for this message
Ryan Beisner (1chb1n) wrote :

FYI - I've set OSCI to run all of its o-c-t jobs with this branch. Will wait 1 day for those schedules, then report back here.

review: Needs Information
Revision history for this message
David Ames (thedac) wrote :

Looks good to me. I love the clean-up of the split message rabbit. Merging shortly.

review: Approve

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: