[SRU] disk over-commit still not correctly calculated during live migration

Bug #1744079 reported by Matthew Booth
14
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Low
int32bit
Queens
In Progress
Low
Lee Yarwood
Rocky
In Progress
Low
Lee Yarwood
Ubuntu Cloud Archive
Fix Released
High
Unassigned
Mitaka
Fix Released
High
Unassigned
Ocata
Fix Released
High
Sahid Orentino
Pike
Fix Released
High
Unassigned
Queens
Fix Released
High
Unassigned
Rocky
Fix Released
High
Unassigned
nova (Ubuntu)
Fix Released
High
Unassigned
Xenial
Fix Released
High
Unassigned
Bionic
Fix Released
High
Unassigned
Cosmic
Fix Released
High
Unassigned
Disco
Fix Released
High
Unassigned

Bug Description

[Impact]
nova compares disk space with disk_available_least field, which is possible to be negative, due to overcommit.

So the migration may fail because of a "Migration pre-check error: Unable to migrate dfcd087a-5dff-439d-8875-2f702f081539: Disk of instance is too large(available on destination host:-3221225472 < need:22806528)" when trying a migration to another compute that has plenty of free space in his disk.

[Test Case]
Deploy openstack environment. Make sure there is a negative disk_available_least and a adequate free_disk_gb in one test compute node, then migrate a VM to it with disk-overcommit (openstack server migrate --live <TEST-COMPUTE-NODE> --block-migration --disk-overcommit <VM-NAME>). You will see above migration pre-check error.

This is the formula to compute disk_available_least and free_disk_gb.

disk_free_gb = disk_info_dict['free']
disk_over_committed = self._get_disk_over_committed_size_total()
available_least = disk_free_gb * units.Gi - disk_over_committed
data['disk_available_least'] = available_least / units.Gi

The following command can be used to query the value of disk_available_least

nova hypervisor-show <ID> |grep disk

Steps to Reproduce:
1. set disk_allocation_ratio config option > 1.0
2. qemu-img resize cirros-0.3.0-x86_64-disk.img +40G
3. glance image-create --disk-format qcow2 ...
4. boot VMs based on resized image
5. we see disk_available_least becomes negative

[Regression Potential]
Minimal - we're just changing from the following line:

disk_available_gb = dst_compute_info['disk_available_least']

to the following codes:

if disk_over_commit:
    disk_available_gb = dst_compute_info['free_disk_gb']
else:
    disk_available_gb = dst_compute_info['disk_available_least']

When enabling overcommit, disk_available_least is possible to be negative, so we should use free_disk_gb instead of it by backporting the following two fixes.

https://git.openstack.org/cgit/openstack/nova/commit/?id=e097c001c8e11110efe8879da57264fcb7bdfdf2
https://git.openstack.org/cgit/openstack/nova/commit/?id=e2cc275063658b23ed88824100919a6dfccb760d

This is the code path for check_can_live_migrate_destination:

_migrate_live(os-migrateLive API, migrate_server.py) -> migrate_server -> _live_migrate -> _build_live_migrate_task -> _call_livem_checks_on_host -> check_can_live_migrate_destination

BTW, redhat also has a same bug - https://bugzilla.redhat.com/show_bug.cgi?id=1477706

[Original Bug Report]
Change I8a705114d47384fcd00955d4a4f204072fed57c2 (written by me... sigh) addressed a bug which prevented live migration to a target host with overcommitted disk when made with microversion <2.25. It achieved this, but the fix is still not correct. We now do:

        if disk_over_commit:
            disk_available_gb = dst_compute_info['local_gb']

Unfortunately local_gb is *total* disk, not available disk. We actually want free_disk_gb. Fun fact: due to the way we calculate this for filesystems, without taking into account reserved space, this can also be negative.

The test we're currently running is: could we fit this guest's allocated disks on the target if the target disk was empty. This is at least better than it was before, as we don't spuriously fail early. In fact, we're effectively disabling a test which is disabled for microversion >=2.25 anyway. IOW we should fix it, but it's probably not a high priority.

int32bit (int32bit)
Changed in nova:
assignee: nobody → int32bit (int32bit)
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/536351

Changed in nova:
status: New → In Progress
Matt Riedemann (mriedem)
tags: added: libvirt
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.openstack.org/536351
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=e2cc275063658b23ed88824100919a6dfccb760d
Submitter: Zuul
Branch: master

