Merge lp:~hopem/charms/trusty/ceph-osd/set-default-high-kernel-pid-max into lp:~openstack-charmers-archive/charms/trusty/ceph-osd/next

Proposed by Edward Hope-Morley
Status: Merged
Merged at revision: 35
Proposed branch: lp:~hopem/charms/trusty/ceph-osd/set-default-high-kernel-pid-max
Merge into: lp:~openstack-charmers-archive/charms/trusty/ceph-osd/next
Diff against target: 18 lines (+4/-2)
1 file modified
config.yaml (+4/-2)
To merge this branch: bzr merge lp:~hopem/charms/trusty/ceph-osd/set-default-high-kernel-pid-max
Reviewer Review Type Date Requested Status
Liam Young (community) Approve
Review via email: mp+246695@code.launchpad.net
To post a comment you must log in.
Revision history for this message
uosci-testing-bot (uosci-testing-bot) wrote :

charm_unit_test #818 ceph-osd-next for hopem mp246695
    UNIT OK: passed

Build: http://10.245.162.77:8080/job/charm_unit_test/818/

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

charm_lint_check #789 ceph-osd-next for hopem mp246695
    LINT OK: passed

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

Revision history for this message
Liam Young (gnuoy) wrote :

Approve

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

charm_amulet_test #974 ceph-osd-next for hopem mp246695
    AMULET OK: passed

Build: http://10.245.162.77:8080/job/charm_amulet_test/974/

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-01-09 16:13:45 +0000
3+++ config.yaml 2015-01-16 11:03:10 +0000
4@@ -123,10 +123,12 @@
5 your network interface.
6 sysctl:
7 type: string
8- default: ''
9+ default: '{ kernel.pid_max : 2097152 }'
10 description: |
11 YAML-formatted associative array of sysctl key/value pairs to be set
12- persistently e.g. '{ kernel.pid_max : 4194303 }'.
13+ persistently. As a default we set pid_max to a high value to avoid
14+ problems with large numbers (>20) of OSDs recovering. very large clusters
15+ should set this value even higher (max 4194303).
16 nagios_context:
17 default: "juju"
18 description: |

Subscribers

People subscribed via source and target branches