Skip to content

Commit

Permalink
bug 1513086: add cores per socket to OpenStack cloud inventory parser
Browse files Browse the repository at this point in the history
https://bugzilla.redhat.com/show_bug.cgi?id=1513086

For OpenStack cloud, cpu_cores_per_socket was always set to 1 in the legacy refresh_parser
since OpenStack flavors only have a single "number of CPUs" param. The new inventory
parser was not setting this param at all, though, leaving it as nil.
  • Loading branch information
sseago committed Aug 29, 2018
1 parent c8d62fb commit 4d0c9df
Showing 1 changed file with 1 addition and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -330,6 +330,7 @@ def vms
hardware = persister.hardwares.find_or_build(server)
hardware.vm_or_template = server
hardware.cpu_sockets = flavor.try(:vcpus)
hardware.cpu_cores_per_socket = 1
hardware.cpu_total_cores = flavor.try(:vcpus)
hardware.cpu_speed = parent_host.try(:hardware).try(:cpu_speed)
hardware.memory_mb = flavor.try(:ram)
Expand Down

0 comments on commit 4d0c9df

Please sign in to comment.