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,

Reply via email to