-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56418/
-----------------------------------------------------------
(Updated Feb. 9, 2017, 11:14 p.m.)
Review request for Ambari, Alejandro Fernandez, Jayush Luniya, Robert
Nettleton, and Sumit Mohanty.
Changes
-------
Incorporating Review Comments - performance improvement, unit tests
Bugs: AMBARI-19909
https://issues.apache.org/jira/browse/AMBARI-19909
Repository: ambari
Description
-------
Export Blueprints does not contain the settings object and hence the credential
store values
Diffs (updated)
-----
ambari-server/src/main/java/org/apache/ambari/server/api/query/render/ClusterBlueprintRenderer.java
1a9ea91
ambari-server/src/test/java/org/apache/ambari/server/api/query/render/ClusterBlueprintRendererTest.java
1fe48df
Diff: https://reviews.apache.org/r/56418/diff/
Testing
-------
Issued
http://c6401.ambari.apache.org:8080/api/v1/clusters/credential?format=blueprint
command to ensure that the settings object is present.
Resultant blueprint has the following object added:
"settings" : [
{
"recovery_settings" : [
{
"recovery_enabled" : "true"
}
]
},
{
"service_settings" : [
{
"name" : "OOZIE",
"credential_store_enabled" : "true"
},
{
"recovery_enabled" : "true",
"name" : "HIVE",
"credential_store_enabled" : "true"
},
{
"recovery_enabled" : "true",
"name" : "HDFS"
},
{
"recovery_enabled" : "true",
"name" : "AMBARI_METRICS"
}
]
},
{
"component_settings" : [
{
"recovery_enabled" : "true",
"name" : "METRICS_COLLECTOR"
},
{
"recovery_enabled" : "true",
"name" : "SECONDARY_NAMENODE"
},
{
"recovery_enabled" : "true",
"name" : "WEBHCAT_SERVER"
}
]
}
]
Thanks,
Madhuvanthi Radhakrishnan
