yahoo-eng-team team mailing list archive

[Travis Tripp <1450150@xxxxxxxxxxxxxxxxxx>]

Public bug reported:

A nova-extensions and settings-service services and directives were
implemented at the very end of Kilo. The directives created for needsto
be able to be attribute based and investigated for how it might be
combined with other conditional behavior based directives.

The Nova Extension check needs to also make use of the Service Catalog
(check if service is even enabled before checking extensions). This
should include that works as well.

See here:
https://review.openstack.org/#/c/164359/12/horizon/static/horizon/js/angular/services/hz.api.keystone.js

See line 12 here for example of directive using extensions that we want
to enhance using the service catalog.

https://review.openstack.org/#/c/166708/22/openstack_dashboard/static/dashboard
/launch-instance/configuration/configuration.html

See line 110 here:
https://review.openstack.org/#/c/171418/13/openstack_dashboard/static/dashboard/launch-instance/source/source.html

See also:
https://review.openstack.org/#/c/170351/

This all introduced a new cloud services.js which created some unwanted extra dependency injection. This can be refactored to its own new utility.
Support attributes vs elements.
Review naming.
Consider using lazy loading via the compile function
https://www.youtube.com/watch?v=UMkd0nYmLzY&t=24m30s

** Affects: horizon
     Importance: Undecided
     Assignee: Travis Tripp (travis-tripp)
         Status: New

** Changed in: horizon
     Assignee: (unassigned) => Travis Tripp (travis-tripp)

-- 
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/1450150

Title:
  Refactor angular cloud services utilities

Status in OpenStack Dashboard (Horizon):
  New

Bug description:
  A nova-extensions and settings-service services and directives were
  implemented at the very end of Kilo. The directives created for
  needsto be able to be attribute based and investigated for how it
  might be combined with other conditional behavior based directives.

  The Nova Extension check needs to also make use of the Service Catalog
  (check if service is even enabled before checking extensions). This
  should include that works as well.

  See here:
  https://review.openstack.org/#/c/164359/12/horizon/static/horizon/js/angular/services/hz.api.keystone.js

  See line 12 here for example of directive using extensions that we
  want to enhance using the service catalog.

  https://review.openstack.org/#/c/166708/22/openstack_dashboard/static/dashboard
  /launch-instance/configuration/configuration.html

  See line 110 here:
  https://review.openstack.org/#/c/171418/13/openstack_dashboard/static/dashboard/launch-instance/source/source.html

  See also:
  https://review.openstack.org/#/c/170351/

  This all introduced a new cloud services.js which created some unwanted extra dependency injection. This can be refactored to its own new utility.
  Support attributes vs elements.
  Review naming.
  Consider using lazy loading via the compile function
  https://www.youtube.com/watch?v=UMkd0nYmLzY&t=24m30s

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