B OpenNebula#6772: Additional fix Host NUMA nodes after VM migration

Signed-off-by: Kristian Feldsam <[email protected]>
FELDSAM-INC · Nov 4, 2024 · 4764133 · 4764133
1 parent 305b33e
commit 4764133
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 8 deletions.
diff --git a/src/lcm/LifeCycleActions.cc b/src/lcm/LifeCycleActions.cc
@@ -348,13 +348,6 @@ void LifeCycleManager::trigger_migrate(int vid, const RequestAttributes& ra,
 
             hpool->add_capacity(vm->get_hid(), sr);
 
-            if ( vm->get_hid() != vm->get_previous_hid() )
-            {
-                hpool->del_capacity(vm->get_previous_hid(), sr);
-
-                vm->release_previous_vnc_port();
-            }
-
             vm->set_stime(the_time);
 
             vm->set_prolog_stime(the_time);
@@ -365,6 +358,18 @@ void LifeCycleManager::trigger_migrate(int vid, const RequestAttributes& ra,
 
             vmpool->update(vm.get());
 
+            if ( vm->get_hid() != vm->get_previous_hid() )
+            {
+                Template tmpl;
+                vm->get_previous_capacity(sr, tmpl);
+
+                hpool->del_capacity(vm->get_previous_hid(), sr);
+
+                vm->release_previous_vnc_port();
+            }
+
+            vmpool->update(vm.get());
+
             //----------------------------------------------------
 
             tm->trigger_prolog_migr(vm.get());
@@ -1038,6 +1043,7 @@ void LifeCycleManager::clean_up_vm(VirtualMachine * vm, bool dispose,
                                    int& image_id, int uid, int gid, int req_id, Template& quota_tmpl)
 {
     HostShareCapacity sr;
+    Template tmpl;
 
     time_t the_time = time(0);
 
@@ -1249,6 +1255,7 @@ void LifeCycleManager::clean_up_vm(VirtualMachine * vm, bool dispose,
                 vm->set_previous_vm_info();
                 vm->set_previous_running_etime(the_time);
 
+                vm->get_previous_capacity(sr, tmpl);
                 hpool->del_capacity(vm->get_previous_hid(), sr);
 
                 vmpool->update_previous_history(vm);
@@ -1272,6 +1279,7 @@ void LifeCycleManager::clean_up_vm(VirtualMachine * vm, bool dispose,
                 vm->set_previous_vm_info();
                 vm->set_previous_running_etime(the_time);
 
+                vm->get_previous_capacity(sr, tmpl);
                 hpool->del_capacity(vm->get_previous_hid(), sr);
 
                 vmpool->update_previous_history(vm);

diff --git a/src/lcm/LifeCycleStates.cc b/src/lcm/LifeCycleStates.cc
@@ -298,7 +298,8 @@ void LifeCycleManager::trigger_deploy_success(int vid)
 
             vmpool->update_previous_history(vm.get());
 
-            vm->get_capacity(sr);
+            Template tmpl;
+            vm->get_previous_capacity(sr, tmpl);
 
             hpool->del_capacity(vm->get_previous_hid(), sr);