Merge lp:~dpb/landscape-bundles/add-machine-contraints into lp:landscape-bundles

Proposed by David Britton
Status: Merged
Approved by: David Britton
Approved revision: 44
Merged at revision: 42
Proposed branch: lp:~dpb/landscape-bundles/add-machine-contraints
Merge into: lp:landscape-bundles
Diff against target: 13 lines (+2/-1)
1 file modified
landscape-template.jinja2 (+2/-1)
To merge this branch: bzr merge lp:~dpb/landscape-bundles/add-machine-contraints
Reviewer Review Type Date Requested Status
🤖 Landscape Builder test results Approve
Francis Ginther (community) Approve
Chad Smith Approve
Review via email: mp+321491@code.launchpad.net

Commit message

bump mem constraints to 4096 for machine 0.

Description of the change

bump mem constraints to 4096 for machine 0. Tested on Maas, openstack, lxd.

To post a comment you must log in.
Revision history for this message
Francis Ginther (fginther) wrote :

Looks good.

review: Approve
Revision history for this message
🤖 Landscape Builder (landscape-builder) :
review: Abstain (executing tests)
Revision history for this message
🤖 Landscape Builder (landscape-builder) wrote :
review: Approve (test results)
Revision history for this message
Chad Smith (chad.smith) :
review: Approve
44. By David Britton

[fginther] added root-disk=30G

Revision history for this message
Francis Ginther (fginther) wrote :

+1 on the added disk constraint.

review: Approve
Revision history for this message
🤖 Landscape Builder (landscape-builder) :
review: Abstain (executing tests)
Revision history for this message
🤖 Landscape Builder (landscape-builder) wrote :
review: Approve (test results)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'landscape-template.jinja2'
2--- landscape-template.jinja2 2017-03-30 01:40:55 +0000
3+++ landscape-template.jinja2 2017-03-30 21:18:46 +0000
4@@ -91,7 +91,8 @@
5 "gui-y": "120"
6 {% if landscape["to"] %}
7 machines:
8- "0": {}
9+ "0":
10+ constraints: "mem=4096 root-disk=30G"
11 {% endif %}
12 relations:
13 - [landscape-server, rabbitmq-server]

Subscribers

People subscribed via source and target branches

to all changes: