Merge lp:~nobuto/landscape-bundles/lp1840209 into lp:landscape-bundles

Proposed by Nobuto Murata
Status: Merged
Approved by: Simon Poirier
Approved revision: 51
Merged at revision: 51
Proposed branch: lp:~nobuto/landscape-bundles/lp1840209
Merge into: lp:landscape-bundles
Diff against target: 12 lines (+1/-1)
1 file modified
render-bundles (+1/-1)
To merge this branch: bzr merge lp:~nobuto/landscape-bundles/lp1840209
Reviewer Review Type Date Requested Status
Simon Poirier (community) Approve
🤖 Landscape Builder test results Approve
Review via email: mp+371428@code.launchpad.net

Commit message

Bump landscape-server's mem contraint to 4096

The current 2048 may cause OOM especially for periodic tasks such as
update_security_db.sh, LP: #1840209

To post a comment you must log in.
Revision history for this message
🤖 Landscape Builder (landscape-builder) :
review: Abstain (executing tests)
Revision history for this message
🤖 Landscape Builder (landscape-builder) wrote :

Command: make ci-test
Result: Success
Revno: 51
Branch: lp:~nobuto/landscape-bundles/lp1840209
Jenkins: https://ci.lscape.net/job/latch-test-xenial/3990/

review: Approve (test results)
Revision history for this message
Simon Poirier (simpoir) wrote :

+1 yes that constraint was a bit low.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'render-bundles'
2--- render-bundles 2019-01-24 23:15:15 +0000
3+++ render-bundles 2019-08-18 16:05:14 +0000
4@@ -17,7 +17,7 @@
5 "haproxy": {},
6 "landscape": {
7 "charm": "cs:bionic/landscape-server-31",
8- "memory": 2048,
9+ "memory": 4096,
10 # We defer to the default "install_sources" from the charm.
11 "install_keys": "4652B4E6"}
12 }

Subscribers

People subscribed via source and target branches

to all changes: