live migration of instance should claim resources on target compute node
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenStack Compute (nova) |
Fix Released
|
Medium
|
Artom Lifshitz | ||
Train |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
I'm looking at the current Icehouse code, but this applies to previous versions as well.
When we create a new instance via _build_instance() or _build_
During a cold migration we call prep_resize() which calls resize_claim() to reserve resources.
However, when we live-migrate or evacuate an instance we don't do this. As far as I can see the current code will just spawn the new instance but the resource usage won't be updated until the audit runs at some unknown time in the future at which point it will add the new instance to self.tracked_
This means that until the audit runs the scheduler has a stale view of system resources.
tags: | added: compute |
Changed in nova: | |
status: | New → Triaged |
importance: | Undecided → Medium |
Changed in nova: | |
assignee: | nobody → Rohan (kanaderohan) |
Changed in nova: | |
assignee: | Rohan (kanaderohan) → Chris Friesen (cbf123) |
Changed in nova: | |
assignee: | jichenjc (jichenjc) → Alex Xu (xuhj) |
Changed in nova: | |
assignee: | Alex Xu (xuhj) → jichenjc (jichenjc) |
tags: | added: live-migrate |
tags: |
added: live-migration removed: live-migrate |
Changed in nova: | |
assignee: | Nikola Đipanov (ndipanov) → Sylvain Bauza (sylvain-bauza) |
Changed in nova: | |
assignee: | Sylvain Bauza (sylvain-bauza) → sahid (sahid-ferdjaoui) |
Changed in nova: | |
assignee: | sahid (sahid-ferdjaoui) → Sylvain Bauza (sylvain-bauza) |
Changed in nova: | |
assignee: | Sylvain Bauza (sylvain-bauza) → sahid (sahid-ferdjaoui) |
Changed in nova: | |
assignee: | sahid (sahid-ferdjaoui) → Stephen Finucane (stephenfinucane) |
Changed in nova: | |
assignee: | Stephen Finucane (stephenfinucane) → Pawel Koniszewski (pawel-koniszewski) |
Changed in nova: | |
assignee: | Pawel Koniszewski (pawel-koniszewski) → sahid (sahid-ferdjaoui) |
Changed in nova: | |
assignee: | sahid (sahid-ferdjaoui) → Pawel Koniszewski (pawel-koniszewski) |
Changed in nova: | |
assignee: | Pawel Koniszewski (pawel-koniszewski) → Andrey Volkov (avolkov) |
Changed in nova: | |
assignee: | Andrey Volkov (avolkov) → Stephen Finucane (stephenfinucane) |
Changed in nova: | |
assignee: | Stephen Finucane (stephenfinucane) → sahid (sahid-ferdjaoui) |
Changed in nova: | |
assignee: | sahid (sahid-ferdjaoui) → Artom Lifshitz (notartom) |
Fix proposed to branch: master /review. openstack. org/79806
Review: https:/