commit e2cc275063658b23ed88824100919a6dfccb760d
Author: int32bit <email address hidden>
Date: Mon Jan 22 20:15:21 2018 +0800

    Refix disk size during live migration with disk over-commit

    Currently available disk of targer host is calculated based on
    local_gb, not available disk(free_disk_gb). This check can be
    negative if the target host has no free disk.

    Change-Id: Iec50269ef31dfe090f0cd4db95a37909661bd910
    closes-bug: 1744079

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

Fix proposed to branch: stable/rocky
Review: https://review.openstack.org/602477

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (stable/queens)

Fix proposed to branch: stable/queens
Review: https://review.openstack.org/602478

Hua Zhang (zhhuabj)
summary: - disk over-commit still not correctly calculated during live migration
+ [SRU] disk over-commit still not correctly calculated during live
+ migration
description: updated
Hua Zhang (zhhuabj)
tags: added: sts
Hua Zhang (zhhuabj)
tags: added: sts-sru
removed: sts
Hua Zhang (zhhuabj)
tags: added: sts-sru-needed
removed: sts-sru
Hua Zhang (zhhuabj)
tags: added: sts-sponsor
Hua Zhang (zhhuabj)
no longer affects: nova (Ubuntu)
Hua Zhang (zhhuabj)
tags: added: sts
Matt Riedemann (mriedem)
Changed in nova:
importance: Undecided → Low
Changed in nova (Ubuntu Bionic):
status: New → Triaged
Changed in nova (Ubuntu Cosmic):
status: New → Triaged
Changed in nova (Ubuntu Disco):
status: New → Triaged
importance: Undecided → High
Changed in nova (Ubuntu Cosmic):
importance: Undecided → High
Changed in nova (Ubuntu Bionic):
importance: Undecided → High
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "lp1744079_xenial.debdiff" seems to be a debdiff. The ubuntu-sponsors team has been subscribed to the bug report so that they can review and hopefully sponsor the debdiff. If the attachment isn't a patch, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are member of the ~ubuntu-sponsors, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issue please contact him.]

tags: added: patch
Revision history for this message
Corey Bryant (corey.bryant) wrote :

I've uploaded new versions of nova with the fix for this bug to the disco, cosmic, and bionic unapproved queues.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package nova - 2:18.0.1-0ubuntu3

---------------
nova (2:18.0.1-0ubuntu3) disco; urgency=medium

  * d/tests/nova-shebangs-py3: Update shebang for Python 3.7.

 -- Corey Bryant <email address hidden> Wed, 07 Nov 2018 09:11:09 -0500

Changed in nova (Ubuntu Disco):
status: Triaged → Fix Released
Revision history for this message
Brian Murray (brian-murray) wrote : Please test proposed package

Hello Matthew, or anyone else affected,

Accepted nova into cosmic-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/nova/2:18.0.1-0ubuntu1.1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-needed-cosmic to verification-done-cosmic. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-cosmic. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Changed in nova (Ubuntu Cosmic):
status: Triaged → Fix Committed
tags: added: verification-needed verification-needed-cosmic
Changed in nova (Ubuntu Bionic):
status: Triaged → Fix Committed
tags: added: verification-needed-bionic
Revision history for this message
Brian Murray (brian-murray) wrote :

Hello Matthew, or anyone else affected,

Accepted nova into bionic-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/nova/2:17.0.6-0ubuntu2 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-needed-bionic to verification-done-bionic. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-bionic. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Changed in cloud-archive:
status: Triaged → Fix Committed
Revision history for this message
Corey Bryant (corey.bryant) wrote :

Hello Matthew, or anyone else affected,

Accepted nova into queens-proposed. The package will build now and be available in the Ubuntu Cloud Archive in a few hours, and then in the -proposed repository.

Please help us by testing this new package. To enable the -proposed repository:

  sudo add-apt-repository cloud-archive:queens-proposed
  sudo apt-get update

Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-queens-needed to verification-queens-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-queens-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

tags: added: verification-queens-needed
Revision history for this message
Hua Zhang (zhhuabj) wrote :

Successfully verified queens-proposed nova 2:17.0.6-0ubuntu2~cloud0

tags: added: verification-queens-done
removed: verification-queens-needed
Revision history for this message
Corey Bryant (corey.bryant) wrote :

Hi Hua, would you also be able to verify cosmic-proposed and bionic-proposed? Those will need to be verified before queens-proposed can be promoted. Thanks.

