You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ambari.apache.org by rl...@apache.org on 2018/08/08 14:24:36 UTC
[ambari] branch branch-2.7 updated: [AMBARI-24375] Adding services
when Kerberos is enabled incorrectly changes unrelated service
configurations
This is an automated email from the ASF dual-hosted git repository.
rlevas pushed a commit to branch branch-2.7
in repository https://gitbox.apache.org/repos/asf/ambari.git
The following commit(s) were added to refs/heads/branch-2.7 by this push:
new 0786da6 [AMBARI-24375] Adding services when Kerberos is enabled incorrectly changes unrelated service configurations
0786da6 is described below
commit 0786da646c0f226277f1de4774502c58eadbeef8
Author: Robert Levas <rl...@hortonworks.com>
AuthorDate: Sun Aug 5 12:52:33 2018 -0400
[AMBARI-24375] Adding services when Kerberos is enabled incorrectly changes unrelated service configurations
---
.../ambari/server/controller/KerberosHelperImpl.java | 19 -------------------
1 file changed, 19 deletions(-)
diff --git a/ambari-server/src/main/java/org/apache/ambari/server/controller/KerberosHelperImpl.java b/ambari-server/src/main/java/org/apache/ambari/server/controller/KerberosHelperImpl.java
index e835380..bff5640 100644
--- a/ambari-server/src/main/java/org/apache/ambari/server/controller/KerberosHelperImpl.java
+++ b/ambari-server/src/main/java/org/apache/ambari/server/controller/KerberosHelperImpl.java
@@ -752,25 +752,6 @@ public class KerberosHelperImpl implements KerberosHelper {
continue;
}
- for (Map.Entry<String, Map<String, Map<String, String>>> config : requestConfigurations.entrySet()) {
- for (Map<String, String> properties : config.getValue().values()) {
- for (Map.Entry<String, String> property : properties.entrySet()) {
- String oldValue = property.getValue();
- String updatedValue = variableReplacementHelper.replaceVariables(property.getValue(), existingConfigurations);
- if (!StringUtils.equals(oldValue, updatedValue) && !config.getKey().isEmpty()) {
- property.setValue(updatedValue);
- if (kerberosConfigurations.containsKey(config.getKey())) {
- kerberosConfigurations.get(config.getKey()).put(property.getKey(), updatedValue);
- } else {
- Map kerberosConfigProperties = new HashMap<>();
- kerberosConfigProperties.put(property.getKey(), updatedValue);
- kerberosConfigurations.put(config.getKey(), kerberosConfigProperties);
- }
- }
- }
- }
- }
-
StackAdvisorRequest request = StackAdvisorRequest.StackAdvisorRequestBuilder
.forStack(stackId.getStackName(), stackId.getStackVersion())
.forServices(services)