Merge lp:~rackspace-titan/nova/minram-mindisk into lp:~hudson-openstack/nova/trunk
Proposed by
Naveed Massjouni
Status: | Rejected | ||||
---|---|---|---|---|---|
Rejected by: | Brian Waldon | ||||
Proposed branch: | lp:~rackspace-titan/nova/minram-mindisk | ||||
Merge into: | lp:~hudson-openstack/nova/trunk | ||||
Diff against target: |
557 lines (+327/-1) 10 files modified
nova/api/openstack/create_instance_helper.py (+4/-0) nova/api/openstack/images.py (+6/-0) nova/api/openstack/schemas/v1.1/image.rng (+6/-0) nova/api/openstack/views/images.py (+5/-0) nova/compute/api.py (+5/-0) nova/exception.py (+8/-0) nova/image/glance.py (+1/-1) nova/tests/api/openstack/test_images.py (+189/-0) nova/tests/image/test_glance.py (+12/-0) nova/tests/test_compute.py (+91/-0) |
||||
To merge this branch: | bzr merge lp:~rackspace-titan/nova/minram-mindisk | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Dan Prince (community) | Approve | ||
Chris Behrens (community) | Approve | ||
Brian Waldon (community) | Approve | ||
Review via email: mp+76491@code.launchpad.net |
Description of the change
This addresses Bug #819990 Add minDisk minRam to OSAPI image details. It also enforces the minram and mindisk constraints if they exist. Corresponding tests have been added.
To post a comment you must log in.
Unmerged revisions
- 1585. By Alex Meade
-
Added the deleting of servers that were created in tests
- 1584. By Alex Meade
-
fixed incorrect exception names
- 1583. By Naveed Massjouni
-
Changing FlavorMemoryToo
Small/FlavorDis kTooSmall exceptions to
InstanceTypeMemoryTooSmall/ InstanceTypeDis kTooSmall. - 1582. By Naveed Massjouni
-
pep8
- 1581. By Naveed Massjouni
-
More tests.
- 1580. By Naveed Massjouni
-
Adding tests for minram/mindisk.
- 1579. By Naveed Massjouni
-
Honor the min_ram and min_disk attributes of images when creating a server
- 1578. By Naveed Massjouni
-
removing print
- 1577. By Naveed Massjouni
-
Merge from trunk
- 1576. By Alex Meade
-
Added support for filtering images with minRam and minDisk
Can you change the new exception names to InstanceTypeDis kTooSmall and InstanceTypeMem oryTooSmall? I think keeping the term 'flavor' limited to the OSAPI view is a good thing to do here.
It looks like we only test passing image filters through to the image service, not that the image service actually does something with them. If you see a way we can test that, I would love to see it :) It's not a blocker for my Approve, though.