ambari-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jaimin Jetly" <jaimin.je...@gmail.com>
Subject Review Request 20116: Stack service component data should be dynamically fetched from server.
Date Tue, 08 Apr 2014 02:19:24 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20116/
-----------------------------------------------------------

Review request for Ambari and Yusaku Sako.


Bugs: AMBARI-5389
    https://issues.apache.org/jira/browse/AMBARI-5389


Repository: ambari


Description
-------

Web ui maintains a copy of service components supported by stack at ambari-web/app/data/service_components.js.
Whenever a new service component is added in a stack, this copy needs to be updated. 
As a part of resolution ambari-web should not rely on this file but get the service component
data from API.


Diffs
-----

  ambari-web/app/app.js e83319e 
  ambari-web/app/assets/data/stacks/HDP-2.1/service_components.json PRE-CREATION 
  ambari-web/app/controllers/global/cluster_controller.js 7a1304f 
  ambari-web/app/controllers/installer.js 4d8e21e 
  ambari-web/app/controllers/main/host/add_controller.js 179c92a 
  ambari-web/app/controllers/main/service/add_controller.js 8b1ce05 
  ambari-web/app/controllers/main/service/item.js f78a57b 
  ambari-web/app/controllers/main/service/manage_config_groups_controller.js bcd7e15 
  ambari-web/app/controllers/wizard.js 4affb3b 
  ambari-web/app/controllers/wizard/step5_controller.js 40c1f99 
  ambari-web/app/controllers/wizard/step6_controller.js b7f4639 
  ambari-web/app/controllers/wizard/step7_controller.js b7a9cfd 
  ambari-web/app/controllers/wizard/step8_controller.js 9958dda 
  ambari-web/app/controllers/wizard/step9_controller.js 5a133ed 
  ambari-web/app/data/HDP2/global_properties.js a3eedaa 
  ambari-web/app/data/service_components.js f066c46 
  ambari-web/app/initialize.js 777779a 
  ambari-web/app/mappers.js PRE-CREATION 
  ambari-web/app/mappers/stack_service_component_mapper.js PRE-CREATION 
  ambari-web/app/models.js ce53c73 
  ambari-web/app/models/stack_service_component.js PRE-CREATION 
  ambari-web/app/utils/ajax.js 4f25d93 
  ambari-web/app/utils/component.js dec45a7 
  ambari-web/app/views/main/host/summary.js d367e89 
  ambari-web/test/app_test.js 4830b58 
  ambari-web/test/installer/step5_test.js 57cef17 
  ambari-web/test/installer/step6_test.js e323f2d 
  ambari-web/test/installer/step9_test.js 4ecd864 

Diff: https://reviews.apache.org/r/20116/diff/


Testing
-------

tested e2e and fixed broken unit tests impacted by the patch.


Thanks,

Jaimin Jetly


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message