You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ambari.apache.org by di...@apache.org on 2017/04/13 21:01:26 UTC

ambari git commit: AMBARI-20697 Stack advisor code in 2.3 refactor logic for ranger Kafka Plugin (Bharat Viswanadham via dili)

Repository: ambari
Updated Branches:
  refs/heads/trunk 14c1ffd32 -> cfde36c07


AMBARI-20697 Stack advisor code in 2.3 refactor logic for ranger Kafka Plugin (Bharat Viswanadham via dili)


Project: http://git-wip-us.apache.org/repos/asf/ambari/repo
Commit: http://git-wip-us.apache.org/repos/asf/ambari/commit/cfde36c0
Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/cfde36c0
Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/cfde36c0

Branch: refs/heads/trunk
Commit: cfde36c077de31dda0802f983602f680afcbce29
Parents: 14c1ffd
Author: Di Li <di...@apache.org>
Authored: Thu Apr 13 17:00:56 2017 -0400
Committer: Di Li <di...@apache.org>
Committed: Thu Apr 13 17:00:56 2017 -0400

----------------------------------------------------------------------
 .../main/resources/stacks/HDP/2.3/services/stack_advisor.py    | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/ambari/blob/cfde36c0/ambari-server/src/main/resources/stacks/HDP/2.3/services/stack_advisor.py
----------------------------------------------------------------------
diff --git a/ambari-server/src/main/resources/stacks/HDP/2.3/services/stack_advisor.py b/ambari-server/src/main/resources/stacks/HDP/2.3/services/stack_advisor.py
index 67532c5..eb7389d 100644
--- a/ambari-server/src/main/resources/stacks/HDP/2.3/services/stack_advisor.py
+++ b/ambari-server/src/main/resources/stacks/HDP/2.3/services/stack_advisor.py
@@ -349,10 +349,10 @@ class HDP23StackAdvisor(HDP22StackAdvisor):
       ranger_kafka_plugin_enabled = services["configurations"]["ranger-env"]["properties"]["ranger-kafka-plugin-enabled"]
       putKafkaRangerPluginProperty("ranger-kafka-plugin-enabled", ranger_kafka_plugin_enabled)
 
-    # Determine if the Ranger/Kafka Plugin is enabled
-    ranger_plugin_enabled = "RANGER" in servicesList
+
+    ranger_plugin_enabled = False
     # Only if the RANGER service is installed....
-    if ranger_plugin_enabled:
+    if "RANGER" in servicesList:
       # If ranger-kafka-plugin-properties/ranger-kafka-plugin-enabled,
       # determine if the Ranger/Kafka plug-in enabled enabled or not
       if 'ranger-kafka-plugin-properties' in configurations and \