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 <sgunt...@hortonworks.com> Authored: Thu Sep 18 14:47:58 2014 -0700 Committer: Srimanth Gunturi <sgunt...@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 },