openstack/nova
Code Issues Proposed changes

Merge "Report reserved_host_disk_mb in GB not KB"

Browse Source
This commit is contained in:
Jenkins 2017-02-02 20:33:31 +00:00 committed by Gerrit Code Review
commit c90acdbc55
2 changed files with 11 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
nova/scheduler/client/report.py
View File

@ -14,6 +14,7 @@
# under the License.
import functools
import math
import re
from six.moves.urllib import parse
import time
@ -115,9 +116,16 @@ def _compute_node_to_inventory_dict(compute_node):
'allocation_ratio': compute_node.ram_allocation_ratio,
}
if compute_node.local_gb > 0:
reserved_disk_gb = 0
# TODO(johngarbutt) We should either move to reserved_host_disk_gb
# or start tracking DISK_MB.
if CONF.reserved_host_disk_mb:
reserved_disk_gb = CONF.reserved_host_disk_mb / 1024.0
# ensure we reserve enough space by rounding up to nearest GB
reserved_disk_gb = int(math.ceil(reserved_disk_gb))
result[DISK_GB] = {
'total': compute_node.local_gb,
'reserved': CONF.reserved_host_disk_mb * 1024,
'reserved': reserved_disk_gb,
'min_unit': 1,
'max_unit': compute_node.local_gb,
'step_size': 1,

4
nova/tests/unit/scheduler/client/test_report.py
View File

@ -556,7 +556,7 @@ class TestComputeNodeToInventoryDict(test.NoDBTestCase):
disk_allocation_ratio=1.0)
self.flags(reserved_host_memory_mb=1000)
self.flags(reserved_host_disk_mb=2000)
self.flags(reserved_host_disk_mb=200)
result = report._compute_node_to_inventory_dict(compute_node)
@ -579,7 +579,7 @@ class TestComputeNodeToInventoryDict(test.NoDBTestCase):
},
'DISK_GB': {
'total': compute_node.local_gb,
'reserved': CONF.reserved_host_disk_mb * 1024,
'reserved': 1, # this is ceil(1000/1024)
'min_unit': 1,
'max_unit': compute_node.local_gb,
'step_size': 1,