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

Review request for Ambari, Jaimin Jetly and Oleg Nechiporenko.


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


Repository: ambari


Description
-------

Currently, there is a notion of tab, section and subsection in UI which is 
completely theme driven from stack API endpoint.
subsection is the last container that contains set of configs. This is achieved 
by registering each config with a subsection in service theme.
As part of this ticket add a notion of subsection-tabs (a group of tabs) which 
can be optionally present inside a subsection. Any config can register only to 
a subsection as it does currently or can register to a subsection and to an 
individual tab of a subsection-tab located inside the subsection.
For now, Layout of a subsection-tabs (a group of tabs) in a subsection is 
always at last. This means that if there are configs that are not mapped to any 
individual tab of subsection-tabs then they will be placed before 
subsection-tabs. Orientation of configs inside subsection-tabs should be in 
order of their occurrence in theme.json.


Diffs
-----

  
ambari-server/src/main/java/org/apache/ambari/server/state/theme/ConfigPlacement.java
 56d2ea282e14c78c47badaaa0672799ffd739231 
  
ambari-server/src/main/java/org/apache/ambari/server/state/theme/Subsection.java
 039754514d0e39c287790527fb034a164f500c43 
  
ambari-server/src/main/resources/stacks/HDP/2.3/services/RANGER/configuration/ranger-admin-site.xml
 7bd7bb3a2d61c72bd38978dc668461781c694ab7 
  
ambari-server/src/main/resources/stacks/HDP/2.3/services/RANGER/configuration/ranger-env.xml
 7f3e6e062b3b66c950d64eafe191cb7421cf6602 
  
ambari-server/src/main/resources/stacks/HDP/2.3/services/RANGER/configuration/ranger-ugsync-site.xml
 de63dd678cdab8aa2bcda3033888f4070fcc0bbb 
  
ambari-server/src/main/resources/stacks/HDP/2.3/services/RANGER/themes/theme.json
 7160a4f691dda0a6ad79d5e57c055de4f1090822 
  ambari-web/app/controllers/installer.js 
194d86224af084631ad820258221ed9d30032c83 
  ambari-web/app/data/HDP2.3/site_properties.js 
8041bc606e49c7eed5ab40ff5a1c5e182188a9cc 
  ambari-web/app/mappers/configs/themes_mapper.js 
b55a695dd32ffffc1a08905dace6b41c7a19435b 
  ambari-web/app/models.js 1b34b0c4b88733e9aec8a53dbab14f7980615211 
  ambari-web/app/models/configs/section.js 
c04665e84b4d13878942e8a92dd21dc201f20516 
  ambari-web/app/models/configs/stack_config_property.js 
76e3b5f6ce9bc35df27e248a98e88febce8457cf 
  ambari-web/app/models/configs/sub_section.js 
b7abb4fad96045ce840daf2341a808191915356a 
  ambari-web/app/models/configs/tab.js 0940022c94f361077deb00e31da532f099f9664f 
  ambari-web/app/models/configs/theme/section.js PRE-CREATION 
  ambari-web/app/models/configs/theme/sub_section.js PRE-CREATION 
  ambari-web/app/models/configs/theme/sub_section_tab.js PRE-CREATION 
  ambari-web/app/models/configs/theme/tab.js PRE-CREATION 
  ambari-web/app/templates/common/configs/service_config_layout_tab.hbs 
69de31521eb43fbd50fc78cb6a8832adca6ad64d 
  ambari-web/app/views/common/configs/service_config_layout_tab_view.js 
6815a0ddc0450a1f2c5d543a71cd1538bc820f9e 
  ambari-web/test/mappers/configs/themes_mapper_test.js 
99456e0bb7ae4f8fefd411342a4986ba1dca57e0 

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


Testing
-------

9083 tests complete (11 seconds)
95 tests pending


Thanks,

Andriy Babiichuk

Reply via email to