Merge lp://staging/~niedbalski/charms/precise/mysql/lp-1296736 into lp://staging/charms/mysql

Proposed by Jorge Niedbalski
Status: Merged
Merged at revision: 119
Proposed branch: lp://staging/~niedbalski/charms/precise/mysql/lp-1296736
Merge into: lp://staging/charms/mysql
Diff against target: 67 lines (+16/-9)
2 files modified
config.yaml (+1/-1)
hooks/config-changed (+15/-8)
To merge this branch: bzr merge lp://staging/~niedbalski/charms/precise/mysql/lp-1296736
Reviewer Review Type Date Requested Status
Charles Butler (community) Approve
Review via email: mp+214844@code.staging.launchpad.net

Description of the change

Fix for bug #1296736

To post a comment you must log in.
118. By Jorge Niedbalski

[config-changed] if not ON, DEMAND , then set the value = 0

Revision history for this message
Charles Butler (lazypower) wrote :

Jorge,

Thank you for this patch! I've reviewed and deployed the code. It looks good to me. Merging this in and closing out the bugs as fixed-release.

This will show up in the store in ~ 15 to 30 minutes post merging. If you have any questions/comments/concerns about the review contact us in #juju on irc.freenode.net or email the mailing list <email address hidden>

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