ambari-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From akovale...@apache.org
Subject ambari git commit: AMBARI-20693. Principal suffix field should be a non mandatory field in configure identities page of Enable kerberos wizard (akovalenko)
Date Thu, 06 Apr 2017 17:18:02 GMT
Repository: ambari
Updated Branches:
  refs/heads/trunk 92894811e -> 65d2f3f6d


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/trunk
Commit: 65d2f3f6d29f05213b2e6469f360f8fc3ff3f397
Parents: 9289481
Author: Aleksandr Kovalenko <akovalenko@hortonworks.com>
Authored: Thu Apr 6 20:17:55 2017 +0300
Committer: Aleksandr Kovalenko <akovalenko@hortonworks.com>
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,


Mime
View raw message