You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ambari.apache.org by ab...@apache.org on 2015/06/22 17:15:28 UTC
[1/3] ambari git commit: AMBARI-12066 Service Configs: config groups
list is not updated. (ababiichuk)
Repository: ambari
Updated Branches:
refs/heads/branch-2.1 f3560893e -> c4d2e839b
AMBARI-12066 Service Configs: config groups list is not updated. (ababiichuk)
Project: http://git-wip-us.apache.org/repos/asf/ambari/repo
Commit: http://git-wip-us.apache.org/repos/asf/ambari/commit/36cc0af0
Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/36cc0af0
Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/36cc0af0
Branch: refs/heads/branch-2.1
Commit: 36cc0af08e18c57f7f941ba168beafc0e93b3033
Parents: f356089
Author: aBabiichuk <ab...@cybervisiontech.com>
Authored: Mon Jun 22 18:03:10 2015 +0300
Committer: aBabiichuk <ab...@cybervisiontech.com>
Committed: Mon Jun 22 18:14:53 2015 +0300
----------------------------------------------------------------------
.../service/manage_config_groups_controller.js | 4 ++
.../app/mappers/configs/config_groups_mapper.js | 53 +++++++++++---------
.../app/mixins/common/configs/configs_loader.js | 4 +-
.../main/service/configs/config_overridable.js | 2 +-
.../app/mixins/main/service/groups_mapping.js | 6 +--
5 files changed, 37 insertions(+), 32 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/ambari/blob/36cc0af0/ambari-web/app/controllers/main/service/manage_config_groups_controller.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/controllers/main/service/manage_config_groups_controller.js b/ambari-web/app/controllers/main/service/manage_config_groups_controller.js
index 0e66270..cdc2c6b 100644
--- a/ambari-web/app/controllers/main/service/manage_config_groups_controller.js
+++ b/ambari-web/app/controllers/main/service/manage_config_groups_controller.js
@@ -532,6 +532,10 @@ App.ManageConfigGroupsController = Em.Controller.extend(App.ConfigOverridable, {
this.set('selectedHosts', selectedConfigGroup.get('hosts'));
this.deleteHosts();
this.get('configGroups').removeObject(selectedConfigGroup);
+ var groupFromModel = App.ServiceConfigGroup.find().filterProperty('serviceName', selectedConfigGroup.get('service.serviceName')).findProperty('name', selectedConfigGroup.get('name'));
+ if (groupFromModel) {
+ App.configGroupsMapper.deleteRecord(groupFromModel);
+ }
this.set('selectedConfigGroup', this.get('configGroups').findProperty('isDefault'));
},
http://git-wip-us.apache.org/repos/asf/ambari/blob/36cc0af0/ambari-web/app/mappers/configs/config_groups_mapper.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/mappers/configs/config_groups_mapper.js b/ambari-web/app/mappers/configs/config_groups_mapper.js
index 55b1185..ae6d049 100644
--- a/ambari-web/app/mappers/configs/config_groups_mapper.js
+++ b/ambari-web/app/mappers/configs/config_groups_mapper.js
@@ -47,7 +47,7 @@ App.configGroupsMapper = App.QuickDataMapper.create({
},
- map: function (json, mapFromVersions, serviceNames, skipDefault) {
+ map: function (json, mapFromVersions, serviceNames) {
if (serviceNames && serviceNames.length > 0) {
var configGroups = [];
@@ -61,7 +61,7 @@ App.configGroupsMapper = App.QuickDataMapper.create({
var hostNamesForService = {};
if (json && json.items) {
- json.items.forEach(function(configGroup) {
+ json.items.forEach(function (configGroup) {
if (configGroup.group_name != 'default') {
if (mapFromVersions) {
configGroup.id = configGroup.service_name + configGroup.group_id;
@@ -71,40 +71,43 @@ App.configGroupsMapper = App.QuickDataMapper.create({
configGroup.service_name = configGroup.ConfigGroup.tag;
}
- if (!skipDefault) {
- /**
- * creating (if not exists) field in <code>hostNamesForService<code> with host names for default group
- */
- if (!hostNamesForService[configGroup.service_name]) {
- hostNamesForService[configGroup.service_name] = $.merge([], App.get('allHostNames'));
- }
-
- /**
- * excluding host names that belongs for current config group from default group
- */
- configGroup.hosts.forEach(function(host) {
- hostNamesForService[configGroup.service_name].splice(hostNamesForService[configGroup.service_name].indexOf(host), 1);
- });
+ /**
+ * creating (if not exists) field in <code>hostNamesForService<code> with host names for default group
+ */
+ if (!hostNamesForService[configGroup.service_name]) {
+ hostNamesForService[configGroup.service_name] = $.merge([], App.get('allHostNames'));
}
+ /**
+ * excluding host names that belongs for current config group from default group
+ */
+ configGroup.hosts.forEach(function (host) {
+ hostNamesForService[configGroup.service_name].splice(hostNamesForService[configGroup.service_name].indexOf(host), 1);
+ });
var template = mapFromVersions ? this.get('config2') : this.get('config');
configGroups.push(this.parseIt(configGroup, template));
}
}, this);
}
- if (!skipDefault) {
- /**
- * generating default config groups
- */
- serviceNames.forEach(function(serviceName) {
+
+ /**
+ * generating default config groups
+ */
+ serviceNames.forEach(function (serviceName) {
+ var defaultGroup = App.ServiceConfigGroup.find().findProperty('id', serviceName + "0");
+ if (!defaultGroup) {
configGroups.push(this.generateDefaultGroup(serviceName, hostNamesForService[serviceName]));
- }, this);
- }
+ } else {
+ defaultGroup.set('hostNames', hostNamesForService[serviceName] || App.get('allHostNames'));
+ }
+ }, this);
+
configGroups.sort(function (configGroupA, configGroupB) {
return configGroupA.config_group_id == -1 || (configGroupA.name > configGroupB.name);
});
App.store.loadMany(this.get('model'), configGroups);
+ App.store.commit();
}
},
@@ -114,14 +117,14 @@ App.configGroupsMapper = App.QuickDataMapper.create({
* @param {string[]} [hostNames=null]
* @returns {{id: string, config_group_id: string, name: string, service_name: string, description: string, host_names: [string], service_id: string}}
*/
- generateDefaultGroup: function(serviceName, hostNames) {
+ generateDefaultGroup: function (serviceName, hostNames) {
var displayName = App.StackService.find(serviceName).get('displayName');
return {
id: serviceName + '0',
config_group_id: '-1',
name: displayName + ' Default',
service_name: serviceName,
- description: 'Default cluster level '+ displayName +' configuration',
+ description: 'Default cluster level ' + displayName + ' configuration',
host_names: hostNames ? hostNames : App.get('allHostNames'),
service_id: serviceName
}
http://git-wip-us.apache.org/repos/asf/ambari/blob/36cc0af0/ambari-web/app/mixins/common/configs/configs_loader.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/mixins/common/configs/configs_loader.js b/ambari-web/app/mixins/common/configs/configs_loader.js
index 6b07d80..9f94fd9 100644
--- a/ambari-web/app/mixins/common/configs/configs_loader.js
+++ b/ambari-web/app/mixins/common/configs/configs_loader.js
@@ -79,7 +79,7 @@ App.ConfigsLoader = Em.Mixin.create(App.GroupsMappingMixin, {
if (this.get('preSelectedConfigVersion')) {
/** handling redirecting from config history page **/
var self = this;
- this.loadConfigGroups(this.get('servicesToLoad'), false).done(function() {
+ this.loadConfigGroups(this.get('servicesToLoad')).done(function() {
var selectedGroup = App.ServiceConfigGroup.find().find(function(g) {
return g.get('serviceName') == self.get('preSelectedConfigVersion.serviceName')
&& (g.get('name') == self.get('preSelectedConfigVersion.groupName') || (self.get('preSelectedConfigVersion.groupName') == 'default' && g.get('isDefault')));
@@ -121,7 +121,7 @@ App.ConfigsLoader = Em.Mixin.create(App.GroupsMappingMixin, {
App.configGroupsMapper.map(data, true, params.serviceNames.split(','));
this.set('selectedConfigGroup', App.ServiceConfigGroup.find().filterProperty('serviceName', this.get('content.serviceName')).findProperty('isDefault'));
this.parseConfigData(data);
- this.loadConfigGroups(params.serviceNames.split(','), true);
+ this.loadConfigGroups(params.serviceNames.split(','));
},
/**
http://git-wip-us.apache.org/repos/asf/ambari/blob/36cc0af0/ambari-web/app/mixins/main/service/configs/config_overridable.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/mixins/main/service/configs/config_overridable.js b/ambari-web/app/mixins/main/service/configs/config_overridable.js
index 61eeb3d..a93f2e9 100644
--- a/ambari-web/app/mixins/main/service/configs/config_overridable.js
+++ b/ambari-web/app/mixins/main/service/configs/config_overridable.js
@@ -445,7 +445,7 @@ App.ConfigOverridable = Em.Mixin.create({
}),
onPrimary:function() {
if (self.get('controller.name') == 'mainServiceInfoConfigsController') {
- self.get('controller').loadConfigGroups([self.get('content.serviceName')], true).done(function() {
+ self.get('controller').loadConfigGroups([self.get('content.serviceName')]).done(function() {
var group = App.ServiceConfigGroup.find().find(function(g) {
return g.get('serviceName') == self.get('content.serviceName') && g.get('name') == groupName;
});
http://git-wip-us.apache.org/repos/asf/ambari/blob/36cc0af0/ambari-web/app/mixins/main/service/groups_mapping.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/mixins/main/service/groups_mapping.js b/ambari-web/app/mixins/main/service/groups_mapping.js
index dbbf5ed..93a760d 100644
--- a/ambari-web/app/mixins/main/service/groups_mapping.js
+++ b/ambari-web/app/mixins/main/service/groups_mapping.js
@@ -28,11 +28,10 @@ App.GroupsMappingMixin = Em.Mixin.create({
/**
* Load config groups
* @param {String[]} serviceNames
- * @param skipDefault
* @returns {$.Deferred()}
* @method loadConfigGroups
*/
- loadConfigGroups: function (serviceNames, skipDefault) {
+ loadConfigGroups: function (serviceNames) {
var dfd = $.Deferred();
if (!serviceNames || serviceNames.length === 0) {
dfd.resolve();
@@ -42,7 +41,6 @@ App.GroupsMappingMixin = Em.Mixin.create({
sender: this,
data: {
serviceNames: serviceNames.join(','),
- skipDefault: skipDefault,
dfd: dfd
},
success: 'saveConfigGroupsToModel'
@@ -59,7 +57,7 @@ App.GroupsMappingMixin = Em.Mixin.create({
* @method saveConfigGroupsToModel
*/
saveConfigGroupsToModel: function (data, opt, params) {
- App.configGroupsMapper.map(data, false, params.serviceNames.split(','), params.skipDefault);
+ App.configGroupsMapper.map(data, false, params.serviceNames.split(','));
params.dfd.resolve();
}
[3/3] ambari git commit: AMBARI-12068 Set Recommended button is shown
for disabled directory-type configs. (ababiichuk)
Posted by ab...@apache.org.
AMBARI-12068 Set Recommended button is shown for disabled directory-type configs. (ababiichuk)
Project: http://git-wip-us.apache.org/repos/asf/ambari/repo
Commit: http://git-wip-us.apache.org/repos/asf/ambari/commit/c4d2e839
Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/c4d2e839
Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/c4d2e839
Branch: refs/heads/branch-2.1
Commit: c4d2e839be44aefa5cc56f22bb37e339345d2973
Parents: 5545881
Author: aBabiichuk <ab...@cybervisiontech.com>
Authored: Mon Jun 22 18:02:50 2015 +0300
Committer: aBabiichuk <ab...@cybervisiontech.com>
Committed: Mon Jun 22 18:15:15 2015 +0300
----------------------------------------------------------------------
.../views/common/configs/widgets/directory_config_widget_view.js | 4 ++++
1 file changed, 4 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/ambari/blob/c4d2e839/ambari-web/app/views/common/configs/widgets/directory_config_widget_view.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/views/common/configs/widgets/directory_config_widget_view.js b/ambari-web/app/views/common/configs/widgets/directory_config_widget_view.js
index 3f132ea..04af5ff 100644
--- a/ambari-web/app/views/common/configs/widgets/directory_config_widget_view.js
+++ b/ambari-web/app/views/common/configs/widgets/directory_config_widget_view.js
@@ -24,6 +24,10 @@ App.DirectoryConfigWidgetView = App.ConfigWidgetView.extend({
templateName: require('templates/common/configs/widgets/directory_config_widget'),
classNames: ['widget-config', 'directory-widget'],
+ disabled: function() {
+ return !this.get('config.isEditable');
+ }.property('config.isEditable'),
+
/**
* Control to edit value.
*
[2/3] ambari git commit: AMBARI-12067 Changed configs popup is not
shown for YARN and MR2. (ababiichuk)
Posted by ab...@apache.org.
AMBARI-12067 Changed configs popup is not shown for YARN and MR2. (ababiichuk)
Project: http://git-wip-us.apache.org/repos/asf/ambari/repo
Commit: http://git-wip-us.apache.org/repos/asf/ambari/commit/55458810
Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/55458810
Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/55458810
Branch: refs/heads/branch-2.1
Commit: 5545881005c83c2a372f5d631345e6ba8b177812
Parents: 36cc0af
Author: aBabiichuk <ab...@cybervisiontech.com>
Authored: Mon Jun 22 17:59:41 2015 +0300
Committer: aBabiichuk <ab...@cybervisiontech.com>
Committed: Mon Jun 22 18:15:05 2015 +0300
----------------------------------------------------------------------
ambari-web/app/controllers/main/service/info/configs.js | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/ambari/blob/55458810/ambari-web/app/controllers/main/service/info/configs.js
----------------------------------------------------------------------
diff --git a/ambari-web/app/controllers/main/service/info/configs.js b/ambari-web/app/controllers/main/service/info/configs.js
index 1121507..d3788a0 100644
--- a/ambari-web/app/controllers/main/service/info/configs.js
+++ b/ambari-web/app/controllers/main/service/info/configs.js
@@ -325,7 +325,7 @@ App.MainServiceInfoConfigsController = Em.Controller.extend(App.ConfigsLoader, A
return null;
}
var hash = {};
- this.get('stepConfigs')[0].configs.forEach(function (config) {
+ this.get('selectedService.configs').forEach(function (config) {
hash[config.get('name')] = {value: config.get('value'), overrides: [], isFinal: config.get('isFinal')};
if (!config.get('overrides')) return;
if (!config.get('overrides.length')) return;