Merge lp:~jjo/charms/trusty/mysql/add-wait-timeout into lp:charms/trusty/mysql

Proposed by JuanJo Ciarlante
Status: Merged
Merged at revision: 143
Proposed branch: lp:~jjo/charms/trusty/mysql/add-wait-timeout
Merge into: lp:charms/trusty/mysql
Diff against target: 42 lines (+12/-1)
2 files modified
config.yaml (+4/-0)
hooks/config-changed (+8/-1)
To merge this branch: bzr merge lp:~jjo/charms/trusty/mysql/add-wait-timeout
Reviewer Review Type Date Requested Status
Jorge Niedbalski (community) Approve
Review via email: mp+251744@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 #2463 mysql for jjo mp251744
    LINT OK: passed

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

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

charm_unit_test #2253 mysql for jjo mp251744
    UNIT FAIL: unit-test failed

UNIT Results (max last 2 lines):
  FAILED (errors=1)
  make: *** [test] Error 1

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

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

charm_amulet_test #2340 mysql for jjo mp251744
    AMULET FAIL: amulet-test missing

AMULET Results (max last 2 lines):
INFO:root:Search string not found in makefile target commands.
ERROR:root:No make target was executed.

Full amulet test output: http://paste.ubuntu.com/10528531/
Build: http://10.245.162.77:8080/job/charm_amulet_test/2340/

Revision history for this message
Jorge Niedbalski (niedbalski) wrote :

LGTM +1

review: Approve

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-02-04 16:16:30 +0000
3+++ config.yaml 2015-03-04 14:12:08 +0000
4@@ -27,6 +27,10 @@
5 default: -1
6 type: int
7 description: Maximum connections to allow. -1 means use the server's compiled in default.
8+ wait-timeout:
9+ default: -1
10+ type: int
11+ description: The number of seconds the server waits for activity on a noninteractive connection before closing it. -1 means use the server's compiled in default.
12 binlog-format:
13 default: 'MIXED'
14 type: string
15
16=== modified file 'hooks/config-changed'
17--- hooks/config-changed 2014-12-23 12:18:57 +0000
18+++ hooks/config-changed 2015-03-04 14:12:08 +0000
19@@ -177,8 +177,14 @@
20 else:
21 configs['max-connections'] = 'max_connections = %s' % configs['max-connections']
22
23+if configs['wait-timeout'] == -1:
24+ configs['wait-timeout'] = '# wait_timeout = ?'
25+else:
26+ configs['wait-timeout'] = 'wait_timeout = %s' % configs['wait-timeout']
27+
28+
29 if configs['prefer-ipv6']:
30- configs['bind-address'] = '::'
31+ configs['bind-address'] = '::'
32 else:
33 configs['bind-address'] = '0.0.0.0'
34
35@@ -255,6 +261,7 @@
36 # the first time they are touched
37 myisam-recover = BACKUP
38 %(max-connections)s
39+%(wait-timeout)s
40 #table_cache = 64
41 #thread_concurrency = 10
42 #

Subscribers

People subscribed via source and target branches

to all changes: