AMBARI-20693. Principal suffix field should be a non mandatory field in configure identities page of Enable kerberos wizard (akovalenko)
Project: http://git-wip-us.apache.org/repos/asf/ambari/repo Commit: http://git-wip-us.apache.org/repos/asf/ambari/commit/65d2f3f6 Tree: http://git-wip-us.apache.org/repos/asf/ambari/tree/65d2f3f6 Diff: http://git-wip-us.apache.org/repos/asf/ambari/diff/65d2f3f6 Branch: refs/heads/branch-feature-AMBARI-12556 Commit: 65d2f3f6d29f05213b2e6469f360f8fc3ff3f397 Parents: 9289481 Author: Aleksandr Kovalenko <[email protected]> Authored: Thu Apr 6 20:17:55 2017 +0300 Committer: Aleksandr Kovalenko <[email protected]> Committed: Thu Apr 6 20:17:55 2017 +0300 ---------------------------------------------------------------------- .../app/data/configs/wizards/kerberos_descriptor_properties.js | 5 +++++ ambari-web/app/mixins/wizard/addSecurityConfigs.js | 2 +- 2 files changed, 6 insertions(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/ambari/blob/65d2f3f6/ambari-web/app/data/configs/wizards/kerberos_descriptor_properties.js ---------------------------------------------------------------------- diff --git a/ambari-web/app/data/configs/wizards/kerberos_descriptor_properties.js b/ambari-web/app/data/configs/wizards/kerberos_descriptor_properties.js index f6e4239..a6ecf20 100644 --- a/ambari-web/app/data/configs/wizards/kerberos_descriptor_properties.js +++ b/ambari-web/app/data/configs/wizards/kerberos_descriptor_properties.js @@ -29,6 +29,11 @@ var properties = [ "name": "additional_realms", "index": 5, "placeholderText": "(" + Em.I18n.t("common.optional") + ")" + }, + { + "name": "principal_suffix", + "index": 6, + "placeholderText": "(" + Em.I18n.t("common.optional") + ")" } ]; http://git-wip-us.apache.org/repos/asf/ambari/blob/65d2f3f6/ambari-web/app/mixins/wizard/addSecurityConfigs.js ---------------------------------------------------------------------- diff --git a/ambari-web/app/mixins/wizard/addSecurityConfigs.js b/ambari-web/app/mixins/wizard/addSecurityConfigs.js index e8da47f..89d3cf7 100644 --- a/ambari-web/app/mixins/wizard/addSecurityConfigs.js +++ b/ambari-web/app/mixins/wizard/addSecurityConfigs.js @@ -204,7 +204,7 @@ App.AddSecurityConfigs = Em.Mixin.create({ for (var propertyName in kerberosProperties) { var predefinedProperty = this.get('kerberosDescriptorProperties').findProperty('name', propertyName); var value = kerberosProperties[propertyName]; - var isRequired = propertyName == 'additional_realms' ? false : value !== ""; + var isRequired = ['additional_realms', 'principal_suffix'].contains(propertyName) ? false : value !== ""; var propertyObject = { name: propertyName, value: value,
