fix nits + validate numa_topology_test
Change-Id: I901c399e53b7c62ac92ac9c54bea0ebea5c040b5
This commit is contained in:
parent
ba71d6ed58
commit
643200e9aa
|
@ -41,7 +41,7 @@ def get_host_numa_placement(instance, vcpus):
|
|||
% out.strip(), shell=True)
|
||||
cgroup = cgroup.split(":")[-1].strip()
|
||||
if cgroup.index('emulator'):
|
||||
cgroup = cgroup + '/..'
|
||||
cgroup += '/..'
|
||||
|
||||
placement = []
|
||||
for i in range(vcpus):
|
||||
|
@ -63,9 +63,8 @@ class NUMARemoteClient(remote_client.RemoteClient):
|
|||
for i in range(int(node_count)):
|
||||
node_cmd = 'cat /sys/devices/system/node/node%d/' % i
|
||||
|
||||
node = {}
|
||||
node['cpu'] = self.exec_command(node_cmd + 'cpulist')
|
||||
node['mem'] = self.exec_command(node_cmd + 'meminfo')
|
||||
node = {'cpu': self.exec_command(node_cmd + 'cpulist'),
|
||||
'mem': self.exec_command(node_cmd + 'meminfo')}
|
||||
nodes.append(node)
|
||||
|
||||
return nodes
|
||||
|
@ -146,11 +145,11 @@ class NUMAServersTest(base.BaseV2ComputeAdminTest):
|
|||
|
||||
# Validate guest topology
|
||||
# TODO(stephenfin): Validate more of the NUMA topology than this
|
||||
numa_nodes = linux_client.get_numa_nodes()
|
||||
numa_nodes = linux_client.get_numa_topology()
|
||||
self.assertEqual(2, len(numa_nodes))
|
||||
|
||||
# Validate host topology
|
||||
placement = self.get_placement(server, 4)
|
||||
placement = get_host_numa_placement(server, 4)
|
||||
self.assertEqual(placement[0], placement[1])
|
||||
self.assertNotEqual(placement[1], placement[2])
|
||||
self.assertEqual(placement[2], placement[3])
|
||||
|
|
Loading…
Reference in New Issue