openstack
/
ironic-python-agent
Code Issues Proposed changes

Add function to calculate memory 

Move logic to calculate memory to its own function.

Change-Id: I5ab98b6450ff45dff35ddae093a83140f37047a8
This commit is contained in:
Riccardo Pittau 2020-04-23 16:32:23 +02:00
parent 8adb7e1a04
commit 2738e57f2a
1 changed files with 22 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

38
ironic_python_agent/hardware.py
View File

@ -162,6 +162,27 @@ def _get_component_devices(raid_device):
return component_devices
def _calc_memory(sys_dict):
physical = 0
for sys_child in sys_dict['children']:
if sys_child['id'] != 'core':
continue
for core_child in sys_child['children']:
if not _MEMORY_ID_RE.match(core_child['id']):
continue
if (not core_child.get("children")
and core_child.get('size')):
value = ("%(size)s %(units)s" % core_child)
physical += int(UNIT_CONVERTER(value).to
('MB').magnitude)
for bank in core_child.get('children', ()):
if bank.get('size'):
value = ("%(size)s %(units)s" % bank)
physical += int(UNIT_CONVERTER(value).to
('MB').magnitude)
return physical
def get_holder_disks(raid_device):
"""Get the holder disks of a Software RAID device.
@ -989,22 +1010,7 @@ class GenericHardwareManager(HardwareManager):
LOG.warning('Could not get real physical RAM from lshw: %s', e)
physical = None
else:
physical = 0
# locate memory information in system_dict
for sys_child in sys_dict['children']:
if sys_child['id'] == 'core':
for core_child in sys_child['children']:
if _MEMORY_ID_RE.match(core_child['id']):
if (not core_child.get("children")
and core_child.get('size')):
value = ("%(size)s %(units)s" % core_child)
physical += int(UNIT_CONVERTER(value).to
('MB').magnitude)
for bank in core_child.get('children', ()):
if bank.get('size'):
value = ("%(size)s %(units)s" % bank)
physical += int(UNIT_CONVERTER(value).to
('MB').magnitude)
physical = _calc_memory(sys_dict)
if not physical:
LOG.warning('Did not find any physical RAM')