Merge lp:~marcoceppi/charms/trusty/mysql/lp1373862 into lp:charms/trusty/mysql

Proposed by Marco Ceppi
Status: Work in progress
Proposed branch: lp:~marcoceppi/charms/trusty/mysql/lp1373862
Merge into: lp:charms/trusty/mysql
Diff against target: 11 lines (+1/-1)
1 file modified
config.yaml (+1/-1)
To merge this branch: bzr merge lp:~marcoceppi/charms/trusty/mysql/lp1373862
Reviewer Review Type Date Requested Status
Ryan Beisner (community) Needs Information
charmers Pending
Review via email: mp+267948@code.launchpad.net
To post a comment you must log in.
Revision history for this message
uosci-testing-bot (uosci-testing-bot) wrote :

charm_lint_check #7988 mysql for marcoceppi mp267948
    LINT OK: passed

Build: http://10.245.162.77:8080/job/charm_lint_check/7988/

Revision history for this message
uosci-testing-bot (uosci-testing-bot) wrote :

charm_unit_test #7403 mysql for marcoceppi mp267948
    UNIT FAIL: unit-test failed

UNIT Results (max last 2 lines):
make: *** [test] Error 1
ERROR:root:Make target returned non-zero.

Full unit test output: http://paste.ubuntu.com/12071051/
Build: http://10.245.162.77:8080/job/charm_unit_test/7403/

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

Please see comment on bug 1373862.

If we do move forward with this default value change, we will need to update the amulet test as well.

The test knows about the former 80% default charm config value, in the interest of making sure the "deployed" == the "expected."

As an aside, the unit tests are known-failing (bug 1398918), as they're not really mocking.

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

Marco - can you rebase your proposed branch to ensure that the other landed changes are pulled into this for testing?

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

Marco - can you rebase your proposed branch to ensure that the other landed changes are pulled into this for testing?

review: Needs Information

Unmerged revisions

147. By Marco Ceppi

Sane dataset-size. Fixes local provider and is recommended value from MySQL experts

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'config.yaml'
2--- config.yaml 2015-04-21 06:59:10 +0000
3+++ config.yaml 2015-08-13 13:27:49 +0000
4@@ -1,6 +1,6 @@
5 options:
6 dataset-size:
7- default: '80%'
8+ default: '128M'
9 description: How much data do you want to keep in memory in the DB. This will be used to tune settings in the database server appropriately. Any more specific settings will override these defaults though. This currently sets innodb_buffer_pool_size or key_cache_size depending on the setting in preferred-storage-engine. If query-cache-type is set to 'ON' or 'DEMAND' 20% of this is given to query-cache-size. Suffix this value with 'K','M','G', or 'T' to get the relevant kilo/mega/etc. bytes. If suffixed with %, one will get that percentage of RAM devoted to dataset and (if enabled) query cache.
10 type: string
11 preferred-storage-engine:

Subscribers

People subscribed via source and target branches

to all changes: