There is no allocation record for migration action

Bug #1621709 reported by Alex Xu
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Medium
Unassigned

Bug Description

In the current RT, the migration case was consider as resource consuming. But we didn't update any allocation record to Placement API.

Tags: placement
Alex Xu (xuhj)
Changed in nova:
assignee: nobody → Alex Xu (xuhj)
tags: added: placement
Matt Riedemann (mriedem)
Changed in nova:
status: New → Confirmed
importance: Undecided → High
importance: High → Medium
Revision history for this message
Alex Xu (xuhj) wrote :

https://review.openstack.org/#/c/369147/

The first step or at least patch I prefer to merge in newton. This can avoid to leak allocation record for migration and failure case.

Changed in nova:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

Fix proposed to branch: master
Review: https://review.openstack.org/369158

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Fix proposed to branch: master
Review: https://review.openstack.org/369172

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Fix proposed to branch: master
Review: https://review.openstack.org/369173

Matt Riedemann (mriedem)
tags: added: newton-rc-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.openstack.org/369147
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=9374566d194be1e861e7d08dd0e3b2a615ff408c
Submitter: Jenkins
Branch: master

commit 9374566d194be1e861e7d08dd0e3b2a615ff408c
Author: He Jie Xu <email address hidden>
Date: Tue Sep 13 11:20:16 2016 +0800

    Clean up allocation when update available resources

    Currently the 'update_available_resource' only update the allocations
    records for the instances which are tracked by RT. Which leads to leak
    allocations record in the DB. The cases for leaking:

    * When execute resize, the instance is moved to another host and the
    instance.host is changed to the new host. But RT only update the
    instance which instance.host is on the same host. It leads to the
    allocation record on the source host leave in the DB.
    * The instance was deleted locally when the compute node is down. After
    the compute node back, the related allocations record can't be cleanup.

    This patch add new method for the scheduler client. It will remove all
    the allocations records for the instance which isn't on the host.

    Change-Id: I28884c4edac8825a4753449f1b6e1eddded0d52b
    Partial-Bug: #1621709

Matt Riedemann (mriedem)
tags: removed: newton-rc-potential
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on nova (master)

Change abandoned by Michael Still (<email address hidden>) on branch: master
Review: https://review.openstack.org/369172
Reason: This patch has been sitting unchanged for more than 12 weeks. I am therefore going to abandon it to keep the nova review queue sane. Please feel free to restore the change if you're still working on it.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Change abandoned by Michael Still (<email address hidden>) on branch: master
Review: https://review.openstack.org/369173
Reason: This patch has been sitting unchanged for more than 12 weeks. I am therefore going to abandon it to keep the nova review queue sane. Please feel free to restore the change if you're still working on it.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Change abandoned by Michael Still (<email address hidden>) on branch: master
Review: https://review.openstack.org/369159
Reason: This patch has been sitting unchanged for more than 12 weeks. I am therefore going to abandon it to keep the nova review queue sane. Please feel free to restore the change if you're still working on it.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote :

Change abandoned by Michael Still (<email address hidden>) on branch: master
Review: https://review.openstack.org/369158
Reason: This patch has been sitting unchanged for more than 12 weeks. I am therefore going to abandon it to keep the nova review queue sane. Please feel free to restore the change if you're still working on it.

Revision history for this message
Sean Dague (sdague) wrote :

There are no currently open reviews on this bug, changing the status back to the previous state and unassigning. If there are active reviews related to this bug, please include links in comments.

Changed in nova:
status: In Progress → Confirmed
assignee: Alex Xu (xuhj) → nobody
Revision history for this message
Chris Dent (cdent) wrote :

This is effectively a duplicate of #1707071 , which has been released, so I'm going to mark this as such.

Changed in nova:
status: Confirmed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.