Unable to boot from volume when flavor disk too small

it's confirmed.But I think we have to discuss whether it's a bug.
For example:
create a 2G bootable volume and the image is large than 1G(maybe 1.5G)
then the error "flavor disk too small" is ok.

Hm, why is it okay in that case? The admin documentation I linked above indicates that the flavor root disk limit does not apply to volumes. It seems like it is fine if the flavor has root disk 1 GB but you want to boot a larger image from it.

Also I'm not certain whether it is comparing the image size against the flavor root disk limit, or comparing the volume size. If it is comparing the volume size then I think this is an even larger problem. But if it's image size it's still bad, since what if you have a 20 GB snapshot that you want to provision volume-backed instance from on a small flavor?

You are right. There is no need to check the disk in flavor if boot from volume.

But in nova code, it is a common function :
https://github.com/openstack/nova/blob/master/nova/compute/api.py#L651

If boot from image,we must check it. So there is two ways to solve this problem.One is to check whether is boot from volume before _check_requested_image().The other one is to check it in _check_requested_image() to avoid it.

I have submit a patch about this problem with the second way.You can test whether it's solved or not :)

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

I looked at the patch, it seems like it still compares the size of the image that volume was created from with the flavor root disk size. I can understand why this check is needed for the min_ram / flavor memory, but why is it needed for the disk?

I'm afraid of situation where user creates a large snapshot, creates a volume from it, and then is unable to boot it from a flavor that has small memory / disk parameters.

My patch just avoid raising FlavorDiskTooSmall error in some ways.

So,the best way is that we don't have to check disk size with flavor if booting from volume.

like this:
        if root_gb and not boot_from_volume: # add a judgment condition
            if int(image.get('size') or 0) > root_gb * (1024 ** 3):
                raise exception.FlavorDiskTooSmall()
            if int(image.get('min_disk') or 0) > root_gb:
                    raise exception.FlavorDiskTooSmall()

is it looks ok?

Change abandoned by wangxiyuan (<email address hidden>) on branch: master
Review: https://review.openstack.org/185322
Reason: I think We should have a discuess about how to check bdm in api side.

First of all,maybe we should fix RT's bug :

https://bugs.launchpad.net/nova/+bug/1334974

This bug is similar to the one below :

https://bugs.launchpad.net/nova/+bug/1334974

and it's difficulte to reach an agreement about how to solve it.

so I hava abandoned the patch.You can follow it if you are interested .

Solving an inconsistency: The bug is "in progress" but without an assignee.
I set the status back to "new" because of comment #8 (possible duplicate?).

Feel free to assign the bug to yourself. If you do so, please set it to
"in progress".

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

Reviewed: https://review.openstack.org/204635
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=642c986f0636d52a9ba279c87e25082b4aa9b3b8
Submitter: Jenkins
Branch: master

commit 642c986f0636d52a9ba279c87e25082b4aa9b3b8
Author: Matthew Booth <email address hidden>
Date: Wed Jul 22 14:56:52 2015 +0100

    Don't check flavor disk size when booting from volume

    When creating a volume from an image, cinder copies the image metadata
    into volume properties. When booting from the volume, we read this
    metadata from the volume and use it as image metadata once again.

    While fixing the check against min_ram,
    change I861a78b5c7efa71e4bf7206d388b8d0d8048c78e introduced a
    regression which prevents a user from booting a volume which is larger
    than the flavor's disk. As we are not creating this disk, this check
    does not make sense. Similarly, it checks the image metadata's
    min_disk against the flavor disk size, which is not being used.

    This change leaves the image metadata check unaltered when creating a
    flavor disk. When booting from a volume, we check min_disk from image
    metadata against the actual size of the volume. We don't check the
    volume size at all. The check against min_ram is retained unaltered.

    Closes-Bug: #1457517
    Closes-Bug: #1459491
    Closes-Bug: #1466305
    Change-Id: I264493172da20b664df571e32876030246c2a87c

Change abandoned by Feodor Tersin (<email address hidden>) on branch: master
Review: https://review.openstack.org/196569

Fix proposed to branch: stable/kilo
Review: https://review.openstack.org/213308

Status changed to 'Confirmed' because the bug affects multiple users.

Reviewed: https://review.openstack.org/213308
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=8794b938dcb983b7c918718807c2396cb255b4ce
Submitter: Jenkins
Branch: stable/kilo

commit 8794b938dcb983b7c918718807c2396cb255b4ce
Author: Matthew Booth <email address hidden>
Date: Wed Jul 22 14:56:52 2015 +0100

    Don't check flavor disk size when booting from volume

    When creating a volume from an image, cinder copies the image metadata
    into volume properties. When booting from the volume, we read this
    metadata from the volume and use it as image metadata once again.

    While fixing the check against min_ram,
    change I861a78b5c7efa71e4bf7206d388b8d0d8048c78e introduced a
    regression which prevents a user from booting a volume which is larger
    than the flavor's disk. As we are not creating this disk, this check
    does not make sense. Similarly, it checks the image metadata's
    min_disk against the flavor disk size, which is not being used.

    This change leaves the image metadata check unaltered when creating a
    flavor disk. When booting from a volume, we check min_disk from image
    metadata against the actual size of the volume. We don't check the
    volume size at all. The check against min_ram is retained unaltered.

    Closes-Bug: #1457517
    Closes-Bug: #1459491
    Closes-Bug: #1466305
    Change-Id: I264493172da20b664df571e32876030246c2a87c
    (cherry picked from commit 642c986f0636d52a9ba279c87e25082b4aa9b3b8)

Adding debdiff, which is also available in the ubuntu vivid upload queue (merged into lp:~ubuntu-server-dev/nova/vivid)

This fix is in the current wily package, so marking fix released there.

Pushed the debdiff from comment #16 to New sru queue - thanks.

Hello Favyen, or anyone else affected,

Accepted nova into vivid-proposed. The package will build now and be available at https://launchpad.net/ubuntu/+source/nova/1:2015.1.1-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 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 to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-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!

I was able to verify this bug fix today by the following:

1. Setup a vivid-kilo openstack cloud
2. Create a 5G volume from a cirros image
3. Boot an m1.tiny instance using the 5G volume created as the boot source,
   --> verify command is rejected due to size issue
4. Log into nova-cloud-controller and upgrade nova-* packages to version 1:2015.1.1-0ubuntu2 from the proposed pocket.
5. Boot an m1.tiny instance using the %G volume created as the boot source
  --> verify command is accepted and instance boots

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.

This bug was fixed in the package nova - 1:2015.1.1-0ubuntu2

---------------
nova (1:2015.1.1-0ubuntu2) vivid; urgency=medium

  [ Corey Bryant ]
  * d/rules: Prevent dh_python2 from guessing dependencies.

  [ Liang Chen ]
  * d/p/not-check-disk-size.patch: Fix booting from volume error
    when flavor disk too small (LP: #1457517)

 -- Corey Bryant <email address hidden> Thu, 13 Aug 2015 15:13:43 -0400