> On Dec. 14, 2015, 8:22 p.m., Jaimin Jetly wrote: > > ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/quicklinks/quicklinks.json, > > lines 15-16 > > <https://reviews.apache.org/r/41047/diff/3/?file=1160343#file1160343line15> > > > > What is the difference between url and template attribute ? When can > > they have different values ? > > Di Li wrote: > hello Jaimin, > > This is the design carried forward from the hardcoded App.QuickLinks > model. The template field is used by the UI to format the actual external > url. UI set the result as the value of the url field, the url field is used > by the UI to create the external URLs. > > Thank you. > > Di
Hi Di, Thanks for the explanation. I feel we can simplify the legacy design by removing one of the field as it seems redundant. This will require some chnage quick_link_views.js. We can track that as a seperate task on Apache Ambari Jira. - Jaimin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41047/#review110299 ----------------------------------------------------------- On Dec. 15, 2015, 5:29 p.m., Di Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/41047/ > ----------------------------------------------------------- > > (Updated Dec. 15, 2015, 5:29 p.m.) > > > Review request for Ambari, Alejandro Fernandez, Jaimin Jetly, Richard Zang, > and Yusaku Sako. > > > Bugs: AMBARI-11268 > https://issues.apache.org/jira/browse/AMBARI-11268 > > > Repository: ambari > > > Description > ------- > > The existing version of the Ambari (2.0) does not have a declarative way to > add "Quick Links" for the custom services. Currently, to add a "Quick Links" > dropdown one has to do add quick links definitions to the UI code, making it > very difficult for custom services to have quick links. > > The proposed changes here introduced a way to allow services to > define/inherite/update quick links via quicklinks.json > > > Diffs > ----- > > > ambari-server/src/main/java/org/apache/ambari/server/api/resources/ResourceInstanceFactoryImpl.java > 27609e7 > > ambari-server/src/main/java/org/apache/ambari/server/api/resources/StackServiceResourceDefinition.java > 45302ff > > ambari-server/src/main/java/org/apache/ambari/server/api/services/AmbariMetaInfo.java > e35e7ac > > ambari-server/src/main/java/org/apache/ambari/server/api/services/StacksService.java > 76397fb > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/AbstractControllerResourceProvider.java > d47d8d3 > > ambari-server/src/main/java/org/apache/ambari/server/controller/internal/QuickLinkArtifactResourceProvider.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/controller/spi/Resource.java > 55816a3 > > ambari-server/src/main/java/org/apache/ambari/server/stack/QuickLinksConfigurationModule.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/stack/ServiceModule.java > c2a2a0c > > ambari-server/src/main/java/org/apache/ambari/server/state/QuickLinksConfigurationInfo.java > PRE-CREATION > ambari-server/src/main/java/org/apache/ambari/server/state/ServiceInfo.java > a58cda3 > > ambari-server/src/main/java/org/apache/ambari/server/state/quicklinks/Check.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/state/quicklinks/Link.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/state/quicklinks/Port.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/state/quicklinks/Protocol.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/state/quicklinks/QuickLinks.java > PRE-CREATION > > ambari-server/src/main/java/org/apache/ambari/server/state/quicklinks/QuickLinksConfiguration.java > PRE-CREATION > > ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/metainfo.xml > ae58b5c > > ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/quicklinks/quicklinks.json > PRE-CREATION > > ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/metainfo.xml > 515f385 > > ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/quicklinks/quicklinks.json > PRE-CREATION > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/OOZIE/metainfo.xml > dad6dc1 > > ambari-server/src/main/resources/stacks/HDP/2.0.6/services/OOZIE/quicklinks/quicklinks.json > PRE-CREATION > > ambari-server/src/main/resources/stacks/HDP/2.2/services/RANGER/metainfo.xml > 4d1b305 > > ambari-server/src/main/resources/stacks/HDP/2.2/services/RANGER/quicklinks/quicklinks.json > PRE-CREATION > ambari-server/src/main/resources/stacks/HDP/2.2/services/SPARK/metainfo.xml > db11d05 > > ambari-server/src/main/resources/stacks/HDP/2.2/services/SPARK/quicklinks/quicklinks.json > PRE-CREATION > > ambari-server/src/main/resources/stacks/HDP/2.3/services/ACCUMULO/metainfo.xml > 352341b > > ambari-server/src/main/resources/stacks/HDP/2.3/services/ACCUMULO/quicklinks/quicklinks.json > PRE-CREATION > ambari-server/src/main/resources/stacks/HDP/2.3/services/ATLAS/metainfo.xml > fee10d3 > > ambari-server/src/main/resources/stacks/HDP/2.3/services/ATLAS/quicklinks/quicklinks.json > PRE-CREATION > ambari-server/src/main/resources/stacks/HDP/2.3/services/HBASE/metainfo.xml > 90a31f5 > > ambari-server/src/main/resources/stacks/HDP/2.3/services/HBASE/quicklinks/quicklinks.json > PRE-CREATION > ambari-server/src/main/resources/stacks/HDP/2.3/services/HDFS/metainfo.xml > ca69195 > > ambari-server/src/main/resources/stacks/HDP/2.3/services/HDFS/quicklinks/quicklinks.json > PRE-CREATION > ambari-server/src/main/resources/stacks/HDP/2.3/services/OOZIE/metainfo.xml > ccf5de3 > > ambari-server/src/main/resources/stacks/HDP/2.3/services/OOZIE/quicklinks/quicklinks.json > PRE-CREATION > > ambari-server/src/main/resources/stacks/HDP/2.3/services/RANGER/metainfo.xml > 2fb8a9a > > ambari-server/src/main/resources/stacks/HDP/2.3/services/RANGER/quicklinks/quicklinks.json > PRE-CREATION > ambari-server/src/main/resources/stacks/HDP/2.3/services/SPARK/metainfo.xml > 2c9680c > > ambari-server/src/main/resources/stacks/HDP/2.3/services/SPARK/quicklinks/quicklinks.json > PRE-CREATION > ambari-server/src/main/resources/stacks/HDP/2.3/services/YARN/metainfo.xml > 5a21200 > > ambari-server/src/main/resources/stacks/HDP/2.3/services/YARN/quicklinks-mapred/quicklinks.json > PRE-CREATION > > ambari-server/src/main/resources/stacks/HDP/2.3/services/YARN/quicklinks/quicklinks.json > PRE-CREATION > > ambari-server/src/test/java/org/apache/ambari/server/api/resources/StackServiceResourceDefinitionTest.java > 65f54d5 > > ambari-server/src/test/java/org/apache/ambari/server/stack/QuickLinksConfigurationModuleTest.java > PRE-CREATION > ambari-server/src/test/resources/child_quicklinks_to_inherit.json > PRE-CREATION > ambari-server/src/test/resources/child_quicklinks_to_merge.json > PRE-CREATION > ambari-server/src/test/resources/child_quicklinks_to_override.json > PRE-CREATION > ambari-server/src/test/resources/parent_quicklinks.json PRE-CREATION > ambari-web/app/assets/data/configurations/quicklinks.json PRE-CREATION > ambari-web/app/assets/data/configurations/quicklinks_services.json > PRE-CREATION > ambari-web/app/mappers.js 312128b > ambari-web/app/mappers/quicklinks_mapper.js PRE-CREATION > ambari-web/app/models.js e1bceec > ambari-web/app/models/quicklinks/quick_links_config.js PRE-CREATION > ambari-web/app/templates/main/service/item.hbs ed989a1 > ambari-web/app/utils/ajax/ajax.js 71ee9db > ambari-web/app/views/common/quick_view_link_view.js e2e8d46 > ambari-web/test/views/common/quick_link_view_test.js fc6b118 > > Diff: https://reviews.apache.org/r/41047/diff/ > > > Testing > ------- > > unit test > patch Ambari 2.2.0 cluster with server, stack, UI changes and see if the > quick links are populated and can open external sites when clicked. > > > Thanks, > > Di Li > >
