Merge lp:~gnuoy/charms/trusty/nova-compute/no-hugepages into lp:~openstack-charmers-archive/charms/trusty/nova-compute/next

Proposed by Liam Young
Status: Merged
Merged at revision: 185
Proposed branch: lp:~gnuoy/charms/trusty/nova-compute/no-hugepages
Merge into: lp:~openstack-charmers-archive/charms/trusty/nova-compute/next
Diff against target: 26 lines (+10/-8)
1 file modified
hooks/nova_compute_utils.py (+10/-8)
To merge this branch: bzr merge lp:~gnuoy/charms/trusty/nova-compute/no-hugepages
Reviewer Review Type Date Requested Status
David Ames (community) Approve
Review via email: mp+279245@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 #14751 nova-compute-next for gnuoy mp279245
    LINT OK: passed

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

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

charm_unit_test #13754 nova-compute-next for gnuoy mp279245
    UNIT OK: passed

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

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

charm_amulet_test #8071 nova-compute-next for gnuoy mp279245
    AMULET FAIL: amulet-test failed

AMULET Results (max last 2 lines):
make: *** [functional_test] Error 124
ERROR:root:Make target returned non-zero.

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

Revision history for this message
David Ames (thedac) wrote :

LGTM, merging

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

charm_amulet_test #8075 nova-compute-next for gnuoy mp279245
    AMULET OK: passed

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

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'hooks/nova_compute_utils.py'
2--- hooks/nova_compute_utils.py 2015-11-25 12:48:50 +0000
3+++ hooks/nova_compute_utils.py 2015-12-02 09:56:26 +0000
4@@ -807,14 +807,16 @@
5 # and support multiple pool sizes - e.g. 2M and 1G.
6 hugepage_size = 2048
7 hugepage_config = config('hugepages')
8- if hugepage_config.endswith('%'):
9- import psutil
10- mem = psutil.virtual_memory()
11- hugepage_config_pct = hugepage_config.strip('%')
12- hugepage_multiplier = float(hugepage_config_pct) / 100
13- hugepages = int((mem.total * hugepage_multiplier) / hugepage_size)
14- else:
15- hugepages = int(hugepage_config)
16+ hugepages = None
17+ if hugepage_config:
18+ if hugepage_config.endswith('%'):
19+ import psutil
20+ mem = psutil.virtual_memory()
21+ hugepage_config_pct = hugepage_config.strip('%')
22+ hugepage_multiplier = float(hugepage_config_pct) / 100
23+ hugepages = int((mem.total * hugepage_multiplier) / hugepage_size)
24+ else:
25+ hugepages = int(hugepage_config)
26 return hugepages
27
28

Subscribers

People subscribed via source and target branches