Revision history for this message
Corey Bryant (corey.bryant) wrote :

This hasn't been tested yet on cosmic-proposed and we have a stable point release of nova to get out so we're going to have to combine this fix with the stable point release. That means this is going to have to live in cosmic-proposed (and rocky-proposed) for another 7 days minimum.

Revision history for this message
Brian Murray (brian-murray) wrote :

Hello Matthew, or anyone else affected,

Accepted nova into cosmic-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/nova/2:18.0.3-0ubuntu1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-needed-cosmic to verification-done-cosmic. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-cosmic. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Revision history for this message
James Page (james-page) wrote :

Hello Matthew, or anyone else affected,

Accepted nova into rocky-proposed. The package will build now and be available in the Ubuntu Cloud Archive in a few hours, and then in the -proposed repository.

Please help us by testing this new package. To enable the -proposed repository:

  sudo add-apt-repository cloud-archive:rocky-proposed
  sudo apt-get update

Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-rocky-needed to verification-rocky-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-rocky-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

tags: added: verification-rocky-needed
Revision history for this message
Corey Bryant (corey.bryant) wrote :

This hasn't been tested yet on bionic-proposed and we have a stable point release of nova to get out so we're going to have to combine this fix with the stable point release. That means this is going to have to live in bionic-proposed (and queens-proposed) for another 7 days minimum.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (stable/rocky)

Reviewed: https://review.openstack.org/602477
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=1c1c373291a93ad6304ff03ea09fea24a5cb5fd0
Submitter: Zuul
Branch: stable/rocky

commit 1c1c373291a93ad6304ff03ea09fea24a5cb5fd0
Author: int32bit <email address hidden>
Date: Mon Jan 22 20:15:21 2018 +0800

    Refix disk size during live migration with disk over-commit

    Currently available disk of targer host is calculated based on
    local_gb, not available disk(free_disk_gb). This check can be
    negative if the target host has no free disk.

    Change-Id: Iec50269ef31dfe090f0cd4db95a37909661bd910
    closes-bug: 1744079
    (cherry picked from commit e2cc275063658b23ed88824100919a6dfccb760d)

Revision history for this message
Brian Murray (brian-murray) wrote : Please test proposed package

Hello Matthew, or anyone else affected,

Accepted nova into bionic-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/nova/2:17.0.7-0ubuntu1 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-needed-bionic to verification-done-bionic. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-bionic. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Revision history for this message
Corey Bryant (corey.bryant) wrote :

Hello Matthew, or anyone else affected,

Accepted nova into queens-proposed. The package will build now and be available in the Ubuntu Cloud Archive in a few hours, and then in the -proposed repository.

Please help us by testing this new package. To enable the -proposed repository:

  sudo add-apt-repository cloud-archive:queens-proposed
  sudo apt-get update

Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-queens-needed to verification-queens-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-queens-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

tags: added: verification-queens-needed
removed: verification-queens-done
Revision history for this message
Hua Zhang (zhhuabj) wrote :

Hi Corey, sorry for the late reply because I am just back from my pto today, I will verify them ASAP.

Revision history for this message
Hua Zhang (zhhuabj) wrote :

Successfully verified cosmic-proposed nova 2:18.0.3-0ubuntu1

tags: added: verification-done-cosmic
removed: verification-needed-cosmic
Revision history for this message
Hua Zhang (zhhuabj) wrote :

Successfully verified bionic-proposed nova 2:17.0.7-0ubuntu1

tags: added: verification-done-bionic
removed: verification-needed-bionic
Revision history for this message
Hua Zhang (zhhuabj) wrote :

Successfully verified queens-proposed nova 2:17.0.7-0ubuntu1~cloud0

tags: added: verification-queens-done
removed: verification-queens-needed
Revision history for this message
Hua Zhang (zhhuabj) wrote :

Successfully verified rocky-proposed nova 2:18.0.3-0ubuntu1~cloud0

tags: added: verification-rocky-done
removed: verification-rocky-needed
Revision history for this message
Hua Zhang (zhhuabj) wrote :

Hi Corey, could you pls nominate this fix to xenial as well? we have a customer using xenial-mitaka, I have uploaded a debdiff 'lp1744079_xenial.debdiff', many thanks.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (stable/queens)

Reviewed: https://review.openstack.org/602478
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=1b2c7483003789ecaa0eea8afb49b783934bbf0f
Submitter: Zuul
Branch: stable/queens

