This is an automated email from the ASF dual-hosted git repository. atkach pushed a commit to branch trunk in repository https://gitbox.apache.org/repos/asf/ambari.git
The following commit(s) were added to refs/heads/trunk by this push: new f4bd455 AMBARI-24921 If service does not have client service component created then "Run Service Check" option should be made hidden f4bd455 is described below commit f4bd455ed5eb67df592169201455e95c614a69d6 Author: Andrii Tkach <atk...@apache.org> AuthorDate: Mon Nov 19 13:22:22 2018 +0200 AMBARI-24921 If service does not have client service component created then "Run Service Check" option should be made hidden --- ambari-web/app/views/main/service/item.js | 3 ++- ambari-web/test/views/main/service/item_test.js | 1 + 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/ambari-web/app/views/main/service/item.js b/ambari-web/app/views/main/service/item.js index 0186b86..430d62a 100644 --- a/ambari-web/app/views/main/service/item.js +++ b/ambari-web/app/views/main/service/item.js @@ -158,7 +158,8 @@ App.MainServiceItemView = Em.View.extend(App.HiveInteractiveCheck, { var allMasters = service.get('hostComponents').filterProperty('isMaster').mapProperty('componentName').uniq(); var allSlaves = service.get('slaveComponents').rejectProperty('totalCount', 0).mapProperty('componentName'); var actionMap = App.HostComponentActionMap.getMap(this); - var serviceCheckSupported = App.get('services.supportsServiceCheck').contains(service.get('serviceName')); + var serviceCheckSupported = App.get('services.supportsServiceCheck').contains(service.get('serviceName')) + && service.get('installedClients') > 0; var hasConfigTab = this.get('hasConfigTab'); var excludedCommands = this.get('mastersExcludedCommands'); var serviceName = service.get('serviceName'); diff --git a/ambari-web/test/views/main/service/item_test.js b/ambari-web/test/views/main/service/item_test.js index 6d91e3a..4071dc3 100644 --- a/ambari-web/test/views/main/service/item_test.js +++ b/ambari-web/test/views/main/service/item_test.js @@ -563,6 +563,7 @@ describe('App.MainServiceItemView', function () { serviceName: testCase.serviceName, displayName: testCase.displayName, serviceTypes: testCase.serviceTypes, + installedClients: 1, passiveState: 'OFF' }), isSeveralClients: false,