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 -- Mailing list: https://launchpad.net/~yahoo-eng-team Post to : yahoo-eng-team@lists.launchpad.net Unsubscribe : https://launchpad.net/~yahoo-eng-team More help : https://help.launchpad.net/ListHelp
