yahoo-eng-team team mailing list archive

[Akihiro Motoki <1348075@xxxxxxxxxxxxxxxxxx>]

Public bug reported:

In api/neutron.py, is_xxxxx_exntension_supproted() is defined per feature, but this style doesn't scale.
is_extension_supported() is memoized and if there is no special reason is_extension_supported() should be used directly.

is_quotas_extension_supported() is an exception because it has extra
logic.

We already have rough consensus on the direction.
http://eavesdrop.openstack.org/meetings/horizon/2014/horizon.2014-07-22-16.00.log.html#l-140

** Affects: horizon
     Importance: Medium
     Assignee: Akihiro Motoki (amotoki)
         Status: New

** Description changed:

  In api/neutron.py, is_xxxxx_exntension_supproted() is defined per feature, but this style doesn't scale.
  is_extension_supported() is memoized and if there is no special reason is_extension_supported() should be used directly.
  
  is_quotas_extension_supported() is an exception because it has extra
  logic.
+ 
+ We already have rough consensus on the direction.
+ http://eavesdrop.openstack.org/meetings/horizon/2014/horizon.2014-07-22-16.00.log.html#l-140

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

Title:
  per-feature extension method in api/neutron.py should be removed

Status in OpenStack Dashboard (Horizon):
  New

Bug description:
  In api/neutron.py, is_xxxxx_exntension_supproted() is defined per feature, but this style doesn't scale.
  is_extension_supported() is memoized and if there is no special reason is_extension_supported() should be used directly.

  is_quotas_extension_supported() is an exception because it has extra
  logic.

  We already have rough consensus on the direction.
  http://eavesdrop.openstack.org/meetings/horizon/2014/horizon.2014-07-22-16.00.log.html#l-140

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