← Back to team overview

yahoo-eng-team team mailing list archive

[Bug 1585914] [NEW] should not use 'in' to tell if the boot request is in aggregate azs string

 

Public bug reported:

code at
https://github.com/openstack/nova/blob/master/nova/scheduler/filters/availability_zone_filter.py#L48
and
https://github.com/openstack/nova/blob/master/nova/tests/unit/scheduler/filters/test_availability_zone_filters.py#L40

shows we can support a list of AZs which is comma separated in one
aggregate.

but the azs is a string in metadata of aggregate, which
https://github.com/openstack/nova/blob/master/nova/scheduler/filters/availability_zone_filter.py#L48
is using 'in' operator.

solution is to:

form a list of azs from aggregate's metadata, and then use 'in'
operator.

** Affects: nova
     Importance: Undecided
     Assignee: yong sheng gong (gongysh)
         Status: New

** Summary changed:

-  should not use 'in' to tell if the boot request in aggregate azs string
+ should not use 'in' to tell if the boot request is in aggregate azs string

** Changed in: nova
     Assignee: (unassigned) => yong sheng gong (gongysh)

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Compute (nova).
https://bugs.launchpad.net/bugs/1585914

Title:
  should not use 'in' to tell if the boot request is in aggregate azs
  string

Status in OpenStack Compute (nova):
  New

Bug description:
  code at
  https://github.com/openstack/nova/blob/master/nova/scheduler/filters/availability_zone_filter.py#L48
  and
  https://github.com/openstack/nova/blob/master/nova/tests/unit/scheduler/filters/test_availability_zone_filters.py#L40

  shows we can support a list of AZs which is comma separated in one
  aggregate.

  but the azs is a string in metadata of aggregate, which
  https://github.com/openstack/nova/blob/master/nova/scheduler/filters/availability_zone_filter.py#L48
  is using 'in' operator.

  solution is to:

  form a list of azs from aggregate's metadata, and then use 'in'
  operator.

To manage notifications about this bug go to:
https://bugs.launchpad.net/nova/+bug/1585914/+subscriptions


Follow ups