----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63749/ -----------------------------------------------------------
Review request for Ambari, Andrii Babiichuk, Andrii Tkach, Balázs Bence Sári, and Nate Cole. Bugs: AMBARI-22424 https://issues.apache.org/jira/browse/AMBARI-22424 Repository: ambari Description ------- OneFS mpack should have a quicklink definition with links that point to the OneFS admin UI. OneFS is a client only service and the hostname of OneFS admin page is an external host. The quicklink engine needed to be improved to allow overriding hostnames. For example the following quicklink will point to http://scisilon.fqdn if core-site/fs.defaultFS=hdfs://scisilon.fqdn. "links": [ { "component_name" : "ONEFS_CLIENT", "name": "onefs_web_ui", "label": "OneFS Web UI", "url": "%@://%@", "host": { "http_property": "fs.defaultFS", "site": "core-site" } } ] } Diffs ----- ambari-server/src/main/java/org/apache/ambari/server/state/quicklinks/Host.java PRE-CREATION ambari-server/src/main/java/org/apache/ambari/server/state/quicklinks/Link.java 1d2e712 ambari-server/src/test/java/org/apache/ambari/server/stack/QuickLinksConfigurationModuleTest.java aac278d ambari-server/src/test/resources/child_quicklinks_to_override.json 4309532 ambari-web/app/views/common/quick_view_link_view.js 5888acb ambari-web/test/views/common/quick_link_view_test.js 1998a9c contrib/management-packs/isilon-onefs-mpack/src/main/resources/addon-services/ONEFS/1.0.0/metainfo.xml 82e2022 contrib/management-packs/isilon-onefs-mpack/src/main/resources/addon-services/ONEFS/1.0.0/quicklinks/quicklinks.json PRE-CREATION Diff: https://reviews.apache.org/r/63749/diff/1/ Testing ------- - installed mpack and clicked on quicklinks existing tests: pending Thanks, Attila Magyar