AggregateTypeAffinityFilter cannot filter on multiple instance_type values

Bug #1399204 reported by sean mooney
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Wishlist
Joe Gordon

Bug Description

AggregateTypeAffinityFilter limits instance_type by aggregate

At present it is not possible to specify multiple instance_types for an Aggregate with this filter.

This prevent operators form creating a single host aggregate for a group of related flavors.
For example a host aggregate for all flavors that support hugepages or a host aggregate for all flavors that support
ssds.

Without this functionality the operator would have to create one host aggregate per flavor/instance_type and add the same host to multiple aggregates.

Changed in nova:
assignee: nobody → sean mooney (sean-k-mooney)
Joe Gordon (jogo)
Changed in nova:
status: New → Confirmed
importance: Undecided → Wishlist
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/141883

Changed in nova:
status: Confirmed → In Progress
Revision history for this message
sean mooney (sean-k-mooney) wrote :

hi

i would like to ask people to review the proposed fix if they have time.

i believe the patch set it is now ready to merge and hope it can be integrated before kilo 2.

regards
sean

Changed in nova:
assignee: sean mooney (sean-k-mooney) → Joe Gordon (jogo)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.openstack.org/141883
Committed: https://git.openstack.org/cgit/openstack/nova/commit/?id=66e1427f14dee477f27cd899a25b9e3cdeb38ff4
Submitter: Jenkins
Branch: master

commit 66e1427f14dee477f27cd899a25b9e3cdeb38ff4
Author: Sean Mooney <email address hidden>
Date: Mon Jan 12 16:57:01 2015 +0000

    Add AggregateTypeAffinityFilter multi values support

    This change allows the AggregateTypeAffinityFilter to function when
    multiple instance_type names are set in the Aggregate Metadata.
    This change implements and documents a new comma separated syntax for
    the aggregate instance_type metadata attribute. The legacy syntax is
    still supported when a single instace_type is specified.
    e.g. 'm1.nano' or "m1.nano,m1.small"

    DocImpact
    Change-Id: I0618a300754d012db62df52faa12cc3cedfe2b65
    Closes-bug: #1399204

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