Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Synchronise 2023.1 with upstream #117

Merged
merged 1 commit into from
Oct 7, 2024

Conversation

github-actions[bot]
Copy link

@github-actions github-actions bot commented Oct 7, 2024

This PR contains a snapshot of 2023.1 from upstream stable/2023.1.

The commit c1ccc1a introduced
a regression when NUMA live migration was done on shared storage

The live migration support for the power mgmt feature means we need to
call driver.cleanup() for all NUMA instances to potentially offline
pcpus that are not used any more after the instance is migrated away.
However this change exposed an issue with the disk cleanup logic. Nova
should never delete the instance directory if that directory is on
shared storage (e.g. the nova instances path is backed by NFS).

This patch will fix that behavior so live migration will function

2023.x only: Conflict resolved in compute/manager.py due to
bluperint libvirt-mdev-live-migrate is not in 2023.x and the related
unit test is removed.

Closes-Bug: #2080436
Change-Id: Ia2bbb5b4ac728563a8aabd857ed0503449991df1
(cherry picked from commit 035b840)
(cherry picked from commit 57e037b)
(cherry picked from commit 0e4406a)
@github-actions github-actions bot requested a review from a team as a code owner October 7, 2024 06:25
@priteau priteau closed this Oct 7, 2024
@priteau priteau reopened this Oct 7, 2024
@priteau priteau merged commit a2f49a7 into stackhpc/2023.1 Oct 7, 2024
3 checks passed
@priteau priteau deleted the upstream/2023.1-2024-10-07 branch October 7, 2024 07:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants