You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ambari.apache.org by jo...@apache.org on 2014/09/19 19:19:14 UTC
[06/29] git commit: AMBARI-7394. Recommend properties were changed
after cluster install (srimanth)
AMBARI-7394. Recommend properties were changed after cluster install (srimanth)
Project: http://git-wip-us.apache.org/repos/asf/ambari/repo
Commit: http://git-wip-us.apache.org/repos/asf/ambari/commit/bf5f1c6d
Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/bf5f1c6d
Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/bf5f1c6d
Branch: refs/heads/branch-alerts-dev
Commit: bf5f1c6da72ed46844298a152906fdd77c1fba28
Parents: 1498061
Author: Srimanth Gunturi <sg...@hortonworks.com>
Authored: Thu Sep 18 14:47:58 2014 -0700
Committer: Srimanth Gunturi <sg...@hortonworks.com>
Committed: Thu Sep 18 14:48:03 2014 -0700
----------------------------------------------------------------------
ambari-web/app/mixins/common/serverValidator.js | 11 ++++++++++-
1 file changed, 10 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/ambari/blob/bf5f1c6d/ambari-web/app/mixins/common/serverValidator.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/mixins/common/serverValidator.js b/ambari-web/app/mixins/common/serverValidator.js
index 8d551f8..2a31a4f 100644
--- a/ambari-web/app/mixins/common/serverValidator.js
+++ b/ambari-web/app/mixins/common/serverValidator.js
@@ -201,13 +201,22 @@ App.ServerValidatorMixin = Em.Mixin.create({
var self = this;
var recommendations = this.get('hostGroups');
recommendations.blueprint.configurations = blueprintUtils.buildConfisJSON(this.get('services'), this.get('stepConfigs'));
+
+ var serviceNames = this.get('serviceNames');
+ if (!self.get('isInstaller')) {
+ // When editing a service we validate only that service's configs.
+ // However, we should pass the IDs of services installed, or else,
+ // default value calculations will alter.
+ serviceNames = App.Service.find().mapProperty('serviceName');
+ }
+
return App.ajax.send({
name: 'config.validations',
sender: this,
data: {
stackVersionUrl: App.get('stackVersionURL'),
hosts: this.get('hostNames'),
- services: this.get('serviceNames'),
+ services: serviceNames,
validate: 'configurations',
recommendations: recommendations
},