AMBARI-13472. KDC Credentials: Retrieve storage type along with Security Status.

ambari-web/app/controllers/global/cluster_controller.js

@@ -197,6 +197,7 @@ App.ClusterController = Em.Controller.extend(App.ReloadPopupMixin, {
 
     App.HttpClient.get(clusterUrl, App.clusterMapper, {
       complete: function (jqXHR, textStatus) {
+        App.set('isCredentialStorePersistent', Em.getWithDefault(App.Cluster.find().findProperty('clusterName', App.get('clusterName')), 'isCredentialStorePersistent', false));
       }
     }, function (jqXHR, textStatus) {
     });

ambari-web/app/controllers/main/admin/kerberos/step2_controller.js

@@ -36,10 +36,6 @@ App.KerberosWizardStep2Controller = App.WizardStep7Controller.extend(App.KDCCred
 
   addMiscTabToPage: false,
 
-  isStorePersisted: function() {
-    return this.get('wizardController.content.secureStoragePersisted');
-  }.property('wizardController.content.secureStoragePersisted'),
-
   /**
    * @type {boolean} true if test connection to hosts is in progress
    */

ambari-web/app/controllers/main/admin/kerberos/wizard_controller.js

@@ -64,8 +64,7 @@ App.KerberosWizardController = App.WizardController.extend(App.InstallComponent,
     services: [],
     advancedServiceConfig: null,
     serviceConfigProperties: [],
-    failedTask: null,
-    secureStoragePersisted: null
+    failedTask: null
   }),
 
   /**
@@ -241,13 +240,7 @@ App.KerberosWizardController = App.WizardController.extend(App.InstallComponent,
       {
         type: 'sync',
         callback: function () {
-          var self = this;
           this.loadKerberosOption();
-          if (App.get('supports.storeKDCCredentials')) {
-            credentialsUtils.isStorePersisted(App.get('clusterName')).then(function(isPersisted) {
-              self.set('content.secureStoragePersisted', isPersisted);
-            });
-          }
         }
       }
     ],

ambari-web/app/mappers/cluster_mapper.js

@@ -25,7 +25,6 @@ App.clusterMapper = App.QuickDataMapper.create({
         var result = json;
         result = this.parseIt(result, this.config);
         App.store.load(this.get('model'), result);
-        
         var cluster = App.Cluster.find(result.id);
         var clusterDesiredConfigs = [];
         // Create desired_configs_array
@@ -57,6 +56,7 @@ App.clusterMapper = App.QuickDataMapper.create({
       stack_name: 'Clusters.stack_name',
       version: 'Clusters.version',
       security_type: 'Clusters.security_type',
-      total_hosts: 'Clusters.total_hosts'
+      total_hosts: 'Clusters.total_hosts',
+      credential_store_properties: 'Clusters.credential_store_properties'
     }
 });

ambari-web/app/mixins/common/kdc_credentials_controller_mixin.js

@@ -30,13 +30,12 @@ App.KDCCredentialsControllerMixin = Em.Mixin.create({
 
   /**
    * Returns <code>true</code> if persisted secure storage available.
-   * Should be implemented as computed property.
    *
    * @type {boolean}
    */
   isStorePersisted: function() {
-    Em.assert("Should be implemented", false);
-  }.property(),
+    return App.get('isCredentialStorePersistent');
+  }.property('App.isCredentialStorePersistent'),
 
   /**
    * List of required UI-only properties needed for storing KDC credentials

ambari-web/app/models/cluster.js

@@ -18,6 +18,7 @@
 
 
 var App = require('app');
+var credentialUtils = require('utils/credentials');
 
 App.Cluster = DS.Model.extend({
   clusterName: DS.attr('string'),
@@ -25,6 +26,7 @@ App.Cluster = DS.Model.extend({
   version: DS.attr('string'),
   totalHosts:DS.attr('number'),
   securityType: DS.attr('string'),
+  credentialStoreProperties: DS.attr('object', {defaultValue: {}}),
   /**
    * Array containing desired configs. New array
    * should be set by instances of class.
@@ -33,7 +35,11 @@ App.Cluster = DS.Model.extend({
 
   isKerberosEnabled: function() {
     return this.get('securityType') === 'KERBEROS';
-  }.property('securityType')
+  }.property('securityType'),
+
+  isCredentialStorePersistent: function() {
+    return this.get('credentialStoreProperties')[credentialUtils.STORE_TYPES.PERSISTENT_PATH] === "true";
+  }.property('credentialStoreProperties')
 });
 
-App.Cluster.FIXTURES = [];
+App.Cluster.FIXTURES = [];

ambari-web/app/views/common/chart/linear_time.js

@@ -511,8 +511,12 @@ App.ChartLinearTimeView = Ember.View.extend(App.ExportMetricsMixin, {
         var avg = 0;
         var min = Number.MAX_VALUE;
         var max = Number.MIN_VALUE;
+        var numberOfNotNullValues = 0;
         for (var i = 0; i < series.data.length; i++) {
           avg += series.data[i]['y'];
+          if (series.data[i]['y'] !== null) {
+            numberOfNotNullValues++;
+          }
           if (!Em.isNone(series.data[i]['y'])) {
             if (series.data[i]['y'] < min) {
               min = series.data[i]['y'];
@@ -522,13 +526,11 @@ App.ChartLinearTimeView = Ember.View.extend(App.ExportMetricsMixin, {
             max = series.data[i]['y'];
           }
         }
-
-
         series.name = string_utils.pad(series.name.length > 36 ? series.name.substr(0, 36) + '...' : series.name, 40, '&nbsp;', 2) + '|&nbsp;' +
           string_utils.pad('min', 5, '&nbsp;', 3) +
           string_utils.pad(self.get('yAxisFormatter')(min), 12, '&nbsp;', 3) +
           string_utils.pad('avg', 5, '&nbsp;', 3) +
-          string_utils.pad(self.get('yAxisFormatter')(avg / series.data.compact().length), 12, '&nbsp;', 3) +
+          string_utils.pad(self.get('yAxisFormatter')(avg / numberOfNotNullValues), 12, '&nbsp;', 3) +
           string_utils.pad('max', 12, '&nbsp;', 3) +
           string_utils.pad(self.get('yAxisFormatter')(max), 5, '&nbsp;', 3);
       }

ambari-web/app/views/common/form/manage_credentials_form_view.js

@@ -36,7 +36,9 @@ App.ManageCredentialsFormView = Em.View.extend({
    * Status of persistent storage. Returns <code>true</code> if persistent storage is available.
    * @type {boolean}
    */
-  storePersisted: false,
+  storePersisted: function() {
+    return App.get('isCredentialStorePersistent');
+  }.property('App.isCredentialStorePersistent'),
 
   /**
    * Disable checkbox if persistent storage not available
@@ -120,11 +122,6 @@ App.ManageCredentialsFormView = Em.View.extend({
 
   prepareContent: function() {
     var self = this;
-    credentialsUtils.isStorePersisted(App.get('clusterName')).then(function(isPersisted) {
-      Em.run.next(function() {
-        self.set('storePersisted', isPersisted);
-      });
-    });
     credentialsUtils.credentials(App.get('clusterName'), function(credentials) {
       Em.run.next(function() {
         self.set('isRemovable', credentialsUtils.isKDCCredentialsPersisted(credentials));

ambari-web/app/views/common/modal_popups/invalid_KDC_popup.js

@@ -42,7 +42,9 @@ App.showInvalidKDCPopup = function (ajaxOpt, message) {
      * Status of persistent storage. Returns <code>true</code> if persistent storage is available.
      * @type {boolean}
      */
-    storePersisted: false,
+    storePersisted: function() {
+      return App.get('isCredentialStorePersistent');
+    }.property('App.isCredentialStorePersistent'),
 
     /**
      * Disable checkbox if persistent storage not available
@@ -76,17 +78,6 @@ App.showInvalidKDCPopup = function (ajaxOpt, message) {
       templateName: require('templates/common/modal_popups/invalid_KDC_popup')
     }),
 
-    willInsertElement: function() {
-      if (App.get('supports.storeKDCCredentials')) {
-        var self = this;
-        credentialsUtils.isStorePersisted(App.get('clusterName')).then(function(isPersisted) {
-          Em.run.next(function() {
-            self.set('storePersisted', isPersisted);
-          });
-        });
-      }
-    },
-
     didInsertElement: function() {
       this._super();
       App.tooltip(this.$('[rel="tooltip"]'));

ambari-web/test/mixins/common/kdc_credentials_controller_mixin_test.js

@@ -33,34 +33,6 @@ describe('App.KDCCredentialsControllerMixin', function() {
     mixedObject.destroy();
   });
 
-  describe('#isStorePersisted', function() {
-    it('should throw error if not overrided in mixed object', function() {
-      var errorThrown = false;
-      try {
-        mixedObject.get('isStorePersisted');
-      } catch (e) {
-        errorThrown = true;
-      } finally {
-        expect(errorThrown).to.be.true;
-      }
-    });
-    it('should not throw error if overrided in mixed object', function() {
-      var errorThrown = false;
-      mixedObject.reopen({
-        isStorePersisted: function() {
-          return true;
-        }.property()
-      });
-      try {
-        mixedObject.get('isStorePersisted');
-      } catch (e) {
-        errorThrown = true;
-      } finally {
-        expect(errorThrown).to.be.false;
-      }
-    });
-  });
-
   describe('#initilizeKDCStoreProperties', function() {
     [
       {

ambari-web/test/views/common/form/manage_kdc_credentials_form_test.js

@@ -69,16 +69,14 @@ describe('#App.ManageCredentialsFormView', function() {
         sinon.stub(credentialUtils, 'credentials', function(clusterName, callback) {
           callback(test.credentials);
         });
-        sinon.stub(credentialUtils, 'isStorePersisted', function() {
-          return $.Deferred().resolve(test.isStorePersistent).promise();
-        });
+        sinon.stub(App, 'get').withArgs('isCredentialStorePersistent').returns(test.e.storePersisted);
         view.prepareContent();
         Em.run.next(function() {
           assert.equal(view.get('isRemovable'), test.e.isRemovable, '#isRemovable property validation');
           assert.equal(view.get('isRemoveDisabled'), test.e.isRemoveDisabled, '#isRemoveDisabled property validation');
           assert.equal(view.get('storePersisted'), test.e.storePersisted, '#storePersisted property validation');
           credentialUtils.credentials.restore();
-          credentialUtils.isStorePersisted.restore();
+          App.get.restore();
           done();
         });
       });