create instance min/max count defaulting logic is reversed in EC2 and native OS APIs

Bug #989799 reported by Eoghan Glynn
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Wishlist
Iccha Sethi

Bug Description

EC2 API: default max to 1, default min to max

OS API: default min to 1, default max to min

It would seem more logical to keep to these two defaulting strategies consistent.

Eoghan Glynn (eglynn)
Changed in nova:
assignee: nobody → Eoghan Glynn (eglynn)
Revision history for this message
Sean Dague (sdague) wrote :

Any progress so far?

Changed in nova:
status: New → Confirmed
Sean Dague (sdague)
tags: added: low-hanging-fruit
Thierry Carrez (ttx)
Changed in nova:
importance: Undecided → Wishlist
Revision history for this message
Bhuvan Arumugam (bhuvan) wrote :

Shall we go with following defaults, for both API?
     default min to 1, default max to min

Changed in nova:
assignee: Eoghan Glynn (eglynn) → Iccha Sethi (iccha-sethi)
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/10618

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

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

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

Reviewed: https://review.openstack.org/10620
Committed: http://github.com/openstack/nova/commit/045b084d34c08aee910cee2362cf209fd80d3c6d
Submitter: Jenkins
Branch: master

commit 045b084d34c08aee910cee2362cf209fd80d3c6d
Author: isethi <email address hidden>
Date: Tue Jul 31 19:52:49 2012 +0000

    Changes default behavior of ec2

    Changes the default behaviour of create instance min/max count logic
    in ec2 to match the default behavior of OS API

    Fixes bug 989799

    Change-Id: I1aa42c7d47674ce68442367fab487152b3e7fa67

Changed in nova:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in nova:
milestone: none → folsom-3
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in nova:
milestone: folsom-3 → 2012.2
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.