Merge ~cjwatson/launchpad-mojo-specs:git-bump-maxconn into launchpad-mojo-specs:master

Proposed by Colin Watson
Status: Merged
Merged at revision: ee866a9edc540d57df1efd59557d09a6b1d2661a
Proposed branch: ~cjwatson/launchpad-mojo-specs:git-bump-maxconn
Merge into: launchpad-mojo-specs:master
Diff against target: 14 lines (+2/-0)
1 file modified
lp-git/bundle.yaml (+2/-0)
Reviewer Review Type Date Requested Status
Simone Pelosi Approve
Review via email: mp+453051@code.launchpad.net

Commit message

git: Bump haproxy net.core.somaxconn

Description of the change

`net.core.somaxconn 4096` is the default as of Linux 5.4, but the haproxies here are still on bionic and thus Linux 4.15.

To post a comment you must log in.
Revision history for this message
Simone Pelosi (pelpsi) wrote :

LGTM!

review: Approve
Revision history for this message
Guruprasad (lgp171188) wrote :

https://code.launchpad.net/~hloeung/launchpad-mojo-specs/+git/launchpad-mojo-specs/+merge/453096 adds a higher value for `maxconn` and this change has already been deployed manually by Haw (on William's advice) to fix the haproxy issue.

Revision history for this message
Colin Watson (cjwatson) wrote :

I've landed Haw's changes and rebased this against them - there are still a few things left as cowboys on production, so I'll land this to resolve those.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/lp-git/bundle.yaml b/lp-git/bundle.yaml
2index a4e9f89..36a9aab 100644
3--- a/lp-git/bundle.yaml
4+++ b/lp-git/bundle.yaml
5@@ -184,7 +184,9 @@ applications:
6 ssl_key: ""
7 {%- endif %}
8 sysctl: |
9+ net.core.somaxconn: 4096
10 net.ipv4.tcp_congestion_control: bbr
11+ net.ipv4.tcp_max_syn_backlog: 2048
12 {%- if not devel %}
13 autocert-haproxy:
14 charm: cs:~autocert-charmers/autocert-17

Subscribers

People subscribed via source and target branches