commit 1b2c7483003789ecaa0eea8afb49b783934bbf0f
Author: int32bit <email address hidden>
Date: Mon Jan 22 20:15:21 2018 +0800

    Refix disk size during live migration with disk over-commit

    Currently available disk of targer host is calculated based on
    local_gb, not available disk(free_disk_gb). This check can be
    negative if the target host has no free disk.

    Change-Id: Iec50269ef31dfe090f0cd4db95a37909661bd910
    closes-bug: 1744079
    (cherry picked from commit e2cc275063658b23ed88824100919a6dfccb760d)

Changed in nova (Ubuntu Xenial):
importance: Undecided → High
status: New → Triaged
Revision history for this message
Corey Bryant (corey.bryant) wrote :

@Hua, we'll have to fix this in pike and ocata before it can be fixed in mitaka.

Revision history for this message
Brian Murray (brian-murray) wrote : Update Released

The verification of the Stable Release Update for nova has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package nova - 2:18.0.3-0ubuntu1

---------------
nova (2:18.0.3-0ubuntu1) cosmic; urgency=medium

  * d/gbp.conf: Create stable/rocky branch.
  * d/p/disk-size-live-migration-overcommit.patch: Cherry-picked from
    https://review.openstack.org/#/c/602477 to ensure proper disk calculation
    during live migration with over-commit (LP: #1744079).
  * New stable point release for OpenStack Rocky (LP: #1806049).

 -- Corey Bryant <email address hidden> Mon, 03 Dec 2018 12:23:48 -0500

Changed in nova (Ubuntu Cosmic):
status: Fix Committed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package nova - 2:17.0.7-0ubuntu1

---------------
nova (2:17.0.7-0ubuntu1) bionic; urgency=medium

  * d/p/disk-size-live-migration-overcommit.patch: Cherry-picked from
    https://review.openstack.org/#/c/602478 to ensure proper disk calculation
    during live migration with over-commit (LP: #1744079).
  * New stable point release for OpenStack Queens (LP: #1806043).

 -- Corey Bryant <email address hidden> Fri, 07 Dec 2018 08:27:02 -0500

Changed in nova (Ubuntu Bionic):
status: Fix Committed → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/nova 18.1.0

This issue was fixed in the openstack/nova 18.1.0 release.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/nova 17.0.9

This issue was fixed in the openstack/nova 17.0.9 release.

Revision history for this message
James Page (james-page) wrote : Update Released

The verification of the Stable Release Update for nova has completed successfully and the package has now been released to -updates. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

Revision history for this message
James Page (james-page) wrote :

This bug was fixed in the package nova - 2:17.0.7-0ubuntu1~cloud0
---------------

 nova (2:17.0.7-0ubuntu1~cloud0) xenial-queens; urgency=medium
 .
   * New upstream release for the Ubuntu Cloud Archive.
 .
 nova (2:17.0.7-0ubuntu1) bionic; urgency=medium
 .
   * d/p/disk-size-live-migration-overcommit.patch: Cherry-picked from
     https://review.openstack.org/#/c/602478 to ensure proper disk calculation
     during live migration with over-commit (LP: #1744079).
   * New stable point release for OpenStack Queens (LP: #1806043).

Revision history for this message
James Page (james-page) wrote :

The verification of the Stable Release Update for nova has completed successfully and the package has now been released to -updates. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

Revision history for this message
James Page (james-page) wrote :

This bug was fixed in the package nova - 2:18.0.3-0ubuntu1~cloud0
---------------

 nova (2:18.0.3-0ubuntu1~cloud0) bionic-rocky; urgency=medium
 .
   * New update for the Ubuntu Cloud Archive.
 .
 nova (2:18.0.3-0ubuntu1) cosmic; urgency=medium
 .
   * d/gbp.conf: Create stable/rocky branch.
   * d/p/disk-size-live-migration-overcommit.patch: Cherry-picked from
     https://review.openstack.org/#/c/602477 to ensure proper disk calculation
     during live migration with over-commit (LP: #1744079).
   * New stable point release for OpenStack Rocky (LP: #1806049).

Revision history for this message
Corey Bryant (corey.bryant) wrote :

Hua, have you attempted to get the pike and ocata patches backported upstream? If possible we need to get them upstream first for affected releases.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on nova (stable/pike)

Change abandoned by Zhang Hua (<email address hidden>) on branch: stable/pike
Review: https://review.openstack.org/631372

Revision history for this message
Hua Zhang (zhhuabj) wrote :
Dan Streetman (ddstreet)
tags: removed: sts-sponsor sts-sru-needed
tags: added: sts-sru-needed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/nova 19.0.0.0rc1

This issue was fixed in the openstack/nova 19.0.0.0rc1 release candidate.

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (stable/pike)

Reviewed: https://review.openstack.org/631372
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=e485938f5e7b2f7f9c69f97b6c1b58b3de8d68ec
Submitter: Zuul
Branch: stable/pike

commit e485938f5e7b2f7f9c69f97b6c1b58b3de8d68ec
Author: int32bit <email address hidden>
Date: Mon Jan 22 20:15:21 2018 +0800

    Refix disk size during live migration with disk over-commit

    Currently available disk of targer host is calculated based on
    local_gb, not available disk(free_disk_gb). This check can be
    negative if the target host has no free disk.

    Change-Id: Iec50269ef31dfe090f0cd4db95a37909661bd910
    closes-bug: 1744079
    (cherry picked from commit e2cc275063658b23ed88824100919a6dfccb760d)
    Signed-off-by: Zhang Hua <email address hidden>

Revision history for this message
Hua Zhang (zhhuabj) wrote :
Revision history for this message
Corey Bryant (corey.bryant) wrote : Please test proposed package

Hello Matthew, or anyone else affected,

Accepted nova into pike-proposed. The package will build now and be available in the Ubuntu Cloud Archive in a few hours, and then in the -proposed repository.

Please help us by testing this new package. To enable the -proposed repository:

  sudo add-apt-repository cloud-archive:pike-proposed
  sudo apt-get update

Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-pike-needed to verification-pike-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-pike-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

tags: added: verification-pike-needed
Revision history for this message
Corey Bryant (corey.bryant) wrote :

Hello Matthew, or anyone else affected,

Accepted nova into ocata-proposed. The package will build now and be available in the Ubuntu Cloud Archive in a few hours, and then in the -proposed repository.

Please help us by testing this new package. To enable the -proposed repository:

  sudo add-apt-repository cloud-archive:ocata-proposed
  sudo apt-get update

Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-ocata-needed to verification-ocata-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-ocata-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

tags: added: verification-ocata-needed
Revision history for this message
Corey Bryant (corey.bryant) wrote :

A new version of nova has been uploaded to the xenial unapproved queue where it is awaiting SRU team review: https://launchpad.net/ubuntu/xenial/+queue?queue_state=1&queue_text=nova

Revision history for this message
Hua Zhang (zhhuabj) wrote :

Successfully verified pike-proposed nova 2:16.1.7-0ubuntu1~cloud2 and ocata-proposed nova 2:15.1.5-0ubuntu1~cloud3

tags: added: verification-ocata-done verification-pike-done
removed: verification-ocata-needed verification-pike-needed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (stable/ocata)

Reviewed: https://review.openstack.org/631376
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=0b0030ea2273840bd428070b2f23e755a30d2a35
Submitter: Zuul
Branch: stable/ocata

commit 0b0030ea2273840bd428070b2f23e755a30d2a35
Author: int32bit <email address hidden>
Date: Mon Jan 22 20:15:21 2018 +0800

    Refix disk size during live migration with disk over-commit

    Currently available disk of targer host is calculated based on
    local_gb, not available disk(free_disk_gb). This check can be
    negative if the target host has no free disk.

    Change-Id: Iec50269ef31dfe090f0cd4db95a37909661bd910
    closes-bug: 1744079
    (cherry picked from commit e2cc275063658b23ed88824100919a6dfccb760d)
    Signed-off-by: Zhang Hua <email address hidden>

James Page (james-page)
tags: added: verification-done
removed: verification-needed
Revision history for this message
James Page (james-page) wrote : Update Released

The verification of the Stable Release Update for nova has completed successfully and the package has now been released to -updates. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

Revision history for this message
James Page (james-page) wrote :

This bug was fixed in the package nova - 2:15.1.5-0ubuntu1~cloud3
---------------

 nova (2:15.1.5-0ubuntu1~cloud3) xenial-ocata; urgency=medium
 .
   * d/p/(re)fix-disk-size-during-live-migration-with-disk-over-commit.patch:
     Cherry-picked from upstream ocata gerrit reviews to ensure disk size check
     is corectly calculated on the destination host for live migration with
     disk over-commit (LP: #1708572) (LP: #1744079).

Revision history for this message
James Page (james-page) wrote :

The verification of the Stable Release Update for nova has completed successfully and the package has now been released to -updates. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

Revision history for this message
James Page (james-page) wrote :

This bug was fixed in the package nova - 2:16.1.7-0ubuntu1~cloud2
---------------

 nova (2:16.1.7-0ubuntu1~cloud2) xenial-pike; urgency=medium
 .
   * d/p/(re)fix-disk-size-during-live-migration-with-disk-over-commit.patch:
     Cherry-picked from upstream stable/pike branch to ensure disk size check
     is corectly calculated on the destination host for live migration with
     disk over-commit (LP: #1708572) (LP: #1744079).

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/nova 16.1.8

This issue was fixed in the openstack/nova 16.1.8 release.

Revision history for this message
Brian Murray (brian-murray) wrote : Please test proposed package

Hello Matthew, or anyone else affected,

Accepted nova into xenial-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/nova/2:13.1.4-0ubuntu4.4 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation on how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested and change the tag from verification-needed-xenial to verification-done-xenial. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed-xenial. In either case, without details of your testing we will not be able to proceed.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance for helping!

N.B. The updated package will be released to -updates after the bug(s) fixed by this package have been verified and the package has been in -proposed for a minimum of 7 days.

Changed in nova (Ubuntu Xenial):
status: Triaged → Fix Committed
tags: added: verification-needed verification-needed-xenial
removed: verification-done
Revision history for this message
Corey Bryant (corey.bryant) wrote :

Hello Matthew, or anyone else affected,

Accepted nova into mitaka-proposed. The package will build now and be available in the Ubuntu Cloud Archive in a few hours, and then in the -proposed repository.

Please help us by testing this new package. To enable the -proposed repository:

  sudo add-apt-repository cloud-archive:mitaka-proposed
  sudo apt-get update

Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-mitaka-needed to verification-mitaka-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-mitaka-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

tags: added: verification-mitaka-needed
Revision history for this message
Hua Zhang (zhhuabj) wrote :

Successfully verified xenial-proposed nova 2:13.1.4-0ubuntu4.4

tags: added: verification-done-xenial
removed: verification-needed-xenial
Revision history for this message
Hua Zhang (zhhuabj) wrote :

Successfully verified mitaka-proposed nova 2:13.1.4-0ubuntu4.4~cloud0

tags: added: sts-sru-done verification-mitaka-done
removed: sts-sru-needed verification-mitaka-needed
tags: added: sts-sru-needed verification-done
removed: sts-sru-done verification-needed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package nova - 2:13.1.4-0ubuntu4.4

---------------
nova (2:13.1.4-0ubuntu4.4) xenial; urgency=medium

  * Refix disk size during live migration with disk over-commit
    - (LP: #1708572) and (LP: #1744079)
    - d/p/0001-Fix-disk-size-during-live-migration-with-disk-over-c.patch
    - d/p/0002-Refix-disk-size-during-live-migration-with-disk-over.patch

 -- Zhang Hua <email address hidden> Tue, 02 Apr 2019 18:48:16 +0800

Changed in nova (Ubuntu Xenial):
status: Fix Committed → Fix Released
Revision history for this message
Corey Bryant (corey.bryant) wrote : Update Released

The verification of the Stable Release Update for nova has completed successfully and the package has now been released to -updates. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regressions.

Revision history for this message
Corey Bryant (corey.bryant) wrote :

This bug was fixed in the package nova - 2:13.1.4-0ubuntu4.4~cloud0
---------------

 nova (2:13.1.4-0ubuntu4.4~cloud0) trusty-mitaka; urgency=medium
 .
   * New update for the Ubuntu Cloud Archive.
 .
 nova (2:13.1.4-0ubuntu4.4) xenial; urgency=medium
 .
   * Refix disk size during live migration with disk over-commit
     - (LP: #1708572) and (LP: #1744079)
     - d/p/0001-Fix-disk-size-during-live-migration-with-disk-over-c.patch
     - d/p/0002-Refix-disk-size-during-live-migration-with-disk-over.patch

Changed in cloud-archive:
status: Fix Committed → Fix Released
tags: added: sts-sru-done
removed: sts-sru-needed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/nova ocata-eol

This issue was fixed in the openstack/nova ocata-eol release.

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.