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