Skip to content

compute: add cpu info topology cells entry#3546

Merged
stephenfin merged 1 commit intogophercloud:mainfrom
vooon:add-nova-cpu-topology-cells
Nov 3, 2025
Merged

compute: add cpu info topology cells entry#3546
stephenfin merged 1 commit intogophercloud:mainfrom
vooon:add-nova-cpu-topology-cells

Conversation

@vooon
Copy link
Copy Markdown
Contributor

@vooon vooon commented Oct 31, 2025

Fixes #3545

Probably better to default value to 1, so Cells * Sockets * Cores * Threads would always work as expected.

Links to the line numbers/files in the OpenStack source code that support the
code in this PR:

https://github.com/openstack/nova/blob/41ae5b7f70481ad1a63eb1cbdba1e890be850ec5/nova/virt/libvirt/driver.py#L8478-L8482

Fix gophercloud#3545

Signed-off-by: Vladimir Ermakov <vooon341@gmail.com>
@github-actions github-actions bot added edit:compute This PR updates compute code semver:minor Backwards-compatible change backport-v2 This PR will be backported to v2 labels Oct 31, 2025
@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 63.823%. remained the same
when pulling 86e7a03 on vooon:add-nova-cpu-topology-cells
into 4879eb5 on gophercloud:main.

@stephenfin stephenfin added this pull request to the merge queue Nov 3, 2025
Merged via the queue into gophercloud:main with commit 019abbf Nov 3, 2025
80 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-v2 This PR will be backported to v2 edit:compute This PR updates compute code semver:minor Backwards-compatible change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

nova: absent hypervisor cpu_info.topology.cells report

3 participants